[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