Issue1936
Created on 2012-06-29.12:27:50 by vmassol, last changed 2013-02-13.23:27:16 by fwierzbicki.
File name |
Uploaded |
Description |
Edit |
Remove |
patch1936.txt
|
olivierH,
2012-09-27.16:06:38
|
|
|
|
msg7275 (view) |
Author: Vincent Massol (vmassol) |
Date: 2012-06-29.12:27:49 |
|
I have exactly the same problem as http://bugs.jython.org/issue1639 (see also http://xwiki.475771.n2.nabble.com/Deploying-on-JBoss-5-td5390212.html) but on JBoss AS 7 (7.1.1Final to be precise).
This time the problem is that JBoss AS7 has introduced the "vfs" protocol (and no longer "vfszip"), thus the JAR is not considered standalone and thus libs in the standalone's Lib/ directory are not taken into account...
The URL I get is from getJarFileName() is:
vfs:/Users/vmassol/Desktop/jboss-as-7.1.1.Final/standalone/deployments/xwiki.war/WEB-INF/lib/jython-standalone-2.5.2.jar/org/python/core/PySystemState.class
For details, see http://jira.xwiki.org/browse/XWIKI-7984
Thanks a lot. Would be awesome if this could make it in 2.5.3 :)
|
msg7278 (view) |
Author: Frank Wierzbicki (fwierzbicki) |
Date: 2012-06-29.14:42:43 |
|
If I'm following correctly it looks like http://bugs.jython.org/issue1639 was applied before beta2 came out - do you still see an issue on beta2? If not do you have a suggested fix?
|
msg7279 (view) |
Author: Vincent Massol (vmassol) |
Date: 2012-06-29.14:46:34 |
|
http://bugs.jython.org/issue1639 was in 2.5.2-rc1 (according to release notes on http://www.jython.org/latest.html).
I'm running 2.5.2 final. But this is not the same issue1639; it's similar to different. In issue 1639 the fix was to add support for the "vfszip" protocol. This issue is about also adding support for the "vfs" protocol which is in use in JBoss AS7. The fix is the same though; except checking for "vfs:" instead of "vfszip:".
Thanks
|
msg7281 (view) |
Author: Frank Wierzbicki (fwierzbicki) |
Date: 2012-06-29.16:30:32 |
|
It sounds like a pretty trivial patch, but someone would need to make sure both "vfszip" and "vfs" will work. I don't have time before 2.5.3beta to look at this myself (since it would involve setting up JBoss instances etc for testing) but I'd be very willing to review a patch that did the same. I'll add Otmar to the nosy list to see if he has input since he accepted the other patch.
|
msg7464 (view) |
Author: Olivier Heurtier (olivierH) |
Date: 2012-09-27.16:06:38 |
|
Here is a proposition of patch based on the implementation done for vfszip. Tested (only) with JBoss AS 7.1.1.Final on Ubuntu12.04.
|
msg7569 (view) |
Author: Vincent Massol (vmassol) |
Date: 2013-01-11.13:51:33 |
|
Guys, any news on this. I'd really love to have this fixed. I see olivierH added a patch. Is there a binary somewhere that I could use to test it (if that's what's holding the closing of the issue)?
Thanks a lot
|
msg7570 (view) |
Author: Frank Wierzbicki (fwierzbicki) |
Date: 2013-01-12.19:59:28 |
|
Vincent: If you can verify that olivierH's patch fixes things for you, I'll be happy to apply it.
|
msg7571 (view) |
Author: Frank Wierzbicki (fwierzbicki) |
Date: 2013-01-12.20:03:11 |
|
olivierH: Do you want me to include your full name in the commit message or should I just go with "olivierH"? The patch is small enough that I won't need to bug you about a contributor agreement - though if you wouldn't mind signing one I'll point you to the instructions.
|
msg7573 (view) |
Author: Olivier Heurtier (olivierH) |
Date: 2013-01-16.20:50:57 |
|
Frank: you can use my full name. I will be happy to sign a contributor
agreement if it can make your work easier.
|
msg7574 (view) |
Author: Frank Wierzbicki (fwierzbicki) |
Date: 2013-01-16.22:23:30 |
|
Olivier Heurtier: cool, I'll plan to use your full name when Vincent verifies the fix. The instructions for signing a contributor agreement are here: http://www.python.org/psf/contrib/ - and the right form is here: http://www.python.org/psf/contrib/contrib-form/ Note that they want a bugs.python.org username and not a bugs.jython.org one ("p" instead of "j")
|
msg7585 (view) |
Author: Olivier Heurtier (olivierH) |
Date: 2013-01-23.20:54:33 |
|
I have signed and sent the contributor agreement today.
|
msg7619 (view) |
Author: Frank Wierzbicki (fwierzbicki) |
Date: 2013-02-05.15:49:26 |
|
Olivier: great! Now if we can just get Vincent to confirm that his issue is fixed. Vincent?
|
msg7626 (view) |
Author: Vincent Massol (vmassol) |
Date: 2013-02-06.15:09:55 |
|
Is there a binary I can get to retest?
|
msg7644 (view) |
Author: Frank Wierzbicki (fwierzbicki) |
Date: 2013-02-08.00:03:43 |
|
Fixed in http://hg.python.org/jython/rev/df829bcbd4f3
Thanks Olivier Heurtier!
Vincent: the fix will be in 2.5.4rc1 and 2.7beta1 which I expect to get out this week. Let me know how it goes! I won't close the bug until you confirm that it is fixed.
|
msg7645 (view) |
Author: Vincent Massol (vmassol) |
Date: 2013-02-08.08:22:25 |
|
Great. I'll retest when 2.5.4rc1 or 2.7beta1 are out, thanks.
|
msg7670 (view) |
Author: Vincent Massol (vmassol) |
Date: 2013-02-13.10:11:51 |
|
Seems like 2.5.4RC1 fixed the issue! See http://jira.xwiki.org/browse/XWIKI-8812
Thanks a lot :)
|
msg7677 (view) |
Author: Frank Wierzbicki (fwierzbicki) |
Date: 2013-02-13.23:00:21 |
|
Vincent: that's great! I'll go ahead and close this now. Thanks again for the patch Olivier!
|
|
Date |
User |
Action |
Args |
2013-02-13 23:27:16 | fwierzbicki | set | status: open -> closed resolution: fixed |
2013-02-13 23:00:21 | fwierzbicki | set | messages:
+ msg7677 |
2013-02-13 10:11:51 | vmassol | set | messages:
+ msg7670 |
2013-02-08 08:22:25 | vmassol | set | messages:
+ msg7645 |
2013-02-08 00:03:43 | fwierzbicki | set | messages:
+ msg7644 |
2013-02-06 15:09:56 | vmassol | set | messages:
+ msg7626 |
2013-02-05 15:49:26 | fwierzbicki | set | messages:
+ msg7619 |
2013-01-23 20:54:33 | olivierH | set | messages:
+ msg7585 |
2013-01-16 22:23:30 | fwierzbicki | set | messages:
+ msg7574 |
2013-01-16 20:50:57 | olivierH | set | messages:
+ msg7573 |
2013-01-12 20:03:11 | fwierzbicki | set | priority: normal assignee: fwierzbicki messages:
+ msg7571 |
2013-01-12 19:59:28 | fwierzbicki | set | messages:
+ msg7570 |
2013-01-11 13:51:33 | vmassol | set | messages:
+ msg7569 |
2012-09-27 16:06:39 | olivierH | set | files:
+ patch1936.txt nosy:
+ olivierH messages:
+ msg7464 |
2012-07-31 07:34:26 | tuska | set | nosy:
+ vmassol |
2012-07-31 07:28:05 | tuska | set | nosy:
+ tuska, - vmassol |
2012-07-17 05:04:25 | amak | set | nosy:
+ amak |
2012-06-29 16:30:32 | fwierzbicki | set | nosy:
+ otmarhumbel messages:
+ msg7281 |
2012-06-29 14:46:34 | vmassol | set | messages:
+ msg7279 |
2012-06-29 14:42:43 | fwierzbicki | set | messages:
+ msg7278 |
2012-06-29 14:38:41 | fwierzbicki | set | nosy:
+ fwierzbicki |
2012-06-29 12:27:50 | vmassol | create | |
|