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 94385 - Bug in jnlp.xml - "Run JNLP Application" failed
Summary: Bug in jnlp.xml - "Run JNLP Application" failed
Status: RESOLVED FIXED
Alias: None
Product: apisupport
Classification: Unclassified
Component: Harness (show other bugs)
Version: 5.x
Hardware: All All
: P3 blocker (vote)
Assignee: Jaroslav Tulach
URL:
Keywords: SIMPLEFIX
Depends on:
Blocks: 90498
  Show dependency tree
 
Reported: 2007-02-05 14:21 UTC by gborkowski
Modified: 2007-02-05 18:11 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 gborkowski 2007-02-05 14:21:01 UTC
If you add entry
jnlp.platform.codebase=http://www.netbeans.org/download/5_0/jnlp/
to platform.properties file in Project Suite, standalone application,
the "Run JNLP Application" action possibly fails.
It is because all platform jars are used from netbeans.org, there is no
build/jnlp/netbeans dir created, ant fails in jnlp.xml (from harness) in target
build-jnlp-nowar, in task
<copy todir="${build.jnlp.local.dir}/netbeans/" >
   <fileset dir="build/jnlp/netbeans">           <----- it fails here, there is
no such dir
       <include name="*.jnlp"/>
   </fileset>
   ...
</copy>

I'd suggest changing it to:
<copy todir="${build.jnlp.local.dir}/netbeans/" >
   <fileset dir="build/jnlp/">
       <include name="netbeans/*.jnlp"/>
   </fileset>
   ...
</copy>

Perhaps also next task
<copy todir="${build.jnlp.local.dir}/app/">
should be changed similarly.
Comment 1 gborkowski 2007-02-05 15:24:17 UTC
corrections:
1.it is build-jnlp-local task, not build-jnlp-nowar
2.suggested, fixed ant task should be obviously:
<copy todir="${build.jnlp.local.dir}" >
   <fileset dir="build/jnlp/">
       <include name="netbeans/*.jnlp"/>
   </fileset>
   ...
</copy>
Comment 2 Jesse Glick 2007-02-05 18:11:45 UTC
Was already fixed in trunk as part of issue #90498.