[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-release-team
Subject:    Re: Perl version in KDELibs
From:       Frederik Schwarzer <schwarzer () kde ! org>
Date:       2011-06-06 21:14:25
Message-ID: 201106062314.26220.schwarzer () kde ! org
[Download RAW message or body]

[Raphael Kubo da Costa - Montag, 6. Juni 2011 11:18:48] 
> Michael Pyne <mpyne@kde.org> writes:
> 
> > On Sunday, June 05, 2011 22:07:25 Raphael Kubo da Costa wrote:
> >> Frederik Schwarzer <schwarzer@kde.org> writes:
> >> > So I changed the affected script and let it require Perl 5.10.
> >> > 
> >> > Now I wanted to backport the change to 4.6 but I am not sure if that is
> >> > OK due to the version bump and the with it changed dependency. So I
> >> > looked at kdelibs' CMakeLists.txt and found out that it does not state
> >> > a version.
> >> > 
> >> > Does that mean it would also be satisfied by Perl 5.4 or something?
> >> 
> >> Yes.
> >> 
> >> > Should we set a minimum version there?
> >> 
> >> Well, we use CMake's FindPerl.cmake, which does not check Perl's version
> >> at all. If that's really needed, we could add a FindPerl.cmake to
> >> kdelibs, but the kde-buildsystem mailing list is a better place to
> >> discuss that.
> >
> > AFAICS CMake's FindPerl supports requesting a minimum version just fine, we 
> > just were not setting a minimum version to look for.
> 
> I'm using CMake 2.8.4 here, and its FindPerl.cmake only looks for an
> executable called "perl", without any version checking.
> 
> FindPerlLibs.cmake, on the other hand, does that version check if
> required. Perhaps we could use that instead of FindPerl
> (FindPerlLibs.cmake does `include(FindPerl)')?

I think it is a different thing. While FindPerl looks for the executable,
FindPerlLibs looks for the library. Yes, the Lib check includes the check
for the executable but also adds another dependency ... at least in Debian
the Lib is a different package.

But yes, this would be a discussion for buildsystem. I would like to
concentrate on the "can I backport a change that raises the required
version for Perl from 5.8 to 5.10" on this list. :)

Regards
_______________________________________________
release-team mailing list
release-team@kde.org
https://mail.kde.org/mailman/listinfo/release-team
[prev in list] [next in list] [prev in thread] [next in thread]