This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 147540 - opensolaris: doesn't find compilers in /opt/SunStudioExpress/bin
Summary: opensolaris: doesn't find compilers in /opt/SunStudioExpress/bin
Status: RESOLVED FIXED
Alias: None
Product: cnd
Classification: Unclassified
Component: -- Other -- (show other bugs)
Version: 6.x
Hardware: Sun All
: P2 blocker (vote)
Assignee: Alexander Simon
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-09-17 21:46 UTC by Thomas Preisler
Modified: 2008-09-27 05:41 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Preisler 2008-09-17 21:46:49 UTC
CND doesn't find SS compilers in /opt/SunStudioExpress/bin on my OpenSolaris. I don't have them in my path. They seemed
to be found OK when I use remote.

bash-3.2$ echo $PATH
/opt/onbld/bin:/opt/SUNWspro/bin:/export/home/demo/bin:/usr/sfw/bin:/usr/bin:/usr/openwin/bin:/usr/ucb:/usr/sbin:/usr/jdk/latest/bin
Comment 1 _ gordonp 2008-09-19 23:37:09 UTC
It was always intended for the compiler search to look in well known locations if compilers
weren't found. The remote search is doing that but local doesn't. Part of this fix was to
make local compiler search look in these well known locations.

This was done for Unix in http://hg.netbeans.org/main?cmd=changeset;node=e755239a3b4f and for
Windows in http://hg.netbeans.org/main?cmd=changeset;node=4735787afa7b.

Any compiler collections already in the user's path will be chosen prior to these additional
ones.
Comment 2 Quality Engineering 2008-09-20 05:59:58 UTC
Integrated into 'main-golden', will be available in build *200809200201* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/e755239a3b4f
User: Gordon Prieur <gordonp@netbeans.org>
Log: Fixex IZ #147540  opensolaris: doesn't find compilers in /opt/SunStudioExpress
Fixed by appending well known compiler locaions to the path during initial path search.
Comment 3 Alexander Simon 2008-09-23 07:35:50 UTC
Fix breaks concept of tool chain personality.
All hardcoded paths and registry keys should be moved into compiler xmls.
Comment 4 _ gordonp 2008-09-23 16:46:06 UTC
> Fix breaks concept of tool chain personality.
> All hardcoded paths and registry keys should be moved into compiler xmls.

In what way? The toolchain personality defines the toolchains which are recognized by
comparing against a user's path. In most cases, if there are no compilers in the user's
path but compilers are installed in these standard locations, the compilers aren't found.
This has been a problem with the toolchain model for several years. My change resolves
this by making sure the path contains these standard locations. They still need to
match against the pattern in the toolchain personality files.

Another thing this fix does is make local and remote work the same. I could remove this
feature from the remote compiler selection but that would essentially mean I had to close
this issue as will-not-fix. I think this fix strengthens lookup and works with the personality
lookup!
Comment 5 Alexander Simon 2008-09-23 18:10:16 UTC
Method Path.getPathWithDefaultCompilerLocations() use internal (hard coded) information 
about standard compiler location.
This paths should be obtained from xmls.
It allows user to extend standard paths and create own compiler description.
Solution is:
- add field/method in tool chain: "list of standard paths location for tool".
- get additional paths for compiler from descriptor.
- make a solution: check this paths before or after rest paths.
Comment 6 Alexander Simon 2008-09-23 18:17:06 UTC
another issue:
Registry keys (cygwin, mingw, msys) should be taken from xml too.
reasons is same:
- allow user to provide new or fix existent tool chains
Comment 7 _ gordonp 2008-09-25 03:29:55 UTC
Backed out the 1st fix and did xml and ToolchainManager fix. Changeset is
http://hg.netbeans.org/main?cmd=changeset;node=0884d006ebc2
Comment 8 Quality Engineering 2008-09-25 17:55:42 UTC
Integrated into 'main-golden', will be available in build *200809251401* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/0884d006ebc2
User: Gordon Prieur <gordonp@netbeans.org>
Log: Fixex IZ #147540  opensolaris: doesn't find compilers in /opt/SunStudioExpress/bin
Comment 9 Alexander Simon 2008-09-26 08:13:22 UTC
not provided writting of new descriptor fields.
It make user impossible to support new fields.
Comment 10 Alexander Simon 2008-09-26 13:20:55 UTC
fixed+test, change set:
http://hg.netbeans.org/main/rev/8f55974993ff
Comment 11 Quality Engineering 2008-09-27 05:41:46 UTC
Integrated into 'main-golden', will be available in build *200809270201* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/8f55974993ff
User: Alexander Simon <alexvsimon@netbeans.org>
Log: fixed: IZ#147540:opensolaris: doesn't find compilers in /opt/SunStudioExpress/bin