search

DPascalBernard / maven-alfresco-archetypes

Automatically exported from code.google.com/p/maven-alfresco-archetypes
0 stars 0 forks source link

Dependent AMPs can't be deployed to Alfresco extension WAR on windows #16

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
See these threads
http://groups.google.com/group/maven-
alfresco/browse_thread/thread/4cf30e77dc9809f6
http://groups.google.com/group/maven-
alfresco/browse_thread/thread/4f5a5e03be2f299b

Errors are 
String index out of range: 0 

And /lib and /config folders in the AMP ending up in the wrong place in the 
war.

Original issue reported on code.google.com by damonr...@gmail.com on 15 Oct 2008 at 10:41

GoogleCodeExporter commented 9 years ago 
This is being caused by file separator problems in the UnAmpArchiver class. 
This is 
why it works on Linux but not on Windows. Attaching a patched version.

Patch applies to r118 of 
https://forge.alfresco.com/svn/maven4alfresco/amp-
plugin/branches/2.0.0/src/main/java/org/alfresco/plexus/archiver/AmpUnArchiver.j
ava

Original comment by damonr...@gmail.com on 15 Oct 2008 at 10:44

Attachments:

GoogleCodeExporter commented 9 years ago 
Hi Damon,
first of all thanks for the contributions...so much appreciated.

BTW, did you already commit this patch in the maven-amp-plugin code so that I 
can
just rerelease it to SS repo or shall I do it? (I see you're a member of that 
project
too)

I'm planning to do it today (today as, geniously, I have a client which is not 
able
to build my app on windows :)

Original comment by colum...@gmail.com on 15 Oct 2008 at 11:36

GoogleCodeExporter commented 9 years ago 
Happy to help. :-)

Actually I don't know if I have commit access to that repos so I'll let you do 
it.. 
Besides I'm not sure it works on Linux now so you better test again before 
committing!

Damon.

Original comment by damonr...@gmail.com on 15 Oct 2008 at 12:03

GoogleCodeExporter commented 9 years ago 
Ok , gonna test now and release after wards ;)

Keep you posted...

Original comment by colum...@gmail.com on 15 Oct 2008 at 12:35

GoogleCodeExporter commented 9 years ago 
Attaching a slightly better patch. My previous one had a problem with the /web 
folder.

Original comment by damonr...@gmail.com on 16 Oct 2008 at 3:08

Attachments: