[jira] [Updated] (TAP5-2004) Tapestry-Ioc fails to build on Windows (ReloadTest.java)

["Bob Harner (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/TAP5-2004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bob Harner updated TAP5-2004:
-----------------------------

    Assignee: Bob Harner
    
> Tapestry-Ioc fails to build on Windows (ReloadTest.java)
> --------------------------------------------------------
>
>                 Key: TAP5-2004
>                 URL: https://issues.apache.org/jira/browse/TAP5-2004
>             Project: Tapestry 5
>          Issue Type: Bug
>          Components: tapestry-ioc
>    Affects Versions: 5.3
>            Reporter: Bob Harner
>            Assignee: Bob Harner
>            Priority: Minor
>              Labels: patch
>         Attachments: Patch for TAP5-2004.diff
>
>
> When building Tapestry itself on a Windows box, 5 tests fail in the ioc module:
> org.apache.tapestry5.ioc.internal.OperationException: Unable to load class com.example.ReloadableServiceImpl: URI is not hierarchical
> 	at org.apache.tapestry5.ioc.internal.OperationTrackerImpl.logAndRethrow(OperationTrackerImpl.java:121)
> 	at org.apache.tapestry5.ioc.internal.OperationTrackerImpl.invoke(OperationTrackerImpl.java:88)
> 	at org.apache.tapestry5.ioc.internal.PerThreadOperationTracker.invoke(PerThreadOperationTracker.java:87)
> 	at org.apache.tapestry5.ioc.internal.RegistryImpl.invoke(RegistryImpl.java:1124)
> 	at org.apache.tapestry5.ioc.internal.AbstractReloadableObjectCreator.createInstance(AbstractReloadableObjectCreator.java:126)
> 	at org.apache.tapestry5.ioc.internal.AbstractReloadableObjectCreator.createObject(AbstractReloadableObjectCreator.java:118)
> 	at $ReloadableService_f12317cafcd3.delegate(Unknown Source)
> 	at $ReloadableService_f12317cafcd3.getStatus(Unknown Source)
> 	at $ReloadableService_f12317cafcd1.getStatus(Unknown Source)
> 	at org.apache.tapestry5.ioc.ReloadTest.delete_class(ReloadTest.java:158)
> ...
> As Chris Poulsen reported on Jun 29, 2011 at http://tapestry.1045711.n5.nabble.com/gradle-test-troubles-when-building-tapestry-ioc-from-trunk-td4536955.html:
>   > It turns out that the creation of the temp-dir in ReloadTest.java doesn't
>   > work 100% like expected on windows. The generated url ends up as:
>   > "file:c:/blahblah/xxxx/" while it should be "file:/c:/blahblah/xxxx/" (slash
>   > before the drive letter).
> Chris' fix is simple enough, and I've verified that it resolves the problem (the build on Windows has 5 failures without this change, and 0 failures with the change in place):
> ReloadTest.java:71
> -        URL classesURL = new URL("file:" + classesDir.getCanonicalPath() + "/");
> +        URL classesURL =  new URL(classesDir.toURI().toString()+"/");
> I'll gladly commit the fix if I can get at least one person on Linux and/or Mac to verify that this change doesn't break the tests there. (I have only a Windows box currently.)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira