Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#40521 closed defect (fixed)

Build error perl5.x returns for Mavericks

Reported by: Serge3leo (Serguei E. Leontiev) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.2.0
Keywords: mavericks Cc: whcordis@…, msy@…, sck@…, wwboy6@…, Ionic (Mihai Moldovan), chema.torrelles@…
Port: perl5.8

Description

Original error #28603 was not completely fixed r76831, r81168 and r96453.

For the Mac OS X 10.9 also need a patch:

--- ../trunk/dports/lang/perl5.8/Portfile	2013-09-14 02:54:54.000000000 +0400
+++ /opt/local/var/macports/sources/rsync.macports.org/release/tarballs/ports/lang/perl5.8/Portfile	2013-09-18 10:28:32.000000000 +0400
@@ -31,16 +31,11 @@
 
 patchfiles  patch-perl.c.diff patch-Configure.diff
 
-platform darwin 11 {
-    # bypass stupid check that assumes fprintf must be in libc.dylib
-    patchfiles-append   patch-hints_darwin.sh-usenm.diff
-}
-platform darwin 12 {
-    # bypass stupid check that assumes fprintf must be in libc.dylib
-    patchfiles-append   patch-hints_darwin.sh-usenm.diff
-}
-
 platform darwin {
+    if {${os.major} >= 11} {
+	# bypass stupid check that assumes fprintf must be in libc.dylib
+	patchfiles-append   patch-hints_darwin.sh-usenm.diff
+    }
     # Prevent miniperl build from finding MacPorts' libstdcxx (#36438).
     patchfiles-append   patch-Makefile.SH.diff
     post-patch {

Attachments (1)

perl5x.patch (915 bytes) - added by Serge3leo (Serguei E. Leontiev) 6 years ago.

Download all attachments as: .zip

Change History (16)

Changed 6 years ago by Serge3leo (Serguei E. Leontiev)

Attachment: perl5x.patch added

comment:1 Changed 6 years ago by larryv (Lawrence Velázquez)

Owner: changed from macports-tickets@… to ricci@…

Thanks for the ticket. In the future, please Cc relevant port maintainers (port info --maintainers).

comment:2 Changed 6 years ago by whcordis@…

Cc: whcordis@… added

Cc Me!

comment:3 Changed 6 years ago by msy@…

Cc: msy@… added

Cc Me!

comment:4 Changed 6 years ago by sck@…

This patch did not work for me on Mavericks.

comment:5 Changed 6 years ago by sck@…

Cc: sck@… added

Cc Me!

comment:6 Changed 6 years ago by ghosthound

Owner: changed from ricci@… to macports-tickets@…

No longer maintainer

comment:7 Changed 6 years ago by ryandesign (Ryan Schmidt)

Has duplicate #41197.

comment:8 Changed 6 years ago by ryandesign (Ryan Schmidt)

Keywords: mavericks added
Owner: changed from macports-tickets@… to macports-tickets@…

comment:9 Changed 6 years ago by wwboy6@…

Cc: wwboy6@… added

Cc Me!

comment:10 Changed 6 years ago by Ionic (Mihai Moldovan)

The patch looks sane and works for me.

comment:11 Changed 6 years ago by Ionic (Mihai Moldovan)

Cc: ionic@… added

Cc Me!

comment:12 Changed 6 years ago by ryandesign (Ryan Schmidt)

Cc: chema.torrelles@… added

Has duplicate #41325.

comment:13 Changed 6 years ago by jeremyhu (Jeremy Huddleston Sequoia)

comment:14 Changed 6 years ago by jeremyhu (Jeremy Huddleston Sequoia)

Resolution: fixed
Status: newclosed

comment:15 Changed 6 years ago by ryandesign (Ryan Schmidt)

Has duplicate #41558.

Note: See TracTickets for help on using tickets.