Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 368115 - Application of Decorator for Excluded Files from Build in some cases is surprising
Summary: Application of Decorator for Excluded Files from Build in some cases is surpr...
Status: NEW
Alias: None
Product: CDT
Classification: Tools
Component: cdt-build (show other bugs)
Version: 8.0   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: cdt-build-inbox@eclipse.org CLA
QA Contact: Jonah Graham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-08 23:56 EST by Andrew Gvozdev CLA
Modified: 2020-09-04 15:20 EDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Gvozdev CLA 2012-01-08 23:56:02 EST
There is a couple of cases where I as a user find decorations of excluded files unexpected:
- inside of excluded folders the decorator is applied to all files even on files unrelated to build or C/C++
- if the project itself is not marked as Source Folder, non-source folders and their files are not decorated as excluded from build
Comment 1 Chris Recoskie CLA 2013-08-13 14:21:43 EDT
The decorators are also being applied to projects that don't even have the cnature.
Comment 2 Chris Recoskie CLA 2013-08-13 14:50:17 EDT
(In reply to comment #0)
> There is a couple of cases where I as a user find decorations of excluded
> files unexpected:
- inside of excluded folders the decorator is applied to
> all files even on files unrelated to build or C/C++

As far as this one, this is tricky.  In the managed build case we mostly know what files will or will not be built, but in the non-managed case we really have no idea what the build command and/or build script will do, and being external to the IDE, the build is almost certainly not going to respect any include/exclude settings in the IDE.  Hence, we don't really know what to include or exclude.  I think in the non-managed case we should not apply decorators at all.

Thoughts?
Comment 3 Andrew Gvozdev CLA 2013-08-14 09:26:18 EDT
C/C++ content types are known (and can be amended by users/ISV). We could figure out content types by file extension and apply decorations only to those. As far as non managed build, users can mark files/folders manually for whatever reason (distinct presentation in UI, to avoid indexing etc.). So it is reasonable for them to expect CDT to decorate those.
Comment 4 Chris Recoskie CLA 2013-08-20 10:05:32 EDT
(In reply to comment #3)
> C/C++ content types are known (and can be amended by users/ISV). We could
> figure out content types by file extension and apply decorations only to
> those. As far as non managed build, users can mark files/folders manually
> for whatever reason (distinct presentation in UI, to avoid indexing etc.).
> So it is reasonable for them to expect CDT to decorate those.

Ok, yes I suppose that is a valid use case.

There are instances however where CDT's builders are used on non-C/C++ content types, e.g. Fortran.  I suppose decorating some files in the project would be better than none, but users are probably going to be confused if they have a mixed C/Fortan project and the UI shows decorations on one of the file types but not the other.  I don't think there's anything we can really do about that, I am just stating it here to make it clear.
Comment 5 Andrew Gvozdev CLA 2013-08-20 10:23:20 EDT
If Fortran plugin does not decorate excluded files of Fortran content types it is up to its developers to provide that, isn't it? Note also that it is possible to disable CDT decorators via Preferences->General->Appearance->Label Decorations.