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

Allen Wittenauer commented on YETUS-868:
----------------------------------------

OK, I've tracked down the failure in whitespace to actually being how 
inconsistent test-patch is with putting ./ in front of things vs. what the 
excludes file has in it.  So as part of this, I think it's inevitable that the 
code needs to make a decision and stick with it.  Given that diff itself does 
not generate a path leader of ./, test-patch should follow the same rules. 

> Patch modifying the excludes file should force a re-read
> --------------------------------------------------------
>
>                 Key: YETUS-868
>                 URL: https://issues.apache.org/jira/browse/YETUS-868
>             Project: Yetus
>          Issue Type: Bug
>          Components: Precommit
>    Affects Versions: 0.10.0
>            Reporter: Allen Wittenauer
>            Assignee: Allen Wittenauer
>            Priority: Critical
>             Fix For: 0.11.0
>
>         Attachments: YETUS-868.00.patch
>
>
> If a patch modifies the file that holds the exclusion data, it should get 
> re-read and re-processed after it has been applied.  Currently, it is only 
> read at launch during determine_needed_tests.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to