Issue Details (XML | Word | Printable)

Key: CHAIN-41
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Simon Kitching
Reporter: Isaac Shabtay
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Commons Chain

Can't build latest source code due to usage of old Digester

Created: 01/Dec/07 11:29 AM   Updated: 20/May/08 09:39 PM
Return to search
Component/s: None
Affects Version/s: 1.1, 1.2, Nightly Builds
Fix Version/s: 1.2

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works patch-build.txt 2007-12-01 11:30 AM Isaac Shabtay 1 kB

Resolution Date: 02/Dec/07 02:46 PM


 Description  « Hide
Currently, build.xml retrieves Digester 1.6 for using as a build path entry.
However, Digester 1.6 doesn't appear to have the Digester.parse(URL) method. This causes org.apache.commons.chain.config.ConfigParser to fail building.

Switched to use Digester 1.8 and builds fine. Testers work as well.



 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Isaac Shabtay made changes - 01/Dec/07 11:29 AM
Field Original Value New Value
Affects Version/s Nightly Builds [ 12311694 ]
Affects Version/s 1.2 [ 12311963 ]
Affects Version/s 1.1 [ 12311930 ]
Isaac Shabtay added a comment - 01/Dec/07 11:30 AM
Proposed path to change build.xml so it retrieves Digester 1.8 instead of 1.6.

Isaac Shabtay made changes - 01/Dec/07 11:30 AM
Attachment patch-build.txt [ 12370740 ]
Simon Kitching added a comment - 02/Dec/07 02:46 PM
Fixed by updating the build.xml file in trunk to depend on digester-1.8, as per pom.xml.
Also updated the myfaces dependency to match the current pom.xml too.
The alternative would be to generate a new ant script from the current pom.xml, but that would need more testing..

The commons project now uses maven 2.x almost exclusively as its build tool, so ant build scripts can easily get out-of-date now.
Maybe we should just start removing ant/maven 1.x scripts now?

The 1.1 release appears ok to me.

Thanks for reporting this, Isaac.


Simon Kitching made changes - 02/Dec/07 02:46 PM
Resolution Fixed [ 1 ]
Status Open [ 1 ] Closed [ 6 ]
Repository Revision Date User Message
ASF #600302 Sun Dec 02 14:50:55 UTC 2007 skitching CHAIN-41 fix bad ant buildfile. Code now depends on digester1.8; the pom.xml had been updated but not the build.xml.
Thanks to Isaac Shabtay for the report.
Files Changed
MODIFY /commons/proper/chain/trunk/build.xml

Niall Pemberton made changes - 20/May/08 09:39 PM
Fix Version/s 1.2 [ 12311963 ]
Assignee Simon Kitching [ skitching ]