[PUSHED] Remove "<foo>::existence" Perl modules.

Michael Meeks michael.meeks at suse.com
Sat Feb 18 01:46:28 PST 2012


Hi Tim,

On Fri, 2012-02-17 at 23:40 +0000, Tim Retout wrote:
> This patch series removes several versions of Perl modules named
> <foo>::existence - the functions can be reimplemented with the "grep"
> keyword.

	Right :-)

> Some error messages may change in patch 9, but the error message logic
> in get_specified_file_by_name was not clear to me, (even checking the
> context of CWS nativefixer6 that introduced it):

	That's fine, no one is parsing the error messages from
make_installer.pl - feel free to improve them any way you like.

> Oh, and there should be two new unit tests in there, too - not
> complete code coverage by any means, but they were useful during
> development to prove that I wasn't breaking the tested functions.

	Right - I guess we need to do some gnumake research to work out how to
hook them up.

> Now that I've done such a huge patch series, I wonder if there was a
> better way to approach this to make it easier to review... :(

	Given that it all looks beautiful & works nicely AFAICS, I wouldn't
worry :-)

	Thanks,

		Michael.

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



More information about the LibreOffice mailing list