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 57379 - Do not require jellytools to be built as a binary JAR for xtest
Summary: Do not require jellytools to be built as a binary JAR for xtest
Status: RESOLVED FIXED
Alias: None
Product: obsolete
Classification: Unclassified
Component: xtest (show other bugs)
Version: 4.x
Hardware: All All
: P3 blocker (vote)
Assignee: issues@qa
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-04-03 03:22 UTC by Jesse Glick
Modified: 2005-09-05 10:16 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jesse Glick 2005-04-03 03:22:36 UTC
Twice in recent weeks I have had to patch jellytools sources (as they were
referring to some obsolete bundle key in NB main sources rather than the
currently used key), and each time I have had to build jellytools manually,
running the jar-library target, and copying the built JAR manually into the
"builds" dir, in order to get commit validation to pass.

Is this really necessary? Can't xtest's build script be made smart enough to use
the current jellytools sources, so that we can patch them normally, as is done
for other test code?
Comment 1 Jiri Skrivanek 2005-04-04 13:13:38 UTC
Well, it was arranged such a way because we didn't want to break tests for
accidentally uncompilable jellytools sources. Now, when commit validation uses
jellytools we might reconsider this.
Comment 2 Jiri Skrivanek 2005-05-03 09:31:04 UTC
For test scripts which imports xtest-qa-functional.xml jellytools jar is built
if sources are available. Otherwise it uses jar from jellytools/builds/.

/cvs/nbbuild/templates/xtest-qa-functional.xml,v  <--  xtest-qa-functional.xml
new revision: 1.6; previous revision: 1.5
Comment 3 Jiri Skrivanek 2005-05-03 12:27:28 UTC
Needed also changes in jellytools/build.xml:

/cvs/jellytools/build.xml,v  <--  build.xml
new revision: 1.46; previous revision: 1.45