You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "Ryan Blue (JIRA)" <ji...@apache.org> on 2016/01/12 00:39:39 UTC

[jira] [Commented] (AVRO-1779) Avro docs convenience artifact missing LICENSE/NOTICE

    [ https://issues.apache.org/jira/browse/AVRO-1779?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15092946#comment-15092946 ] 

Ryan Blue commented on AVRO-1779:
---------------------------------

I'm attaching a patch that fixes the build script to include LICENSE and NOTICE from AVRO-1771. The problem was that the tar command was run from build/ but the copy used the same path. That caused the commands to copy the files to avro-doc-VERS instead of into build/avro-doc-VERS/. I ran dist and verified that the files are included, which I evidently failed to do for AVRO-1771.

The attached diff also fixes a problem I ran into while testing this. The C++ build would halt dist because the "doc" folder was missing. Evidently, the default output directory for my version of doxygen is . but for others it must be doc. Since the output directory is expected to be doc, I added it to the Doxyfile. I confirmed that this produces the same documentation structure as the last RC in my local copy.

> Avro docs convenience artifact missing LICENSE/NOTICE
> -----------------------------------------------------
>
>                 Key: AVRO-1779
>                 URL: https://issues.apache.org/jira/browse/AVRO-1779
>             Project: Avro
>          Issue Type: Bug
>          Components: doc
>    Affects Versions: 1.8.0
>            Reporter: Sean Busbey
>            Assignee: Ryan Blue
>            Priority: Blocker
>             Fix For: 1.8.0
>
>         Attachments: AVRO-1779.1.diff
>
>
> for releases we generate a convenience artifact with our docs. at present, this tarball is missing our needed LICENSE/NOTICE files



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)