Details
Task
Major
Description
Currently (ICEfaces 3.1-) the icepush.jar library is included in the icefaces svn checkout via the icepush.jar being staged / committed directly into the icefaces/lib folder. This requires that the icepush.jar be generated and committed each time the source icepush/trunk repo is updated, which is error prone and labour intensive.
In addition, currently the JavaDoc API docs for icepush are not included in the icefaces release bundles.
An improvement would be to instead include the ice push/trunk (or tag as req.) directly in the icefaces/trunk checkout via the use svn externals. This would also allow the generation of the icepush Javadoc as part of the ICEfaces release bundle build.
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
The problem with the missing icepush.js file from the icepush.jar in the src bundle is because the yui-compressor.jar is being removed from the src bundle:
<delete>
<fileset dir="$
/icepush/lib">
<include name="jstl-1.2"/>
<include name="yui-compressor.jar"/>
</fileset>
</delete>
Is the removal of the yui-compressor.jar intentional?
Show
Jack Van Ooststroom
added a comment - - edited The problem with the missing icepush.js file from the icepush.jar in the src bundle is because the yui-compressor.jar is being removed from the src bundle:
<delete>
<fileset dir="$
{bundle.dir.src}
/icepush/lib">
<include name="jstl-1.2"/>
<include name="yui-compressor.jar"/>
</fileset>
</delete>
Is the removal of the yui-compressor.jar intentional?
Alright, the following is complete:
- Remove staged icepush.jar
- Remove concrete icepush dir which only help the icepush.pom file.
- Added new svn external to include the icepush dir from icepish/trunk.
- Update the build.xml to:
- add new icepush build target
- include icepush target in world and allclean builds.
- include icepush jar and icepush javadoc in binary bundle build.
- remove icepush samples and none-core files from src and bin bundle builds.
Show
Ken Fyten
added a comment - - edited Alright, the following is complete:
Remove staged icepush.jar
Remove concrete icepush dir which only help the icepush.pom file.
Added new svn external to include the icepush dir from icepish/trunk.
Update the build.xml to:
add new icepush build target
include icepush target in world and allclean builds.
include icepush jar and icepush javadoc in binary bundle build.
remove icepush samples and none-core files from src and bin bundle builds.
The icepush/lib move was addressed in ICE-8688. Marking this one as FIXED now.
Show
Jack Van Ooststroom
added a comment - - edited The icepush/lib move was addressed in ICE-8688. Marking this one as FIXED now.
Using the following scratchpad repo branch:
http://10.18.39.25:8888/svn/ossrepo/icefaces3/scratchpads/ICE-8654
I've established the icepush dir under icefaces being included via svn externals from the ice push/trunk.
Also modified the icefaces/build.xml to add an icepush build target, included in world target, and added icepush to the release bundle builds.
However, there remains an issue whereby if you build icefaces using the src bundle that is generated by "ant bundle", the resulting icepush.jar is missing the following resource file:
This causes any deployed applications to fail with:
So this build issue needs to be debugged. Note that the binary bundle contents generated by "ant bundle" work fine.
Also need to add in the generation and copying of the icepush JavaDoc to the binary bundle build.