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 67628 - Unable to change Suite's target platform
Summary: Unable to change Suite's target platform
Status: VERIFIED FIXED
Alias: None
Product: apisupport
Classification: Unclassified
Component: Project (show other bugs)
Version: 5.x
Hardware: All All
: P2 blocker (vote)
Assignee: Jesse Glick
URL:
Keywords:
: 67702 (view as bug list)
Depends on:
Blocks: 64667
  Show dependency tree
 
Reported: 2005-10-27 00:01 UTC by Martin Krauskopf
Modified: 2005-10-31 13:49 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
failingTest.diff (3.04 KB, patch)
2005-10-27 00:02 UTC, Martin Krauskopf 		Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Krauskopf 2005-10-27 00:01:23 UTC 
It seems it was introduced by you in the commit:

http://apisupport.netbeans.org/source/browse/apisupport/project/src/org/netbeans/modules/apisupport/project/suite/SuiteProject.java?r1=1.15&r2=1.16

Also Jesse's commit could caused the regression:

http://apisupport.netbeans.org/source/browse/apisupport/project/src/org/netbeans/modules/apisupport/project/suite/SuiteProject.java?r1=1.21&r2=1.22

Probably nbplatform.active should be always used. If such platform is not valid
we should switch to "default".

Attaching failing test.
Comment 1 Martin Krauskopf 2005-10-27 00:02:08 UTC 
Created attachment 26368 [details]
failingTest.diff
Comment 2 Martin Krauskopf 2005-10-27 00:03:23 UTC 
Commited for convenience.
Comment 3 Jesse Glick 2005-10-27 20:40:47 UTC 
*** Issue 67702 has been marked as a duplicate of this issue. ***
Comment 4 Jesse Glick 2005-10-27 20:47:00 UTC 
Has nothing to do with Yarda's commit. Some problem in the evaluator.
Comment 5 Jesse Glick 2005-10-27 20:54:20 UTC 
There is no regression in the suite evaluator, only in someone's usage of
netbeans.dest.dir instead of nbplatform.active. I am modifying the test to more
clearly show what is wrong. The corresponding fix for standalone modules was
NbModuleProject.java 1.116 (unfiled). The fix for suite projects should be
analogous.
Comment 6 Jesse Glick 2005-10-29 05:23:14 UTC 
committed   * Up-To-Date  1.23       
apisupport/project/src/org/netbeans/modules/apisupport/project/suite/SuiteProject.java
committed   * Up-To-Date  1.6        
apisupport/project/test/unit/src/org/netbeans/modules/apisupport/project/suite/SuiteProjectTest.java
Comment 7 Tomas Danek 2005-10-31 13:49:47 UTC 
testPlatformPropertiesFromEvaluatorAreUpToDate passes now. Verified in current trunk