Bug 867080 - Seam booking, seamspace: Broken ant scripts (unclosed version strings, bad dom4j version)
Summary: Seam booking, seamspace: Broken ant scripts (unclosed version strings, bad do...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise WFK Platform 2
Classification: Retired
Component: Seam
Version: 2.1.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: CR1
: 2.1.0
Assignee: Marek Novotny
QA Contact: Marek Schmidt
URL:
Whiteboard: Seam 2.3
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-10-16 18:08 UTC by Ron Šmeral
Modified: 2016-11-01 01:36 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-11-30 15:34:58 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Ron Šmeral 2012-10-16 18:08:50 UTC
Description of problem:
There's a typo in the ant script of all modules (ear, ejb, tests, web). 
The value of the version attribute is unclosed -- it misses the right quotation mark.
The positions are: 
- booking-ear/build.xml:43, 
- booking-ejb/build.xml:58, 
- web/build.xml:45, 
- web/build.xml:48, 
- tests/build.xml:45, 
- tests/build.xml:52.

After fixing this, calling "ant clean package" in the example's directory fails with missing dependency: org.jboss.seam:bom:pom:2.3.0.Final-redhat-1, even though this dependency is present in the jboss-wfk-repository. (Only after calling "mvn clean package" first, the "ant clean package" finds the missing dependency.)

Also, in booking-ejb, the dependency dom4j has version 1.6.1-brew, which can't be found in any of the relevant repositories (central, jboss-wfk-repository, jboss-eap6-repository).


Version-Release number of selected component (if applicable):
2.3.0.Final-redhat-1


Steps to Reproduce:
1. Download and properly unpack 
- jboss-wfk-2.1.0.ER2-seam23-bin.zip, 
- jboss-wfk-2.1.0.ER2-seam23-demo.zip, 
- jboss-wfk-2.1.0.ER2-maven-repository.zip, 
- jboss-eap-6.0.0-maven-repository.zip.

2. Run "ant clean package" in jboss-wfk-2.1.0.ER2/jboss-seam-2.3.0.Final-redhat-1/examples/booking.

Actual results:
The build fails.

Expected results:
The build should pass and produce seam-booking.ear

Comment 1 Ron Šmeral 2012-10-18 13:13:23 UTC
There's another problem, with the Ant build script. If the zips (bin, demo) are unpacked in the same folder, the Ant build complains about missing build.lib.classpath, which points to ${root.dir}/build/lib, where root.dir points to "../../..". 
This is not very safe, since it only works once the 'examples' dir is moved to the unpacked Seam home. This is, however, not documented anywhere. 
A safer solution (instead of the root.dir) would be to introduce a seam.home variable which would have to be set to the Seam home directory and this would be documented in the appropriate readme.txt file.

Comment 2 Marek Schmidt 2012-10-19 13:11:04 UTC
Note that similar problems are also in the seamspace example, which also contains ant build files.

Comment 3 Marek Novotny 2012-11-05 16:54:17 UTC
I have removed scripts from seamspace example

Comment 5 Karel Piwko 2012-11-30 15:34:58 UTC
Distributed as a part of WFK 2.1.0.GA release.


Note You need to log in before you can comment on or make changes to this bug.