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 26319 - Simplify modules vs. fixedmodules
Summary: Simplify modules vs. fixedmodules
Status: RESOLVED FIXED
Alias: None
Product: www
Classification: Unclassified
Component: Builds & Repositories (show other bugs)
Version: 3.x
Hardware: PC Linux
: P2 blocker (vote)
Assignee: rbalada
URL:
Keywords:
: 26536 (view as bug list)
Depends on:
Blocks: 26123
  Show dependency tree
 
Reported: 2002-08-06 16:35 UTC by Jesse Glick
Modified: 2003-02-20 16:33 UTC (History)
1 user (show)

See Also:
Issue Type: TASK
Exception Reporter:

Attachments
Suggested patch (10.15 KB, patch)
2002-08-06 16:37 UTC, Jesse Glick 		Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jesse Glick 2002-08-06 16:35:48 UTC 
The master build script does much too much work to
define the modules and fixedmodules. This patch
simplifies it and makes it easier to override
whatever you want to. It also gets rid of two Ant
tasks.

BTW I suggest the moduleconfig 'stable' be renamed
'standard', which would match the CVS module
alias, and also just make more sense.

I am also changing the module config for
'platform' to not include core/execution and
core/compilation, BTW - not sure why these were
ever in there, but I am fixing it.
Comment 1 Jesse Glick 2002-08-06 16:37:20 UTC 
Created attachment 7031 [details]
Suggested patch
Comment 2 Michal Zlamal 2002-08-12 21:34:30 UTC 
Accepting.
Comment 3 Jesse Glick 2002-08-15 20:25:21 UTC 
*** Issue 26536 has been marked as a duplicate of this issue. ***
Comment 4 Michal Zlamal 2002-08-16 15:16:20 UTC 
 I applied proposed patch.
Comment 5 Jesse Glick 2002-08-16 16:06:07 UTC 
Uh, no; the change to nbbuild/build.properties was not committed, nor
to nbbuild/.cvsignore. Changes to
nbbuild/antsrc/org/netbeans/nbbuild/CheckValue.java and
nbbuild/antsrc/org/netbeans/nbbuild/ConfigureModules.java were
committed and mentioned #26319. The patch was committed to
nbbuild/build.xml along with some other changes, though the commit
message did not mention #26319 at all.

When you commit build.properties and .cvsignore, please mark this
FIXED right away.
Comment 6 Jesse Glick 2002-08-16 16:42:43 UTC 
Committing the rest of the patch. (The continuous build was not
working with half of it.)
Comment 7 thierrylach 2002-08-16 16:54:31 UTC 
Please either delete moduledefs-tmp.properties after use 
or add it to .cvsignore.
Comment 8 Jesse Glick 2002-08-16 17:23:49 UTC 
Adding it to .cvsignore was part of the second half of the patch.
Comment 9 Michal Zlamal 2003-02-20 16:33:13 UTC 
Changed target milestone