You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org> on 2012/02/08 20:53:01 UTC
[jira] [Commented] (FLUME-958) Miscellaneous build improvements
[ https://issues.apache.org/jira/browse/FLUME-958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13203904#comment-13203904 ]
jiraposter@reviews.apache.org commented on FLUME-958:
-----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3809/
-----------------------------------------------------------
Review request for Flume.
Summary
-------
1. Update the SCM URL to have the correct prefix per http://maven.apache.org/scm/subversion.html
2. Generate aggregated javadocs (across the sub-modules) at package time
3. Run Rat at test time instead of verify time so that it will always run (typically, you would only run the verify phase if you did an mvn install or deploy)
This addresses bug FLUME-958.
https://issues.apache.org/jira/browse/FLUME-958
Diffs
-----
pom.xml e4b40ab
Diff: https://reviews.apache.org/r/3809/diff
Testing
-------
Tested this patch on pristine source + FLUME-957.
1. SCM URL change: "mvn site" no longer fails (note however that the generated site is not used by the Flume project)
2. Javadocs: aggregated javadocs are generated by "mvn package"
3. Rat: "mvn package" leaves rat.txt files in the respective target/ subdirs of the sub-modules, evidence that it has run. Note that it succeeds only if the FLUME-957 patch has been applied.
Thanks,
Mike
> Miscellaneous build improvements
> --------------------------------
>
> Key: FLUME-958
> URL: https://issues.apache.org/jira/browse/FLUME-958
> Project: Flume
> Issue Type: Improvement
> Components: Build
> Affects Versions: v1.0.0
> Reporter: Mike Percy
> Assignee: Mike Percy
> Priority: Minor
> Fix For: v1.1.0
>
>
> There are a few things that could be improved with the build:
> 1. Add correct prefixes to the SCM URLs in the POM
> 2. Run Rat in the test phase instead of the verify phase (verify is not run when "mvn package" is executed) so we don't have to deal with last minute license issues at release time
> 3. Generate aggregated javadocs for the multi-module project for use on the maven site
> It seems to make more sense to lump these smaller issues into one jira, especially since only the top-level pom.xml needs to be patched.
--
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