Skip Menu |
Report information
Id: 128980
Status: open
Priority: 0/
Queue: perl5

Owner: Nobody
Requestors: mojca.miklavec [at] macports.org
Cc: craig.a.berry [at] gmail.com
jhi <jhi [at] iki.fi>
AdminCc:

Operating System: (no value)
PatchStatus: (no value)
Severity: low
Type: unknown
Perl Version: (no value)
Fixed In: (no value)



CC: "Jarkko Hietaniemi" <jhi [...] iki.fi>, "Craig Berry" <craig.a.berry [...] gmail.com>
Subject: Please backport #126360: MACOSX_DEPLOYMENT_TARGET to 5.22[.3]
Download (untitled) / with headers
text/plain 551b
Building perl < 5.24 is broken on OS X 10.12. (Partially due to an accidental omission of one library, but it's generally useless to set the target to 10.3 anyway.) The following patch fixed it: * http://perl5.git.perl.org/perl.git/commit/53d1d41c81e1de9cc6416dcae828c13d4c5a470a * https://rt.perl.org/Ticket/Display.html?id=126360 but it is only available in perl5.24 and 5.25. Can you please backport the patch also to 5.22? On a related note, why does the code include MACOSX_DEVELOPMENT_TARGET at one place rather than MACOSX_DEPLOYMENT_TARGET?
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 229b
Show quoted text
> On a related note, why does the code include MACOSX_DEVELOPMENT_TARGET > at one place rather than MACOSX_DEPLOYMENT_TARGET?
Typo, fixed in http://perl5.git.perl.org/perl.git/commitdiff/970ebd6f29eaeb8f281ee05c3de6b064ac181673
From: "Craig A. Berry" <craig.a.berry [...] gmail.com>
CC: bugs-bitbucket [...] rt.perl.org
Date: Thu, 18 Aug 2016 08:27:53 -0500
To: "Perl5 Porters (E-mail)" <perl5-porters [...] perl.org>
Subject: Re: [perl #128980] Please backport #126360: MACOSX_DEPLOYMENT_TARGET to 5.22[.3]
Download (untitled) / with headers
text/plain 1.3k
On Thu, Aug 18, 2016 at 1:23 AM, Mojca Miklavec <perlbug-followup@perl.org> wrote: Show quoted text
> # New Ticket Created by Mojca Miklavec > # Please include the string: [perl #128980] > # in the subject line of all future correspondence about this issue. > # <URL: https://rt.perl.org/Ticket/Display.html?id=128980 > > > > Building perl < 5.24 is broken on OS X 10.12. (Partially due to an accidental omission of one library, but it's generally useless to set the target to 10.3 anyway.) > > The following patch fixed it: > * http://perl5.git.perl.org/perl.git/commit/53d1d41c81e1de9cc6416dcae828c13d4c5a470a > * https://rt.perl.org/Ticket/Display.html?id=126360 > but it is only available in perl5.24 and 5.25. > > Can you please backport the patch also to 5.22?
Changes have to be binary compatible to be candidates for backporting. I don't think this one is, at least not in its current form, because the build target in a default configuration will be different from what it was before. It may be that overriding the default by honoring the setting of MACOSX_DEPLOYMENT_TARGET would be acceptable for older perls. Show quoted text
> On a related note, why does the code include MACOSX_DEVELOPMENT_TARGET at one place rather than MACOSX_DEPLOYMENT_TARGET?
That's been fixed: <http://perl5.git.perl.org/perl.git/commitdiff/970ebd6f29eaeb8f281ee05c3de6b064ac181673>
Subject: Re: [perl #128980] Please backport #126360: MACOSX_DEPLOYMENT_TARGET to 5.22[.3]
To: "Perl5 Porters (E-mail)" <perl5-porters [...] perl.org>
Date: Thu, 18 Aug 2016 09:06:03 -0500
CC: bugs-bitbucket [...] rt.perl.org
From: "Craig A. Berry" <craig.a.berry [...] gmail.com>
Download (untitled) / with headers
text/plain 1.4k
On Thu, Aug 18, 2016 at 8:27 AM, Craig A. Berry <craig.a.berry@gmail.com> wrote: Show quoted text
> On Thu, Aug 18, 2016 at 1:23 AM, Mojca Miklavec > <perlbug-followup@perl.org> wrote:
>> # New Ticket Created by Mojca Miklavec >> # Please include the string: [perl #128980] >> # in the subject line of all future correspondence about this issue. >> # <URL: https://rt.perl.org/Ticket/Display.html?id=128980 > >> >> >> Building perl < 5.24 is broken on OS X 10.12. (Partially due to an accidental omission of one library, but it's generally useless to set the target to 10.3 anyway.) >> >> The following patch fixed it: >> * http://perl5.git.perl.org/perl.git/commit/53d1d41c81e1de9cc6416dcae828c13d4c5a470a >> * https://rt.perl.org/Ticket/Display.html?id=126360 >> but it is only available in perl5.24 and 5.25. >> >> Can you please backport the patch also to 5.22?
> > Changes have to be binary compatible to be candidates for backporting. > I don't think this one is, at least not in its current form, because > the build target in a default configuration will be different from > what it was before. It may be that overriding the default by honoring > the setting of MACOSX_DEPLOYMENT_TARGET would be acceptable for older > perls.
However, if there are versions of OS X for which 5.22 won't build at all, then "no Perl for you" isn't the preferred mechanism to preserve binary compatibility :-). But I remember warnings, not build failures, so educate me about which combinations don't work at all, if any.
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 303b
On Wed Aug 17 23:23:51 2016, mojca.miklavec.lists@gmail.com wrote: Show quoted text
> Building perl < 5.24 is broken on OS X 10.12. (Partially due to an > accidental omission of one library, but it's generally useless to set > the target to 10.3 anyway.)
Could you show here how it fails? Not all of us have Sierra.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

For issues related to this RT instance (aka "perlbug"), please contact perlbug-admin at perl.org