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

[email protected] commented on FLUME-1122:
------------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4859/#review7192
-----------------------------------------------------------



flume-ng-dist/src/main/assembly/dist.xml
<https://reviews.apache.org/r/4859/#comment15890>

    Suggest calling it just docs



pom.xml
<https://reviews.apache.org/r/4859/#comment15892>

    Suggest you change it to just docs unless that causes some other problems.


- Arvind


On 2012-04-24 20:47:09, Hari Shreedharan wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4859/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-04-24 20:47:09)
bq.  
bq.  
bq.  Review request for Flume and Arvind Prabhakar.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Flume docs now staged under target/flume-docs. Dist build moves it to top 
level.
bq.  
bq.  
bq.  This addresses bug FLUME-1122.
bq.      https://issues.apache.org/jira/browse/FLUME-1122
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    flume-ng-dist/src/main/assembly/dist.xml 6a84922 
bq.    pom.xml 281f8b8 
bq.  
bq.  Diff: https://reviews.apache.org/r/4859/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Ran a full build. No top level flume-docs directory created. Flume-docs 
directory created under target. Also, untar the dist tarball, top level 
flume-docs directory found with same content as top level target/flume-docs.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Hari
bq.  
bq.


                
> Flume documentation layout should be refactored
> -----------------------------------------------
>
>                 Key: FLUME-1122
>                 URL: https://issues.apache.org/jira/browse/FLUME-1122
>             Project: Flume
>          Issue Type: Improvement
>            Reporter: Arvind Prabhakar
>            Assignee: Hari Shreedharan
>         Attachments: FLUME-1122.patch
>
>
> Currently a full build of Flume generates documentation in the top-level 
> flume-docs directory. This causes a lot of flume-docs directories to show up 
> as untracked files in the version control after build:
> {noformat}
> $ svn stat
> ?       flume-docs
> ?       flume-ng-sdk/flume-docs
> ?       flume-ng-node/flume-docs
> ?       flume-ng-legacy-sources/flume-docs
> ?       flume-ng-legacy-sources/flume-thrift-source/flume-docs
> ?       flume-ng-legacy-sources/flume-avro-source/flume-docs
> ?       flume-ng-sinks/flume-docs
> ?       flume-ng-sinks/flume-irc-sink/flume-docs
> ?       flume-ng-sinks/flume-hdfs-sink/flume-docs
> ?       flume-ng-core/flume-docs
> ?       flume-ng-channels/flume-docs
> ?       flume-ng-channels/flume-file-channel/flume-docs
> ?       flume-ng-channels/flume-recoverable-memory-channel/flume-docs
> ?       flume-ng-channels/flume-jdbc-channel/flume-docs
> ?       flume-ng-clients/flume-docs
> ?       flume-ng-clients/flume-ng-log4jappender/flume-docs
> ?       flume-ng-dist/flume-docs
> $
> {noformat}
> It would be better if the documentation was generated within the target 
> direcatory (target/docs), and promoted to the top-level during the build of 
> flume-ng-dist module. That way only the generated distribution will contain 
> docs at the top level and not the rest of the workspace.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to