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 214438

Summary: Caches may not have valid URIs on Windows
Product: platform Reporter: Jaroslav Tulach <jtulach>
Component: Module SystemAssignee: Jaroslav Tulach <jtulach>
Status: RESOLVED FIXED    
Severity: normal CC: mmirilovic, tpavek
Priority: P1    
Version: 7.2   
Hardware: PC   
OS: Linux   
Issue Type: DEFECT Exception Reporter:
Attachments: Proposed fix for release 7.2
Real fix for 7.2

Description Jaroslav Tulach 2012-06-19 11:05:03 UTC
java.io.FileNotFoundException: Cannot find


jar:file:C:\Jdev\Main\jdeveloper\netbeans\bridge/modules/com-oracle-jdevelope
r-nbwindowsystem.jar!/com/ora
cle/jdeveloper/nbwindowsystem/resources/Standard.xml

org.netbeans.core.startup.layers.BinaryFS$BFSFile.getInputStream(:882)
org.openide.filesystems.MultiFileObject.getInputStream(:65)

Clearly usage of backslashed in URIs is not right.
Comment 1 Jaroslav Tulach 2012-06-19 11:09:51 UTC
ergonomics#0ba965b4027e
Comment 2 Marian Mirilovic 2012-06-19 11:32:11 UTC
Jarda, have you ever seen that in NB ? Or do we have any report from NB ?
Comment 3 Jaroslav Tulach 2012-06-19 13:33:07 UTC
I am not aware of any other example of similar failure in the NetBeans IDE. It probably depends on the content of layers. A registration like this may be present, we just don't know about it.
Comment 4 Jaroslav Tulach 2012-06-20 13:36:21 UTC
Improved in ergonomics#6495f8932dc4
Comment 5 Jaroslav Tulach 2012-06-21 11:57:57 UTC
Created attachment 121167 [details]
Proposed fix for release 7.2
Comment 6 Jaroslav Tulach 2012-06-21 11:58:41 UTC
Created attachment 121168 [details]
Real fix for 7.2
Comment 7 Tomas Pavek 2012-06-21 13:14:21 UTC
Yes, this is really needed if the caches should work cross-platform.
The fix looks good to me.
Comment 8 Tomas Pavek 2012-06-21 13:15:15 UTC
Please integrate into 7.2.
Comment 9 Tomas Hurka 2012-06-21 16:38:22 UTC
I reviewed the fix together with Jarda and I think that it is OK to integrate it into release72 branch.
Comment 10 Jaroslav Tulach 2012-06-21 16:45:53 UTC
Merged as a14597f5091b: Thank you guys for your deep review. I know it is not easy to review this code, but I'll do my best to not spoil anything.
Comment 11 Quality Engineering 2012-06-22 04:21:25 UTC
Integrated into 'releases', will be available in build *201206212341* or newer. Wait for official and publicly available build.
Changeset: http://hg.netbeans.org/releases/rev/0ba965b4027e
User: Jaroslav Tulach <jtulach@netbeans.org>
Log: #214438: Convert the cluster paths to URIs.
Comment 12 Quality Engineering 2012-06-22 04:51:08 UTC
Integrated into 'main-golden', will be available in build *201206220002* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/0ba965b4027e
User: Jaroslav Tulach <jtulach@netbeans.org>
Log: #214438: Convert the cluster paths to URIs.
Comment 13 Jaroslav Tulach 2012-06-27 15:36:33 UTC
The all-manifests.dat cache is confused with slashes and backslashes as well.
Comment 14 Jaroslav Tulach 2012-06-27 15:45:19 UTC
Fix in 7.3, please review: http://hg.netbeans.org/ergonomics/rev/07ff902f8d06
Comment 15 Jaroslav Tulach 2012-06-27 15:57:29 UTC
Improved fix. Result of Tomáš Pávek's review: http://hg.netbeans.org/ergonomics/rev/d508f61b1b55
Comment 16 Tomas Pavek 2012-06-27 16:00:43 UTC
I think it's correct now. We need this fixed on all places, so this should be integrated into 7.2 as well.
Comment 17 Jaroslav Tulach 2012-06-27 16:06:49 UTC
Merged as c181fe96cf32 into release72.
Comment 18 Jaroslav Tulach 2012-06-27 16:39:52 UTC
Ignore previous comment. To speed propagation up, I've transplanted the change as http://hg.netbeans.org/releases/rev/585a5f5ff554
Comment 19 Quality Engineering 2012-06-28 04:55:15 UTC
Integrated into 'main-golden', will be available in build *201206280002* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/07ff902f8d06
User: Jaroslav Tulach <jtulach@netbeans.org>
Log: #214438: all-manifests.dat needs to replace / with backslashes
Comment 20 Quality Engineering 2012-06-28 05:09:12 UTC
Integrated into 'releases', will be available in build *201206272359* or newer. Wait for official and publicly available build.
Changeset: http://hg.netbeans.org/releases/rev/585a5f5ff554
User: Jaroslav Tulach <jtulach@netbeans.org>
Log: #214438: Transplanted 4th fix to 7.2: changeset d508f61b1b55