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 92412 - Duplicated and wrongly located insanelib.jar
Summary: Duplicated and wrongly located insanelib.jar
Status: RESOLVED FIXED
Alias: None
Product: apisupport
Classification: Unclassified
Component: Harness (show other bugs)
Version: 6.x
Hardware: PC Windows XP
: P2 blocker (vote)
Assignee: Petr Nejedly
URL:
Keywords: ARCH
Depends on:
Blocks:
 
Reported: 2007-01-12 09:39 UTC by Jaroslav Tulach
Modified: 2007-07-03 15:28 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 Jaroslav Tulach 2007-01-12 09:39:58 UTC
There is 
apisupport/timers/external/insanelib.jar
in CVS which is wrong - external directories are allowed only as direct 
children of <module> directories. See 
http://wiki.netbeans.org/wiki/view/DevFaqExternalLibraries

Moreover insanelib.jar is not external library! It is our own piece of code 
and as such it should be built from source. Here is the mail I wrote about it:

<quote who="pnejedly@netbeans.org">
> User: pnejedly
> Date: 2007/01/10 07:19:41
>
> Added:
>    apisupport/timers/external/insanelib.jar

This does not look good either - the insanelib.jar is already part of nbjunit 
module. This means that effectively it will be duplicated in the IDE for many 
developers. That does not look like a good idea especially because this is a 
library that we have fully under control. I would prefer it to become regular 
module that others can have dependency on.

-jst

PS: I know that its sources are currently in performance module and I do not 
have any preference to whether include performance in standard alias or move 
the sources somewhere else (like libs or xtest module).
Comment 1 Petr Nejedly 2007-06-29 15:07:38 UTC
Created an autoload module around the insanelib:
performance/insanelib/src/org/netbeans/insane/Bundle.properties,v1.1
performance/insanelibmodule/.cvsignore,v1.1
performance/insanelibmodule/build.xml,v1.1
performance/insanelibmodule/manifest.mf,v1.1
performance/insanelibmodule/nbproject/.cvsignore,v1.1
performance/insanelibmodule/nbproject/project.properties,v1.1
performance/insanelibmodule/nbproject/project.xml,v1.1

Added it to the build:
nbbuild/cluster.properties,v1.324



Switched timers from binary copy to the library module: 
apisupport/timers/build.xml,v1.4
apisupport/timers/external/insanelib.jar,vDelete
apisupport/timers/nbproject/project.properties,v1.5
apisupport/timers/nbproject/project.xml,v1.7

Also switched insane module to use the the module instead of building the library itself:
performance/insane/build.xml,v1.12
performance/insane/nbproject/project.properties,v1.14
performance/insane/nbproject/project.xml,v1.11

There is still one binary copy of insanelib.jar in xtest, working on it.
Comment 2 Petr Nejedly 2007-07-03 15:28:13 UTC
Removed the last binary copy of the library, changed the build scripts to take over the source-compiled one:

xtest/build.xml,v1.114
xtest/external/insanelib.jar, delete
xtest/nbjunit/arch.xml,v1.8
xtest/nbjunit/build.xml,v1.30
xtest/nbjunit/nbproject/project.properties,v1.19
xtest/nbjunit/nbproject/project.xml,v1.12
xtest/nbjunit/src/org/netbeans/junit/NbTestCase.java,v1.65
xtest/nbjunit/src/org/netbeans/junit/nb-junit-library.xml,v1.6