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 28471 - Import Wizard isn't again shown during the first start
Summary: Import Wizard isn't again shown during the first start
Status: VERIFIED FIXED
Alias: None
Product: ide
Classification: Unclassified
Component: Import Settings (show other bugs)
Version: 3.x
Hardware: All All
: P1 blocker (vote)
Assignee: Jesse Glick
URL:
Keywords:
Depends on: 30502
Blocks: 24888
  Show dependency tree
 
Reported: 2002-11-05 11:01 UTC by Marian Mirilovic
Modified: 2008-12-22 19:39 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Proposed source patch (7.56 KB, patch)
2002-11-27 21:35 UTC, Jesse Glick
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Marian Mirilovic 2002-11-05 11:01:51 UTC
[nb_dev](20021105), [jdk1.4.1](01)

Assigne to Richard: Richard I am not sure you are
responsible now, feel free to reassigne.
Comment 1 Richard Gregor 2002-11-05 11:16:15 UTC
Reassignig to upgrade because it is not caused by installer.
Comment 2 Marian Mirilovic 2002-11-14 09:33:41 UTC
This issue is pointed out as future Q-build stopper, please fix it before
11/20/2002.
Report http://qa.netbeans.org/q-builds/Q-build-report-200211131011.html.
Comment 3 rmatous 2002-11-14 15:57:10 UTC
There is problem that in NonGui is called Class.forName for
UpgradeWizard and that ClassNotFoundException is fired. So, I tried to
use SystemClassLoader for forName invocation. But Lookup.lookup
(ClssLoader) returned me  instance of org.netbeans.JarClassLoader
instead of expected
org.netbeans.core.modules.ModuleManager$SystemClassLoader. 

I would like to ask Jesse to investigate where the problem is (so
reassigned to jglick). 
 
Comment 4 Jesse Glick 2002-11-14 16:39:52 UTC
Of course SystemClassLoader is not returned - there is no system class
loader. This call is made before the module system has even been
created. Funny, because you are trying to load a class from a module.

I think I can find some simple workaround.
Comment 5 Jiri Kovalsky 2002-11-27 16:02:57 UTC
Increasing priority because this becomes automatically Q-build
showstopper. Please fix this issue as soon as possible.
Comment 6 Jesse Glick 2002-11-27 18:01:04 UTC
Yes, I have a patch, just need to (re)test & commit.
Comment 7 Marian Mirilovic 2002-11-27 18:17:15 UTC
Jesse, can I help you test your patch ? 

If yes, please attach patched jar to this issue.
Comment 8 Jesse Glick 2002-11-27 21:35:33 UTC
Created attachment 8075 [details]
Proposed source patch
Comment 9 rmatous 2002-11-28 13:13:57 UTC
Thanks Jesse, I`m going to test and commit.
Comment 10 Jesse Glick 2002-11-28 14:11:20 UTC
Don't bother, it is assigned to me, I am just running a final CVS
update and will run unit tests now... (Normally I would have committed
this a week ago, but I got interrupted by traveling.)
Comment 11 Jesse Glick 2002-11-28 14:55:30 UTC
committed   * Up-To-Date  1.9        
core/ide/src/org/netbeans/core/upgrade/CopyUtil.java
committed   * Up-To-Date  1.3        
core/ide/src/org/netbeans/core/upgrade/PreUpgradePanel.java
committed   * Up-To-Date  1.3        
core/ide/src/org/netbeans/core/upgrade/TransferSettingsPanel.java
committed   * Up-To-Date  1.5        
core/ide/src/org/netbeans/core/upgrade/UpgradePanel.java
committed   * Up-To-Date  1.86       
core/src/org/netbeans/core/NonGui.java
Comment 12 Jesse Glick 2002-11-28 18:05:20 UTC
Do you want this in QBE200211270100 too?
Comment 13 Jiri Kovalsky 2002-11-29 08:00:46 UTC
Of course. It's a showstopper. Thanks.
Comment 14 Jesse Glick 2002-11-29 13:37:08 UTC
Fixed in QBE200211270100:

Checking in core/ide/src/org/netbeans/core/upgrade/CopyUtil.java;
/cvs/core/ide/src/org/netbeans/core/upgrade/CopyUtil.java,v  <-- 
CopyUtil.java
new revision: 1.8.8.1; previous revision: 1.8
done
Checking in core/ide/src/org/netbeans/core/upgrade/PreUpgradePanel.java;
/cvs/core/ide/src/org/netbeans/core/upgrade/PreUpgradePanel.java,v 
<--  PreUpgradePanel.java
new revision: 1.2.16.1; previous revision: 1.2
done
Checking in
core/ide/src/org/netbeans/core/upgrade/TransferSettingsPanel.java;
/cvs/core/ide/src/org/netbeans/core/upgrade/TransferSettingsPanel.java,v
 <--  TransferSettingsPanel.java
new revision: 1.2.16.1; previous revision: 1.2
done
Checking in core/ide/src/org/netbeans/core/upgrade/UpgradePanel.java;
/cvs/core/ide/src/org/netbeans/core/upgrade/UpgradePanel.java,v  <-- 
UpgradePanel.java
new revision: 1.4.16.1; previous revision: 1.4
done
Processing log script arguments...
More commits to come...
Checking in core/src/org/netbeans/core/NonGui.java;
/cvs/core/src/org/netbeans/core/NonGui.java,v  <--  NonGui.java
new revision: 1.84.2.1; previous revision: 1.84
done
Processing log script arguments...
Mailing the commit message to cvs@core.netbeans.org (from
jglick@netbeans.org)
Comment 15 Jiri Kovalsky 2002-11-29 13:45:14 UTC
Mariane, could you please verify this ? Thanks.
Comment 16 Marian Mirilovic 2002-11-29 15:25:07 UTC
verified in [nb_dev](20021129-0521)