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.
Summary: | adding Web Applications plugin to Ruby install fails | ||
---|---|---|---|
Product: | platform | Reporter: | Patrick Keegan <pkeegan> |
Component: | Autoupdate | Assignee: | Jiri Rechtacek <jrechtacek> |
Status: | VERIFIED FIXED | ||
Severity: | blocker | CC: | anebuzelsky |
Priority: | P1 | ||
Version: | 7.4 | ||
Hardware: | All | ||
OS: | Windows XP | ||
Issue Type: | DEFECT | Exception Reporter: | |
Bug Depends on: | 113798 | ||
Bug Blocks: |
Description
Patrick Keegan
2007-09-02 15:31:18 UTC
Investigating... Worksforme on my Ubuntu (Ruby build: netbeans-rubyide-hudson-3631.zip). Reproducible of Windows. Depends of issue 113798 - locked file while unpacking NBMs which causes early abort of NBM/Unpack. Fixed in main trunk. Patrick, please verify on next nbrubyide build (http://deadlock.netbeans.org/hudson/job/ruby/) if the fix works for you. If the fix works then I will backport the fix into beta1 branch. Thanks Checking in src/org/netbeans/modules/autoupdate/services/InstallSupportImpl.java; /shared/data/ccvs/repository/autoupdate/services/src/org/netbeans/modules/autoupdate/services/InstallSupportImpl.java,v <-- InstallSupportImpl.java new revision: 1.22; previous revision: 1.21 done Checking in libsrc/org/netbeans/updater/ModuleUpdater.java; /shared/data/ccvs/repository/autoupdate/services/libsrc/org/netbeans/updater/ModuleUpdater.java,v <-- ModuleUpdater.java new revision: 1.9; previous revision: 1.8 done using netbeans-hudson-trunk-3020-ruby.zip the modules install and I am able to see them in action (web and java project types installed) on restart. A couple of small things: *I didn't notice anything saying that I needed to restart to get the modules installed *When I restarted, I get a java.lang.reflect.InvocationTargetException (added to issue http://www.netbeans.org/issues/show_bug.cgi?id=112217). I don't think this is related, but I'm not totally sure. What should be my next steps? Should I try a different build and try to verify again? It works perfectly now in the following hudson build in Win XP: http://deadlock.netbeans.org/hudson/job/ruby/lastSuccessfulBuild/artifact/ruby/rubyide/dist/netbeans-rubyide-hudson-3672.zip Patrick, could you please verify it on the same build? Jirka will then integrate the fix to beta branch. Thanks. I will try http://deadlock.netbeans.org/hudson/job/ruby/lastSuccessfulBuild/artifact/ruby/rubyide/dist/netbeans-rubyide-hudson-3672.zip but note that I just tried http://deadlock.netbeans.org/hudson/job/trunk/3038/artifact/nbbuild/dist/zip/netbeans-hudson-trunk-3038.zip and I didn't work Also note that I'm on Vista, not XP See attachment (id=48135) for previous log file (sorry I accidentally attached to wrong issue) http://www.netbeans.org/nonav/issues/showattachment.cgi/48135/messages.log.1 It works for me perfectly on build http://deadlock.netbeans.org/hudson/job/ruby/lastSuccessfulBuild/artifact/ruby/rubyide/dist/netbeans-rubyide-hudson-3672.zip in fact, I don't even need a restart, like I did on build netbeans-hudson-trunk-3020-ruby.zip So, my question is, are we sure that we are comparing to the right bits? The bug is still reproducible with build with enabled assertions. I guess we want this fixed for Beta1, changing TM and priority. Checking in ModuleUpdater.java; /shared/data/ccvs/repository/autoupdate/services/libsrc/org/netbeans/updater/ModuleUpdater.java,v <-- ModuleUpdater.java new revision: 1.10; previous revision: 1.9 done *** Issue 114284 has been marked as a duplicate of this issue. *** Was this the patch that added a call to File.getCanonicalFile()? Be careful with this method. On Unix it can cause symlink targets to "leak" into the rest of the system in an undesirable way. Best to use FileUtil.normalizeFile when available (I know it is not available in updater.jar). the update worked for me in build http://deadlock.netbeans.org/hudson/job/trunk/3048/ Patrick, thanks for your verification. Jesse, you are right that File.getCanonicalFile() could be risky on unix like systems. But transformation like this is in ModuleUpdater for ages, I'll change it and generally fix it for beta2 - tracked as issue 114863. Checking in src/org/netbeans/modules/autoupdate/services/InstallSupportImpl.java; /shared/data/ccvs/repository/autoupdate/services/src/org/netbeans/modules/autoupdate/services/InstallSupportImpl.java,v <-- InstallSupportImpl.java new revision: 1.21.2.1; previous revision: 1.21 done Checking in libsrc/org/netbeans/updater/ModuleUpdater.java; /shared/data/ccvs/repository/autoupdate/services/libsrc/org/netbeans/updater/ModuleUpdater.java,v <-- ModuleUpdater.java new revision: 1.8.8.1; previous revision: 1.8 done |