[Libreoffice] [PATCH] installer::archivefiles - remove external find and chmod forks

Michael Meeks michael.meeks at novell.com
Fri Jan 7 08:15:53 PST 2011


Hi Steve,

On Mon, 2010-12-20 at 20:17 +1000, Steve Butler wrote:
> I just noticed my patch uses 0755 instead of 0775 as well - I'm not
> sure why you'd give group write access but that's what the original
> does.

	Apologies that this took so long to get reviewed and merged :-) I just
committed it. Whether it is truly useful or not, what is there must be
truly slow on Windows, which can't fork/exec anything in linear time :-)
[ and which is the only platform we do this on ]. Hopefully it'll speed
up the installer there.

	Thank you ! :-)

	It'd be great if you could perhaps clone / cut/paste part of the
relevant Easy Hack to the 'completed easy hacks' page, and/or its great
to have a statement saying the patch is LGPLv2+/MPL as you post it.

	Thanks again & what else are you interested in ? [ there is a lot of
low-hanging perl fruit in our make_installer world I think - simply
running it under a profiler and doing some optimisation would be highly
fruitful I think ].

	All the best,

		Michael.

-- 
 michael.meeks at novell.com  <><, Pseudo Engineer, itinerant idiot




More information about the LibreOffice mailing list