Bug 38441 - Allow src folder exclusion filters with no path that exclude all files of a certain type
Summary: Allow src folder exclusion filters with no path that exclude all files of a c...
Status: RESOLVED INVALID
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 2.1   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.0 M2   Edit
Assignee: Philipe Mulet CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-06-04 18:17 EDT by Matthew King CLA
Modified: 2003-06-11 13:01 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew King CLA 2003-06-04 18:17:10 EDT
It is common to want to exclude all files of a certain type from a source 
folder's output folder, e.g., all *.html files, regardless of where in the 
source tree the *.html file exists. Right now, it seems the best one can do is 
build an exclusion filter for each level of a source tree. 
e.g., /packagename/*.html, /packagename/*/*.html, /packagename/*/*/*.html ...

Why not allow simply *.html and have exclude all *.html files regardless of 
where they are in the source folder. Or, as an alternative, */*.html with no 
leading slash could represent any number of levels in the source tree.
Comment 1 Philipe Mulet CLA 2003-06-05 05:07:16 EDT
It should work using "**/*.html" as an exclusion pattern, but I agree with you 
this should be equivalent to "*.html".

Currently, we transform the relative exclusion pattern into a workspace 
relative exclusion pattern: "*.html" --> "/Proj/src/*.html". If the exclusion 
pattern isn't absolute, we should rather append "**/*.html".
Need to investigate.

Comment 2 Philipe Mulet CLA 2003-06-11 05:20:53 EDT
Actually, originally we had implemented the exclusion patterns to be exactly 
doing so, and we switched to the current story. Cannot remember why we did so.
Comment 3 Philipe Mulet CLA 2003-06-11 06:06:21 EDT
I take my previous comments back. Also see bug 29761 which explains why we did 
change our original (wrong) behavior.


Ant patternset rules are not treating 'a' as '**/a' by default. Only trailing 
separator is special ('a/' is equivalent to 'a/**').

Thus you should explicitly qualify your exclusion pattern with a leading '**/'.
Comment 4 Matthew King CLA 2003-06-11 13:01:04 EDT
When I submitted the enhancement request, I was not aware of the **/... 
option.Maybe this should be closed as the functionality is already available. I 
see now that the option is described in the field help, but that was not read 
by my screen reader. The 5/28 build allows field help and error messages to be 
tabbed to, making them accessible to a screen reader. but for some reason that 
feature does not work in the exclusion filter dialog. I will report that in 
that feature's record.