Re: Heads Up: big change to deployment
Well looks like this doesn't work on windows. I don't think windows is letting me move the temp directory to the config store because the URLClassLoader on windows holds locks open on the jar files inside of the config store. I'll add some code that copies the files from the temp dir into the config store. I hope later we can just have our own implementation of URLClassLoader that we can call close on to release open file locks on windows. This shouldn't take about an hour to fix. -dain -- Dain Sundstrom Chief Architect Gluecode Software 310.536.8355, ext. 26 On Oct 4, 2004, at 4:01 PM, Dain Sundstrom wrote: I just committed a big change to deployment... I have changed deployment to no longer build into a packed Jar file, instead it builds into a temp directory and once complete moves the temp directory into the config store. If a packed car file is needed, the deployer simply jars up the temp directory. This should make adding support for manifest class path entries trivial, as the standard URLClassLoader can handle like 80% of the cases. I also added code to meticulously track temp files and delete them asap... my old implementation left tons of temp files around. I haven't tested this on windows yet, but I heading over to my windows machine right now to test it. Can someone on linux check if the build works? Thanks, -dain -- Dain Sundstrom Chief Architect Gluecode Software 310.536.8355, ext. 26
Re: Heads Up: big change to deployment
On Mon, 4 Oct 2004, Dain Sundstrom wrote: Can someone on linux check if the build works? At first glance, no: BUILD FAILED File.. /home/ammulder/.maven/cache/maven-multiproject-plugin-1.3.1/plugin.jelly Element... maven:reactor Line.. 217 Column 9 Unable to obtain goal [multiproject:install-callback] -- /data/cvs/geronimo/modules/assembly/maven.xml:205:15: bootstrap:bootstrap Could not move source directory into config store: source=/tmp/geronimo-deployment-64539.tmp, destination=/data/cvs/geronimo/modules/assembly/target/geronimo-1.0-SNAPSHOT/config-store/13 Total time: 3 minutes 1 seconds Finished at: Mon Oct 04 19:47:58 EDT 2004 Aaron
Re: Heads Up: big change to deployment
All that stuff worked for me on a fresh checkout on the windows pc I have hiding in the corner :) It looks like you didn't update openejb. I get an exception moving the temp dir into the config store. -dain On Oct 4, 2004, at 5:22 PM, Aaron Mulder wrote: Okay, when I tried a maven complete-rebuild, I got: The build cannot continue because of the following unsatisfied dependency: geronimo-client-builder-1.0-SNAPSHOT.jar If I try to build the client-builder module, it can't find some openejb stuff: data/cvs/geronimo/modules/client-builder/src/java/org/apache/geronimo/ client/builder/AppClientModuleBuilder.java:70: package org.openejb.client.naming does not exist import org.openejb.client.naming.RemoteEJBObjectFactory; ^ /data/cvs/geronimo/modules/client-builder/src/java/org/apache/ geronimo/client/builder/AppClientModuleBuilder.java:71: package org.openejb.client.naming does not exist import org.openejb.client.naming.RemoteEJBRefAddr; Then I tried a maven multiproject:install, and I got this: /data/cvs/geronimo/openejb/modules/core/src/java/org/openejb/ deployment/OpenEJBModuleBuilder.java:67: cannot resolve symbol symbol : class IOUtil location: package util import org.apache.geronimo.deployment.util.IOUtil; ^ /data/cvs/geronimo/openejb/modules/core/src/java/org/openejb/ deployment/OpenEJBModuleBuilder.java:68: cannot resolve symbol symbol : class JarUtil location: package util import org.apache.geronimo.deployment.util.JarUtil; ... It's hard to troubleshoot when I get a different error every time. :) Aaron On Mon, 4 Oct 2004, Aaron Mulder wrote: On Mon, 4 Oct 2004, Dain Sundstrom wrote: Can someone on linux check if the build works? At first glance, no: BUILD FAILED File.. /home/ammulder/.maven/cache/maven-multiproject-plugin-1.3.1/ plugin.jelly Element... maven:reactor Line.. 217 Column 9 Unable to obtain goal [multiproject:install-callback] -- /data/cvs/geronimo/modules/assembly/maven.xml:205:15: bootstrap:bootstrap Could not move source directory into config store: source=/tmp/geronimo-deployment-64539.tmp, destination=/data/cvs/geronimo/modules/assembly/target/geronimo-1.0- SNAPSHOT/config-store/13 Total time: 3 minutes 1 seconds Finished at: Mon Oct 04 19:47:58 EDT 2004 Aaron
Re: Heads Up: big change to deployment
There is still a bug with resolving URIs on Windows. It looks like you can't resolve an absolute file location (e.g., c:\a\b\c) against a file uri unless you specify the file protocol on the uri. Anyway, I'm about to commit it... just verifying the change. -dain -- Dain Sundstrom Chief Architect Gluecode Software 310.536.8355, ext. 26 On Oct 4, 2004, at 6:46 PM, Aaron Mulder wrote: On Mon, 4 Oct 2004, Dain Sundstrom wrote: All that stuff worked for me on a fresh checkout on the windows pc I have hiding in the corner :) It looks like you didn't update openejb. I get an exception moving the temp dir into the config store. You're right -- I had to update all the child projects first. Then I was back to the same error as before, but it seems to be working fine after your latest commit. Aaron