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 35523 - contrib/bookmarks/build.xml edits
Summary: contrib/bookmarks/build.xml edits
Status: RESOLVED FIXED
Alias: None
Product: contrib
Classification: Unclassified
Component: Code (show other bugs)
Version: 3.x
Hardware: All All
: P2 blocker (vote)
Assignee: David Strupl
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-08-20 17:14 UTC by Jesse Glick
Modified: 2003-08-27 12:40 UTC (History)
0 users

See Also:
Issue Type: TASK
Exception Reporter:


Attachments
Suggested patch (3.28 KB, patch)
2003-08-20 17:15 UTC, Jesse Glick
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jesse Glick 2003-08-20 17:14:24 UTC
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).
Comment 1 Jesse Glick 2003-08-20 17:15:15 UTC
Created attachment 11385 [details]
Suggested patch
Comment 2 David Strupl 2003-08-20 20:46:38 UTC
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?
Comment 3 Jesse Glick 2003-08-20 22:11:25 UTC
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.).
Comment 4 David Strupl 2003-08-24 21:27:42 UTC
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.
Comment 5 Jesse Glick 2003-08-25 05:40:59 UTC
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>
Comment 6 David Strupl 2003-08-27 12:40:34 UTC
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).