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: | contrib/bookmarks/build.xml edits | ||
|---|---|---|---|
| Product: | contrib | Reporter: | Jesse Glick <jglick> |
| Component: | Code | Assignee: | David Strupl <dstrupl> |
| Status: | RESOLVED FIXED | ||
| Severity: | blocker | ||
| Priority: | P2 | ||
| Version: | 3.x | ||
| Hardware: | All | ||
| OS: | All | ||
| Issue Type: | TASK | Exception Reporter: | |
| Attachments: | Suggested patch | ||
Created attachment 11385 [details]
Suggested patch
Thanks for the patch. I assume the changes are valid for trunk. I have to run the module with release35 for now. Should I branch to release35 first? But if I branch I would have to do all changes twice. What do you think? Yes, the patch was intended for the trunk, though a similar version could be used for release35 (just skip openide-loaders.jar in classpath). I guess it's up to you whether or not to branch it. If you don't, it makes it harder to keep it up to date with changes in the trunk (mainly in build system but occasionally bulk cleanups of deprecated things etc.). Applied the changes without the openide-loaders.jar. So it should still build with release35. I will branch later - when the module is considered stable. As I see it now it should be during 9/2003. OK. FYI, it is probably possible to make it work in either situation,
e.g. include in <classpath>:
<fileset dir="${nbroot}/openide">
<include name="loaders/netbeans/lib/openide-loaders.jar"/>
</fileset>
I have added the o-loaders.jar after verifying that it still works with release35 of openide. It should now be also on the alpha autoupdate (trunk version). |
To make it more consistent with other build scripts. With this patch you ought to be able to add contrib/bookmarks to ${config.modules.daily-alpha-nbms} in nbbuild/build.properties so as to publish the module on the daily dev alpha AU server, which would expose it to a broader audience (just bump up the spec version in CVS whenever you want to push a new build).