[ https://issues.apache.org/jira/browse/YETUS-1202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Allen Wittenauer updated YETUS-1202: ------------------------------------ Description: The core problem is that in patch mode, CHANGED_FILES is based upon the patch file without any extra processing. All files that are touched in some way are sent to filefilter in order to trigger the appropriate tests. The problem comes in that files that are added may also be excluded in the same patch. The software won't be able to update CHANGED_FILES until the patch is applied fully applied. At that point, any 'local' file list copy will be incorrect. Plug-ins that absolutely, positively must keep track of files here must ALSO filter their list against EXCLUDE_FILES. But realistically, it would better if no such file list was kept and only used CHANGED_FILES directly. was: The core problem is that in patch mode, CHANGED_FILES is based upon the patch file. Files that are added are still sent to filefilter in order to trigger the test. The problem comes in that files that are added may also be excluded. The software won't know that until the patch is applied. Plug-ins that absolutely, positively must keep track of files here must ALSO filter their list in patch mode. But realistically, it would better if no such file list was kept and only used CHANGED_FILES directly. > plug-ins cannot use filefilter to build lists of files > ------------------------------------------------------ > > Key: YETUS-1202 > URL: https://issues.apache.org/jira/browse/YETUS-1202 > Project: Yetus > Issue Type: Improvement > Components: Precommit > Affects Versions: 0.14.0 > Reporter: Allen Wittenauer > Assignee: Allen Wittenauer > Priority: Blocker > Fix For: 0.15.0 > > > The core problem is that in patch mode, CHANGED_FILES is based upon the patch > file without any extra processing. All files that are touched in some way > are sent to filefilter in order to trigger the appropriate tests. The > problem comes in that files that are added may also be excluded in the same > patch. The software won't be able to update CHANGED_FILES until the patch is > applied fully applied. At that point, any 'local' file list copy will be > incorrect. > Plug-ins that absolutely, positively must keep track of files here must ALSO > filter their list against EXCLUDE_FILES. But realistically, it would better > if no such file list was kept and only used CHANGED_FILES directly. -- This message was sent by Atlassian Jira (v8.20.10#820010)