How do we deploy the war file? As far as I know, deploying a war file to tomcat would not strip executed bit on files, tomcat just simple unzip it
> -----Original Message----- > From: Min Chen [mailto:min.c...@citrix.com] > Sent: Tuesday, March 05, 2013 2:56 PM > To: cloudstack-dev@incubator.apache.org > Cc: Hugo Trippaers (trip...@gmail.com) > Subject: Re: copying scripts in client/pom.xml > > This fix will still not solve the issue for developer environment, since we > are > running scripts from client/target/cloud-client-ui-4.2.0-SNAPSHOT/WEB- > INF/classes/scripts when we launch MS through maven. When maven > deploys war to that folder, it will strip the permission set in generated- > webapp. > > Thanks > -min > > On 3/5/13 12:11 AM, "Hugo Trippaers" <htrippa...@schubergphilis.com> > wrote: > > >Hey Frank, > > > >I think I found the same thing, please check this commit > >https://git-wip-us.apache.org/repos/asf?p=incubator-cloudstack.git;a=co > >mmi > >t;h=d922f6fd4b41d939ee00c55638d8c5e8b56c8b77 > > > >Does that work for you? > > > >Cheers, > > > >Hugo > > > >> -----Original Message----- > >> From: Frank Zhang [mailto:frank.zh...@citrix.com] > >> Sent: Tuesday, March 05, 2013 2:33 AM > >> To: cloudstack-dev@incubator.apache.org > >> Cc: Hugo Trippaers (trip...@gmail.com) > >> Subject: RE: copying scripts in client/pom.xml > >> > >> No worry. > >> I found a solution that explicitly call 'chmod' task of ant to add > >>executed bits. > >> I will fix it tomorrow > >> > >> > -----Original Message----- > >> > From: Hugo Trippaers [mailto:htrippa...@schubergphilis.com] > >> > Sent: Saturday, March 02, 2013 11:35 AM > >> > To: cloudstack-dev@incubator.apache.org > >> > Cc: Hugo Trippaers (trip...@gmail.com) > >> > Subject: RE: copying scripts in client/pom.xml > >> > > >> > Hey Frank, > >> > > >> > I reopened the ticket with the following comment: > >> > > >> > I reverted this commit because it breaks the system for developers > >> > who use an operating system that has no cp or mkdir command (the > >> > mkdir was later added by Rohit) > >> > > >> > I did not notice the original issue because the executable bit is > >> > not an issue on windows systems. We should find an alternative way > >> > to do this to solve this problem in a way that works on all > >> > operating systems. It's mainly a developer problem as the > >> > permissions are set using the packaging scripts for deployments. > >> > > >> > A check if mkdir and cp exists before doing the copy should work i > >> > think, but maybe there is an ant task that will take care of the > >> > permissions on unix based systems. Or maybe explicitly call bash > >> > instead of depending to the os to execute the script. I don't have > >> > much time this weekend, but I'm willing to see if I can find a > >>solution next > >> week. > >> > > >> > > >> > Cheers, > >> > > >> > Hugo > >> > > >> > > -----Original Message----- > >> > > From: Chip Childers [mailto:chip.child...@sungard.com] > >> > > Sent: zaterdag 2 maart 2013 20:01 > >> > > To: cloudstack-dev@incubator.apache.org > >> > > Cc: Hugo Trippaers (trip...@gmail.com) > >> > > Subject: Re: copying scripts in client/pom.xml > >> > > > >> > > On Fri, Mar 01, 2013 at 03:20:38PM -0800, Frank Zhang wrote: > >> > > > Hi Hugo: > >> > > > I noticed your recent change as below. It will cause bug > >> > > > https://issues.apache.org/jira/browse/CLOUDSTACK-1304 that I > >> > > > have fixed some days ago > >> > > > > >> > > > > >> > > > be141f6e (Hugo Trippaers 2013-03-01 08:22:40 +0100 271) > >> <copy > >> > > todir="${basedir}/target/generated-webapp/WEB-INF/classes/scripts > >> > > "> > >> > > > be141f6e (Hugo Trippaers 2013-03-01 08:22:40 +0100 272) > >> > > <fileset dir="${basedir}/../scripts" /> > >> > > > be141f6e (Hugo Trippaers 2013-03-01 08:22:40 +0100 273) > >> > > </copy> > >> > > > > >> > > > > >> > > > my fixes basically does: > >> > > > > >> > > > <exec executable="cp"> > >> > > > <arg value="-r" /> > >> > > > <arg value="${basedir}/../scripts" /> > >> > > > <arg > >> > > > value="${basedir}/target/generated-webapp/WEB- > >> > > INF/classes/scripts" /> > >> > > > </exec> > >> > > > > >> > > > Is there any special reason you changed it back? > >> > > > Thank you > >> > > > > >> > > > >> > > Frank / Hugo - Should we reopen CLOUDSTACK-1304? > >> > > > >> > > I'd also suggest that the description of the bug makes it a bit > >> > > higher than the Major designation (i.e.: we should fix it in 4.1).
