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 37736

Summary: CVS trunk broken
Product: serverplugins Reporter: _ tboudreau <tboudreau>
Component: InfrastructureAssignee: Nam Nguyen <nnguyen>
Status: RESOLVED FIXED    
Severity: blocker CC: nbbuild-issues, rkubacki
Priority: P1    
Version: 3.x   
Hardware: All   
OS: All   
URL: http://j2eeserver.netbeans.org/source/browse/j2eeserver/src/org/netbeans/modules/j2ee/deployment/impl/ServerString.java.diff?r1=1.5&r2=1.6
Issue Type: DEFECT Exception Reporter:
Attachments: hotfix patch file to make the cvs trunk buildable again

Description _ tboudreau 2003-12-04 10:32:09 UTC
compile:
    [javac] Compiling 95 source files to 
C:\space\nb_all\j2eeserver\src
    [javac] 
C:\space\nb_all\j2eeserver\src\org\netbeans\modul
es\j2ee\deployment\
impl\ServerString.java:49: variable targets 
might not have been initialized
    [javac]             return;
    [javac]             ^
    [javac] Note: Some input files use or 
override a deprecated API.
    [javac] Note: Recompile with -deprecation 
for details.
    [javac] 1 error
Comment 1 _ rkubacki 2003-12-04 10:59:52 UTC
Compiles with JDK1.4.0 but doesn't work with JDK1.4.2 (+Ant 1.5.3)

The reason should be:
j2eeserver/src/org/netbeans/modules/j2ee/deployment/impl/ServerString.java
revision 1.6
date: 2003/12/03 20:59:45;  author: nnguyen;  state: Exp;  lines: +26 -5
IN=18221 Make sure there is alway a default server
----------------------------
Comment 2 _ tboudreau 2003-12-04 11:09:23 UTC
I've put in a hotfix so the trunk is buildable again, but you need 
to review it - I have no idea if the targets variable should be 
null, or what it does - it just compiles now.
Comment 3 _ tboudreau 2003-12-04 11:09:55 UTC
Created attachment 12412 [details]
hotfix patch file to make the cvs trunk buildable again
Comment 4 Nam Nguyen 2003-12-04 16:18:35 UTC
I put in the actual fix.  The final variable targets should be not null.
Thanks Tim for taking care of this.
I wonder if I shoudl use 1.4.2 too.  Why continuous build does not use
1.4.2?
Comment 5 _ tboudreau 2003-12-04 20:54:29 UTC
No idea.  I filed issue 37761 against nbbuild for them to upgrade the
compiler.

Now the question is, how do we mail the golem to California?
Comment 6 Petr Jiricka 2003-12-05 12:52:12 UTC
Nam, so can this be marked as FIXED?
Comment 7 Nam Nguyen 2003-12-05 16:13:44 UTC
Fix the status :-).  I need this habit now.