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

Yetus QA commented on YETUS-221:
--------------------------------

| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s 
{color} | {color:blue} Precommit patch detected. {color} |
| {color:blue}0{color} | {color:blue} docker {color} | {color:blue} 0m 5s 
{color} | {color:blue} Docker command 'docker' not found/broken. Disabling 
docker. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s 
{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:blue}0{color} | {color:blue} shellcheck {color} | {color:blue} 0m 1s 
{color} | {color:blue} Shellcheck was not available. {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 
0s {color} | {color:green} Patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 
5s {color} | {color:green} Patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 0m 15s {color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12776733/YETUS-221.01.patch |
| JIRA Issue | YETUS-221 |
| Optional Tests |  asflicense  shellcheck  |
| uname | Linux jenkins-ubuntu-1404-4gb-146 3.13.0-30-generic #54-Ubuntu SMP 
Mon Jun 9 22:45:01 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | nobuild |
| git revision | master / 76653b4 |
| modules | C:  U:  |
| Powered by | Apache Yetus    http://yetus.apache.org |
| Console output | 
https://builds.apache.org/job/PreCommit-YETUS-Build/158/console |


This message was automatically generated.



> asflicense check exits multi-module Maven build early if there are 
> pre-existing license problems.
> -------------------------------------------------------------------------------------------------
>
>                 Key: YETUS-221
>                 URL: https://issues.apache.org/jira/browse/YETUS-221
>             Project: Yetus
>          Issue Type: Bug
>          Components: Test Patch
>            Reporter: Chris Nauroth
>            Assignee: Allen Wittenauer
>         Attachments: YETUS-221.00.patch, YETUS-221.01.patch
>
>
> The asflicense check runs {{mvn apache-rat:check}} from the root of the 
> project (unless told otherwise).  For a multi-module build, the build exits 
> with failure after the first occurrence of a module with a license problem.  
> It won't check subsequent modules.  If the patch in question is touching one 
> of those subsequent modules, then asflicense won't check it.  A reviewer 
> might incorrectly conclude that the patch is fine, because the pre-existing 
> license problems are unrelated.



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

Reply via email to