Hi Mat, On Thu, 2008-04-03 at 01:14 +0100, Mat Booth wrote: > On Tue, Apr 1, 2008 at 3:29 AM, Andrew Overholt <overholt@xxxxxxxxxx> wrote: > > > > Yes. I didn't want to mess with copy-platform since I wanted to get > > this script into a build of the SDK. If you -- or anyone else -- has a > > few minutes to make this modification, I'd appreciate any patches. > > > > Andrew > > > > Sure thing, see attached. This patch changes > "eclipse-copy-platform.sh" and "eclipse.spec" from the Rawhide eclipse > package. I hope that's ok. Awesome, thanks! My only nit is this: -sed -e's/^\(.*\)$/\1 \1/' -e's,^,ln -s $eclipse/,' >> copy-platform +sed -e's/^\(.*\)$/\1/' -e's,^\(.*\),[ ! -e \1 ] \&\& ln -s $eclipse/\1 \1,' >> copy-platform I don't think you need the first sed command, do you? Would removing it still accomplish the same thing? Like this: sed -e's,^\(.*\),[ ! -e \1 ] \&\& ln -s $eclipse/\1 \1,' Did you want to just commit this? If you request commit on the eclipse module in the account system, I'll grant it. Thanks again, Andrew -- fedora-devel-java-list mailing list fedora-devel-java-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/fedora-devel-java-list
