Re: Minimum Perl version requirement (was: Re: [PUSHED] Replace make_absolute_path with rel2abs in solenv)

2012-02-20 Thread Tim Retout
On 20 February 2012 16:07, Michael Meeks  wrote:
>        I poked Fridrich, the oracle on this sort of thing - wrt. our old
> build-systems. On windows we have perl 5.10 but on Linux 5.8.8 ( we have
> an artificially old Linux machine to produce forward-compatible builds).

There are some nice things in 5.10 - "defined-or", named regexp
captures, given/when...

5.8 is just about okay - it's what I still use at work ;) - so I'll
ping you again in 3 years or so. :)

-- 
Tim Retout 
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Minimum Perl version requirement (was: Re: [PUSHED] Replace make_absolute_path with rel2abs in solenv)

2012-02-20 Thread Michael Meeks
Hi Tim,

On Mon, 2012-02-20 at 13:47 +, Tim Retout wrote:
> $ corelist File::Spec

Nice :-)

> Which leads to the question: what is the minimum version of Perl we
> can rely on in LibreOffice?  Currently configure.in checks for Perl 5,

I poked Fridrich, the oracle on this sort of thing - wrt. our old
build-systems. On windows we have perl 5.10 but on Linux 5.8.8 ( we have
an artificially old Linux machine to produce forward-compatible builds).

> Would it be reasonable to bump the configure.in check to look for Perl
> 5.10, for now?  I might try to replace the macro with one from the
> autoconf archive:

So I guess 5.8 is a sensible baseline ? or does that loose us some
things we really want ?

ATB,

Michael.

-- 
michael.me...@suse.com  <><, Pseudo Engineer, itinerant idiot

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Minimum Perl version requirement (was: Re: [PUSHED] Replace make_absolute_path with rel2abs in solenv)

2012-02-20 Thread Tim Retout
On 20 February 2012 11:30, Michael Meeks  wrote:
> On Sun, 2012-02-19 at 14:39 +, Tim Retout wrote:
>        Great :-) thanks for that, it's always nice to use standard methods
> instead of custom / hand-coded ones. Clearly we want to introduce new
> perl dependencies on shared modules only with some care - it's nice
> having solenv/ be reasonably self-contained.

Indeed - a useful tool is 'corelist', which will print the first
version of Perl which shipped a given CPAN module in the core:

$ corelist File::Spec

File::Spec was first released with perl 5.00405

(i.e. Perl 5.4)

Which leads to the question: what is the minimum version of Perl we
can rely on in LibreOffice?  Currently configure.in checks for Perl 5,
but the Perl currently in solenv almost certainly requires Perl 5.8
(first released 2002).

Even Perl 5.10 (first released 2007) is no longer maintained upstream,
although it is still the one shipped in Debian squeeze/lenny (current
stable/oldstable):

 http://www.cpan.org/src/
 http://packages.debian.org/squeeze/perl

Would it be reasonable to bump the configure.in check to look for Perl
5.10, for now?  I might try to replace the macro with one from the
autoconf archive:

http://git.savannah.gnu.org/gitweb/?p=autoconf-archive.git;a=blob_plain;f=m4/ax_prog_perl_version.m4

Kind regards,

-- 
Tim Retout 
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PUSHED] Replace make_absolute_path with rel2abs in solenv

2012-02-20 Thread Michael Meeks
Hi Tim,

On Sun, 2012-02-19 at 14:39 +, Tim Retout wrote:
> These three patches remove some home-grown path manipulation from
> solenv, and replace it with functions shipped with the Perl core.

Great :-) thanks for that, it's always nice to use standard methods
instead of custom / hand-coded ones. Clearly we want to introduce new
perl dependencies on shared modules only with some care - it's nice
having solenv/ be reasonably self-contained.

Thanks !

Michael.

-- 
michael.me...@suse.com  <><, Pseudo Engineer, itinerant idiot

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice