[
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)