Bug 201604 - "Available tools" problem and enhauncements
Summary: "Available tools" problem and enhauncements
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-core (show other bugs)
Version: 4.0.1   Edit
Hardware: PC Linux
: P3 enhancement (vote)
Target Milestone: 4.0.1   Edit
Assignee: Oleg Krasilnikov CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-08-29 12:32 EDT by Miwako Tokugawa CLA
Modified: 2007-09-17 10:55 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Miwako Tokugawa CLA 2007-08-29 12:32:00 EDT
I already talked with Mikhail S. on this issue.

Enhancement 1
It would be nice for CDT to provide means (probably a tool attribute) to tool-integrators to specify whether a tool should be displayed in the "Available tools" list. Currently no such exist and we have a problem with invalid tools (for example, "Intel Linker" which, simply being the superclass of Intel linkers, does not represent any real tool).

Enhancement 2
It would be nice if "Available tools" list was alphabetized.

Problem (Related to 1)
CDT also displays "Intel C Linker", which is not a real tool. (We have "Intel C Linker" specified in our xml and CDT attaches version numbers at the end to represent real tools (ex. "Intel C Linker (v9.1.0)", "Intel C Linker (v10.0.0)")
"Available tools" has these real tools, and the "fake tool" listed in addition.)
Comment 1 Oleg Krasilnikov CLA 2007-08-30 05:27:10 EDT
Regarding problem 1:
would it be acceptable for you to set "isAbstract" or 
"isSystem" attribute for tools which should be hidden ?
Currently, system and/or abstract tools are not displayed.

Regarding problem 2:
Optional alphabetical sorting will be added ASAP.
Comment 2 Mikhail Sennikovsky CLA 2007-08-30 07:02:25 EDT
(In reply to comment #0)
Hi Miwako,

> I already talked with Mikhail S. on this issue.
> 
> Enhancement 1
> It would be nice for CDT to provide means (probably a tool attribute) to
> tool-integrators to specify whether a tool should be displayed in the
> "Available tools" list. Currently no such exist and we have a problem with
> invalid tools (for example, "Intel Linker" which, simply being the superclass
> of Intel linkers, does not represent any real tool).
As Oleg says, the "system" and "abstract" tools should never be displayed in the tool-chain modification UI. You'd better use the "abstract" property rather than the "isSystem" one since the latter is used to define Build System-internal/specific objects. Do you see the "isAbstract" tools being displayed in the tool-chain modification UI?

> Enhancement 2
> It would be nice if "Available tools" list was alphabetized.
The "select tools" dialog contains two panes: one displaying the list of available tools and the other displaying the tool-chain tools.
I agree with having the former list presented in the alphabetical order, however the latter should remain unsorted since the tool order in the tool-chain may determine the tool-chain behavior and should be preserved and kept unmodified in UI. E.g. suppose a tool-chain has two tools that accept the .c files. In this case if the first tool in the list would be used for building the .c files, while the second one will be ignored. I guess this information should not be hidden from the user, thus the I think that the "Used tools" list should remain unsorted.

Mikhail
Comment 3 Oleg Krasilnikov CLA 2007-08-30 07:40:20 EDT
Done with tools list sorting.
Leaving this bug open until "isAbstract()" discussion continues.

Comment 4 Oleg Krasilnikov CLA 2007-09-04 09:04:34 EDT
It seems like there will be no additional changes.
Closing bug as fixed.