Opened 5 years ago

Closed 4 years ago

Last modified 4 years ago

#44405 closed defect (fixed)

Ports should require perl5.16 not perl5.12

Reported by: mf2k (Frank Schima) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: afb@…, alexandernx@…, blair (Blair Zajac), cbellot@…, cfaerber@…, ctreleaven (Craig Treleaven), daniel@…, dennisvd@…, danielluke (Daniel J. Luke), drkp (Dan Ports), gjasny@…, jjstickel@…, jmpp@…, jmroot (Joshua Root), kmayer (Ken Mayer), landonf@…, lockhart (Thomas Lockhart), marioscheel@…, mww@…, ooburai@…, pixilla (Bradley Giesbrecht), rod@…, zapman@…, mojca (Mojca Miklavec), dbevans (David B. Evans), cooljeanius (Eric Gallager), petrrr
Port: ackmate amavisd-new archway argus-monitor backuppc bugzilla cssh cws2fws-pl cyrus-imapd docbook-utils dvdrip extractopinion frozenbubble2 ftpsync fwknop gdmap ghsum globus-core gpredict grid-packaging-tools GTorrentViewer hugin-app imageindex imapsync liboping-devel libqalculate mailqfmt makepasswd mediaserv monarch mtn-browse munin muttprint myrepos mysql-zrm mythtv-core.25 mythtv-core.26 mythtv-core.27 mythweb.25 mythweb.26 mythweb.27 nagios-plugins nedi nfsen nrg pearl perlprimer postgrey PsyncX rancid rsnapshot sendpage smokeping snownews spidereyeballs sqlgrey stardict sympa t-prot thieriot tinyca2 tsung vantages vcheck wml xfce-utils xqf xserve-raid-tools

Description (last modified by dbevans (David B. Evans))

All ports that require perl5.12 or p5.12-* ports should be switched to use perl 5.16 versions since that is the Macports default currently.

The list of ports is culled from the following command:

$ port echo depends:perl5.12 or depends:p5.12-*

Ports which require maintainer approval

DONE!

Ports with openmaintainer

DONE!

Ports with nomaintainer

DONE!

