Issue Details (XML | Word | Printable)

Key: CONF-9804
Type: Task Task
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Paul Curren [Atlassian]
Reporter: Justin Koke [Atlassian]
Votes: 0
Watchers: 0
Operations

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

Move the XFire dependency for Crowd from 1.2.1 to 1.2.6

Created: 24/Oct/07 02:54 AM   Updated: 12/Nov/07 06:24 PM
Component/s: Crowd Integration
Affects Version/s: 2.6.0
Fix Version/s: 2.6.2

Time Tracking:
Not Specified

Participants: David Taylor [Atlassian], Justin Koke [Atlassian], Matt Ryall [Atlassian] and Paul Curren [Atlassian]
Since last comment: 42 weeks, 6 days ago
Internal Complexity: 1
Resolution Date: 12/Nov/07 06:24 PM
Internal Value: 6
Labels:


 Description  « Hide
The Crowd libraries in Confluence need updating to get around an error found with XFire http://jira.codehaus.org/browse/XFIRE-888

This should mean the following for Confluence's parent pom.

<dependency>
  <groupId>woodstox</groupId>
  <artifactId>wstx-asl</artifactId>
  <version>3.2.0</version>
  <scope>runtime</scope>
</dependency>
<dependency>
   <groupId>org.codehaus.xfire</groupId>
   <artifactId>xfire-all</artifactId>
  <version>1.2.6</version>
  <scope>runtime</scope>
</dependency>


 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Matt Ryall [Atlassian] added a comment - 24/Oct/07 03:23 AM
Stable change; we don't use either of these in Confluence.

Make these dependencies scope "runtime".


David Taylor [Atlassian] added a comment - 07/Nov/07 06:07 PM
FYI - these dependencies are just in the pom to make the Crowd extension installation easier. Don't look for references in the confluence source because you won't find any

David Taylor [Atlassian] added a comment - 07/Nov/07 07:31 PM
Justin pointed out that the XFire dependencies have hardly changed since 1.2.1 (http://fisheye.codehaus.org/browse/xfire/trunk/xfire/pom.xml?r1=2186&r2=1918)
jetty might not be excluded but it's optional = true so the point may be moot.

Other than that, a perfectly boring change. Thank you, come again!


Paul Curren [Atlassian] added a comment - 12/Nov/07 06:24 PM
Simple POM change - no review comments.