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

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} Docker mode activated. {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:green}+1{color} | {color:green} shellcheck {color} | {color:green} 0m 
0s {color} | {color:green} There were no new shellcheck issues. {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 
0s {color} | {color:green} Patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 17m 30s {color} 
| {color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.9.1 Server=1.9.1 Image:yetus/yetus:577e74f |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12776553/YETUS-221.00.patch |
| JIRA Issue | YETUS-221 |
| Optional Tests |  shellcheck  |
| uname | Linux 2e8389965d8f 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed 
Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | nobuild |
| git revision | master / 76653b4 |
| shellcheck | v0.4.1 |
| modules | C:  U:  |
| Powered by | Apache Yetus    http://yetus.apache.org |
| Console output | 
https://builds.apache.org/job/PreCommit-YETUS-Build/156/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
>
>
> 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