Fixed

  • ackmate (nomaintainer): r122695
  • amavisd-new (pixilla,openmaintainer): r123823
  • archway (jmr,openmaintainer): r123804
  • argus-monitor (nomaintainer): r128944
  • backuppc (nomaintainer): r125013
  • bugzilla (nomaintainer): r122731
  • cssh (nomaintainer): r122698
  • cws2fws-pl (nomaintainer): r122578
  • cyrus-imapd (nomaintainer): r129759
  • docbook-utils (gmail.com:lockhart1976,openmaintainer): r122724
  • dvdrip (web.de:marioscheel): r129667, maintainer timeout
  • extractopinion (hum,openmaintainer): r122599
  • frozenbubble2 (nomaintainer): r123265
  • ftpsync (mww,openmaintainer): r123806
  • fwknop (blair): r129677, maintainer timeout
  • gdmap (eborisch,openmaintainer): r122476
  • ghsum (pu-gh.com:rod): r129680, maintainer timeout.
  • globus-core (nikhef.nl:dennisvd,openmaintainer): r123534
  • gnucash (dports,openmaintainer): r123211
  • gpredict (gmail.com:zapman): r129733, maintainer timeout
  • grid-packaging-tools (nikhef.nl:dennisvd,openmaintainer): r123534
  • GTorrentViewer (jmpp,openmaintainer): r123808
  • imapsync (mww,openmaintainer): r123810
  • hugin-app (nomaintainer): r122466
  • imageindex (nomaintainer): r122729
  • liboping-devel (danner.de:daniel,openmaintainer): r123811
  • libqalculate (vcn.com:jjstickel): r129734, maintiner timeout.
  • mailqfmt (sky.fr:cbellot): r129735, maintainer timeout.
  • makepasswd (gmail.com:ooburai): r129670, maintainer timeout.
  • mediaserv (ecronin,openmaintainer): r122616
  • monarch (nomaintainer): r129017
  • mtn-browse (nomaintainer): r123262
  • munin (nomaintainer): r122727
  • muttprint (nomaintainer): r123266
  • myrepos (googlemail.com:gjasny,openmaintainer): r123814
  • mysql-zrm (bitwrangler.com:kmayer): r129736, maintainer timeout.
  • mythtv-core.25 (ctreleaven,openmaintainer): r122736
  • mythweb.25 (ctreleaven,openmaintainer): r122709
  • mythtv-core.26 (ctreleaven,openmaintainer):
  • mythtv-core.27 (ctreleaven,openmaintainer):
  • mythweb.26 (ctreleaven,openmaintainer):
  • mythweb.27 (ctreleaven,openmaintainer):
  • nagios-plugins (markd,openmaintainer): r123816
  • nedi (nomaintainer): r128981
  • nfsen (nomaintainer): r129021
  • nrg (nomaintainer): r128987
  • pearl (pu-gh.com:rod): r129737, maintainer timeout.
  • perlprimer (nomaintainer): r123268
  • postgrey (sky.fr:cbellot): r129738, maintainer timeout.
  • PsyncX (ryandesign,openmaintainer): r122473
  • rancid (markd,openmaintainer): r123818
  • rsnapshot (mww): r129739, maintainer timeout.
  • sendpage (nomaintainer): r129025
  • smokeping (markd,openmaintainer): r123820
  • snownews (nomaintainer): r123270
  • spidereyeballs (nomaintainer): r123203
  • sqlgrey (pixilla): r126932
  • stardict (gmail.com:alexandernx): r129740, maintainer timeout.
  • sympa (nomaintainer): r129078
  • t-prot (nomaintainer): r123272
  • thieriot (pu-gh.com:rod): r123340 and r123341
  • tinyca2 (landonf): (#44491, r124925)
  • tsung (nomaintainer): r123277
  • vantages (nomaintainer): r123012
  • vcheck (afb,openmaintainer): r122747
  • wml (cfaerber.name:cfaerber,openmaintainer): r122694 (see #44454)
  • xfce-utils (afb): r122878
  • xqf (nomaintainer): r123338
  • xserve-raid-tools (nomaintainer): r128986

Attachments (6)

wml-patch-perl5.16.diff (1.6 KB) - added by cfaerber@… 5 years ago.
Patch for www/wml
frozenbubble2-patch-perl5.16.diff (1.2 KB) - added by mojca (Mojca Miklavec) 5 years ago.
Replace perl5.12 with perl5.16 in frozenbubble2
Portfile-thieriot.diff (1.2 KB) - added by rod@… 5 years ago.
Patch for Thieriot
Portfile-ghsum.diff (1.9 KB) - added by rod@… 5 years ago.
Patch for ghsum
Portfile-ghsum.2.diff (2.1 KB) - added by rod@… 5 years ago.
Portfile-pearl.diff (1.5 KB) - added by rod@… 5 years ago.
Patch for Pearl

Download all attachments as: .zip

Change History (95)

comment:1 Changed 5 years ago by mojca (Mojca Miklavec)

Cc: mojca@… added

Cc Me!

comment:2 Changed 5 years ago by mojca (Mojca Miklavec)

I already opened #43193 a while ago, but didn't add the list of ports.

comment:3 Changed 5 years ago by danielluke (Daniel J. Luke)

Port: subversion-perlbindings-5.12 removed

comment:4 Changed 5 years ago by danielluke (Daniel J. Luke)

(removed subversion-perlbindings-5.12)

comment:5 Changed 5 years ago by dbevans (David B. Evans)

Cc: devans@… added

Cc Me!

comment:6 Changed 5 years ago by mojca (Mojca Miklavec)

hugin-app fixed in r122466.

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

Cc: ryandesign@… removed

PsyncX fixed in r122473.

comment:8 Changed 5 years ago by eborisch (Eric A. Borisch)

Cc: eborisch@… removed

gdmap fixed in r122476.

comment:9 Changed 5 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:10 Changed 5 years ago by mf2k (Frank Schima)

extractopinion fixed in r122599.

comment:11 Changed 5 years ago by mf2k (Frank Schima)

cws2fws-pl fixed in r122578.

comment:12 Changed 5 years ago by ecronin (Eric Cronin)

Cc: ecronin@… removed

mediaserv in r122616

comment:13 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

comment:14 Changed 5 years ago by mojca (Mojca Miklavec)

Cc: tlockhart1976@… added; lockhart@… removed

comment:15 Changed 5 years ago by mojca (Mojca Miklavec)

Description: modified (diff)

Changed 5 years ago by cfaerber@…

Attachment: wml-patch-perl5.16.diff added

Patch for www/wml

comment:16 Changed 5 years ago by cfaerber@…

I've uploaded a patch for www/wml. Can someone with commit rights please add it to the repository? Thanks.

comment:17 Changed 5 years ago by mojca (Mojca Miklavec)

www/wml committed in r122694. Thanks.

Next time please try to change

--- a/dports/www/wml/Portfile
+++ b/dports/www/wml/Portfile

into something like:

--- Portfile.orig
+++ Portfile

if possible. (I don't know if there's an automated way to do this, I usually do it manually.) I know that there's a -p flag for patching that can strip off the path, but it has been a convention in MacPorts to use -p0.

comment:18 Changed 5 years ago by mojca (Mojca Miklavec)

Description: modified (diff)

comment:19 Changed 5 years ago by mojca (Mojca Miklavec)

Description: modified (diff)

ackmate fixed in r122695. However the port seems to install files to the wrong place.

comment:20 Changed 5 years ago by mojca (Mojca Miklavec)

Description: modified (diff)

cssh upgraded and fixed in r122698

comment:21 Changed 5 years ago by mojca (Mojca Miklavec)

Description: modified (diff)

comment:22 Changed 5 years ago by mojca (Mojca Miklavec)

Description: modified (diff)

I'm attaching a patch for frozenbubble2, but I'm unable to try it out until libsdl_pango gets fixed (#44453).

Changed 5 years ago by mojca (Mojca Miklavec)

Replace perl5.12 with perl5.16 in frozenbubble2

comment:23 Changed 5 years ago by mojca (Mojca Miklavec)

Description: modified (diff)

docbook-utils fixed in r122724

comment:24 Changed 5 years ago by dbevans (David B. Evans)

Description: modified (diff)

munin fixed in r122727

comment:25 Changed 5 years ago by dbevans (David B. Evans)

Description: modified (diff)

imageindex fixed in r122729.

comment:26 Changed 5 years ago by dbevans (David B. Evans)

Description: modified (diff)

bugzilla fixed in r122731 including update to latest stable version.

comment:27 Changed 5 years ago by ctreleaven (Craig Treleaven)

mythweb.25 fixed in r122709, mythtv-core.25 fixed in r122736. Both with other minor updates.

comment:28 Changed 5 years ago by dbevans (David B. Evans)

Description: modified (diff)

comment:29 Changed 5 years ago by dbevans (David B. Evans)

Description: modified (diff)

tinyca2 patch available in #44491.

comment:30 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

vcheck fixed in r122747. xfce-utils fixed in r122878.

comment:31 Changed 5 years ago by dbevans (David B. Evans)

Description: modified (diff)

vantages fixed in r123012.

comment:32 Changed 5 years ago by petrrr

Cc: petr@… added

Cc Me!

comment:33 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

spidereyeballs in r123203.

comment:34 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

mtn-browse fixed in r123262.

comment:35 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

frozenbubble2 fixed in r123265.

comment:36 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

muttprint fixed in r123266.

comment:37 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

perlprimer fixed in r123268.

comment:38 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

snownews fixed in r123270

comment:39 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

t-prot fixed in r123272.

comment:40 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

tsung fixed in r123277.

comment:41 Changed 5 years ago by drkp (Dan Ports)

Description: modified (diff)
Port: gnucash-devel removed

fixed gnucash-devel in r123211 by merging various changes from gnucash

comment:42 Changed 5 years ago by humem (humem)

Cc: hum@… removed

Changed 5 years ago by rod@…

Attachment: Portfile-thieriot.diff added

Patch for Thieriot

comment:43 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

xqf fixed in r123338.

comment:44 Changed 5 years ago by mf2k (Frank Schima)

@rod: Your patchfile for ghsum changes the version from 0.1.0 to 0.0.2. If you are really intending to downgrade the version, then the epoch needs to increased so users will see the change. Please confirm if that is your intention.

Changed 5 years ago by rod@…

Attachment: Portfile-ghsum.diff added

Patch for ghsum

comment:45 Changed 5 years ago by rod@…

Doh! Corrected.

comment:46 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

@rod: The checksums need to be updated for the new versions in your patches. I have updated thieriot in r123340 and r123341.

comment:47 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

cyrus-imapd fixed in r123368.

comment:48 Changed 5 years ago by petrrr

grid-packaging-tools and globus-core fixed in r123534.

I do not like the way perl is handled by these ports too much. But I have no idea if it could be improved in this particular case neither.

Changed 5 years ago by rod@…

Attachment: Portfile-ghsum.2.diff added

Changed 5 years ago by rod@…

Attachment: Portfile-pearl.diff added

Patch for Pearl

comment:49 Changed 5 years ago by mf2k (Frank Schima)

@rod: For some reason with the new patches I cannot download either ghsum or pearl. I see an error like this:

Error: Failed to fetch pearl: The requested URL returned error: 404 Not Found
Error: See /opt/local/var/macports/logs/_opt_mports_trunk_dports_devel_pearl/pearl/main.log for details.

comment:50 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

comment:51 Changed 5 years ago by mojca (Mojca Miklavec)

I believe that ports which weren't fixed count as "maintainer timeout" already? Is it OK to fix a port straight to 5.20 (assuming that all the dependencies are present and that it works of course)?

comment:52 in reply to:  51 Changed 5 years ago by dbevans (David B. Evans)

Replying to mojca@…:

I believe that ports which weren't fixed count as "maintainer timeout" already? Is it OK to fix a port straight to 5.20 (assuming that all the dependencies are present and that it works of course)?

I think the current recommended policy is to depend on the current default version for port perl5 (currently perl5.16). This minimizes the number of different perl versions that need to be simultaneously installed for the casual MacPorts user.

comment:53 Changed 5 years ago by mf2k (Frank Schima)

I totally agree with Dave. We need to stick to using perl 5.16 until such time that Macports switches to perl 5.20.

comment:54 Changed 5 years ago by mf2k (Frank Schima)

And yes, I believe we are long past maintainer timeout and can switch all the ports over.

comment:55 Changed 5 years ago by mojca (Mojca Miklavec)

archway fixed in r123804 (r123803).

comment:56 Changed 5 years ago by mojca (Mojca Miklavec)

Last edited 5 years ago by mojca (Mojca Miklavec) (previous) (diff)

comment:57 Changed 5 years ago by mojca (Mojca Miklavec)

Description: modified (diff)

comment:58 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

Update ctreleaven ports now that he has a Macports handle.

comment:59 Changed 5 years ago by mf2k (Frank Schima)

Cc: ctreleaven@… added; ctreleaven@… removed

comment:60 Changed 5 years ago by ctreleaven (Craig Treleaven)

Thanks, I'll modify the myth* ports soon.

comment:61 Changed 5 years ago by ctreleaven (Craig Treleaven)

Description: modified (diff)

comment:62 Changed 5 years ago by ctreleaven (Craig Treleaven)

mythweb.26/27 and mythtv-core.26/27 fixed in r124024, r124025, r124026, r124027

comment:63 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

comment:64 Changed 5 years ago by dbevans (David B. Evans)

Description: modified (diff)

comment:65 Changed 5 years ago by dbevans (David B. Evans)

  • tinyca fixed in r124925, maintainer timeout.

comment:66 Changed 5 years ago by mf2k (Frank Schima)

Cc: jameskyle@… removed
Description: modified (diff)

jameskyle has retired. See #44641.

comment:67 Changed 5 years ago by mf2k (Frank Schima)

Description: modified (diff)

backuppc fixed in r125013.

comment:68 Changed 4 years ago by mf2k (Frank Schima)

Cc: markd@… removed
Description: modified (diff)

comment:69 Changed 4 years ago by mf2k (Frank Schima)

Description: modified (diff)

comment:70 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

argus-monitor fixed in r128944.

comment:71 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

nedi fixed in r128981.

comment:72 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

xserve-raid-tools fixed in r128986.

comment:73 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

nrg fixed in r128987.

comment:74 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

comment:75 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

monarch fixed in r129017.

comment:76 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

nfsen fixed in r129021.

comment:77 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

sendpage fixed in r129025
sqlgrey fixed in r126932 (by pixilla)
sympa fixed in r129078

All nomaintainer ports now fixed.

comment:78 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

comment:79 Changed 4 years ago by petrrr

The remaining ports could probably be considered abandoned by now. Should we start compiling "Port Abandoned" tickets?

comment:80 Changed 4 years ago by dbevans (David B. Evans)

As mentioned some time ago in comment:54, any remaining maintained ports that haven't been fixed by their maintainer can be considered fair game to fix by anyone desiring to do so under the maintainer timeout policy. Folks without commit access can attach a patch for review and those with access can just go ahead, commit a fix and add a comment to that effect to this ticket.

Of course, if the maintainers of these ports would like to do the work or if there is a reason why there is a problem with one of their ports, this is a good time to speak up!

At this point, there are only about a dozen left so it won't take much time to clear these if everyone chips in. Once this is done, the way will be paved to go ahead and remove support for perl5.14 and below as jmr has begun to do with p5.8-* in r128785.

comment:81 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

dvdrip fixed in r129667, maintainer timeout.

comment:82 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

makepasswd fixed in r129670, maintainer timeout.

comment:83 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

fwknop fixed in r129677, maintainer timeout.
ghsum fixed in r129680, maintainer timeout.

comment:84 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)

gpredict fixed in r129733
libqalculate fixed in r129734
mailqfmt fixed in r129735
mysql-zrm fixed in r129736
pearl fixed in r129737
postgrey fixed in r129738
rsnapshot fixed in r129739
stardict fixed in r129740

Only cyrus-imapd remains.

comment:85 Changed 4 years ago by dbevans (David B. Evans)

Description: modified (diff)
Resolution: fixed
Status: newclosed

cyrus-imapd fixed in r129759.

comment:86 Changed 4 years ago by mojca (Mojca Miklavec)

Is there any volunteer to open the same kind of ticket as this one for a switch to Perl 5.20? The ticket #46005 is tracking the progress, but I believe that at this point the number of ports that don't support 5.20 yet is small enough that it shouldn't cause any issues any longer.

(Also, last time it took 5 months to close this ticket. If it takes so long again, the missing ports will be either all supporting 5.22 by then or still broken.)

comment:87 in reply to:  86 Changed 4 years ago by dbevans (David B. Evans)

Replying to mojca@…:

Is there any volunteer to open the same kind of ticket as this one for a switch to Perl 5.20? The ticket #46005 is tracking the progress, but I believe that at this point the number of ports that don't support 5.20 yet is small enough that it shouldn't cause any issues any longer.

(Also, last time it took 5 months to close this ticket. If it takes so long again, the missing ports will be either all supporting 5.22 by then or still broken.)

Mojca, since this ticket is closed and your query is not a ticket issue, it would be best to take this discussion to the devel list.

comment:88 Changed 4 years ago by mojca (Mojca Miklavec)

I'm aware of both facts, but I was hoping that one of the people subscribed to this ticket would be willing to step up and collect the ports that need a change. If nobody volunteers, I'll ask on the mailing list.

(And I believe that everyone else subscribed to the ticket should be free to start the transition to 5.20 even if the new ticket isn't opened yet.)

comment:89 in reply to:  88 Changed 4 years ago by dbevans (David B. Evans)

Replying to mojca@…:

I'm aware of both facts, but I was hoping that one of the people subscribed to this ticket would be willing to step up and collect the ports that need a change. If nobody volunteers, I'll ask on the mailing list.

(And I believe that everyone else subscribed to the ticket should be free to start the transition to 5.20 even if the new ticket isn't opened yet.)

If by 'the transition to 5.20' you mean changing the default perl version to 5.20, then it would be better IMO to develop some concensus on the devel list first. It involves more that just changing the dependencies of these ports.

Please open a discussion there and CC whoever you think might be interested.

Note: See TracTickets for help on using tickets.