Skip to content
This repository was archived by the owner on Mar 29, 2020. It is now read-only.

Conversation

@f4tq
Copy link

@f4tq f4tq commented Jun 2, 2011

Hi,
Thanks for the trqbox-demo. I'm porting a Seam 2 project to jruby via torquebox so I immediately started trying to extend your demo. Once I added a maven dependency for softwaremill-cdi, I hit this library compatability issue. Since this is particular to a jboss container, I'm not sure what course you would prefer to handle this but maybe via a maven profile? I imagine a lot of people use jboss 6.0 or torquebox 1.1.

My Commit Message:

Reference TorqueBox 1.1/jboss 6.0's Weld 1.1.0.Final version. More importantly, regress softwaremill-common back to guava r06 because the jbossas 6 classloader throws 'Unsatisfied link' exception loading libraries that otherwise work just fine with trqbox-demo tests. softwaremill-cdi was explicitely using guava r07. This will be frustrating to anyone trying to follow and extend the demo to intergrate other CDIs into torquebox. All tests worked on the regression/installation of 47-SNAPSHOT.

Thanks,
Chris

…portantly, regress softwaremill-common back to guava r06 because the jbossas 6 classloader throws 'Unsatisfied link' exception loading libraries that otherwise work just fine with trqbox-demo tests. softwaremill-cdi was explicitely using guava r07. This will be frustrating to anyone trying to follow and extend the demo to intergrate other CDIs into torquebox. All tests worked on the regression/installation of 47-SNAPSHOT
@adamw
Copy link
Member

adamw commented Jun 8, 2011

Ah, right, we just work around this issue by marking guava as provided in our pom and copying it to jboss/server/default/lib. But in the demo's pom.xml that's already done, I guess you must have hit the issue when trying to re-create the demo?
Anyway, I think the best thing to do here would be to add a note to the README, how do you think!

Thanks for trying out the demo, btw.! :)

Adam

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants