Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#34784 closed enhancement (fixed)

Add perl5.branches to all p5 ports (adding p5.16 for perl5.16)

Reported by: ryandesign (Ryan Schmidt) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.1.1
Keywords: Cc: pixilla (Bradley Giesbrecht), BjarneDMat, cooljeanius (Eric Gallager)
Port:

Description

In r93584 the perl5 portgroup was updated to add "5.16" to perl5.branches, but this is not the correct way to do this. Instead, every p5 port needs the perl5.branches line added to it, indicating which branches of perl that module is compatible with (except the ports that already have that line; they need to have that line updated to include "5.16" if applicable to that port).

Attachments (2)

perl5branches.bash (726 bytes) - added by BjarneDMat 7 years ago.
script to add the perl5.branches lines to all p5-* ports
perl5branches.diff (372.6 KB) - added by BjarneDMat 7 years ago.
diff adding perl5.branches to all p5-* ports

Download all attachments as: .zip

Change History (6)

comment:1 in reply to:  description Changed 7 years ago by ryandesign (Ryan Schmidt)

Replying to ryandesign@…:

In r93584 the perl5 portgroup was updated to add "5.16" to perl5.branches

I reverted this in r94024.

but this is not the correct way to do this.

The reason this does not work is that the p5.16 subports don't get added to the portindex until the ports are reindexed. This only happens when a port is updated. So ever since r93584 this means when a port is updated, that has dependencies that haven't been updated since that change, the buildbot will fail to build the p5.16 subport, because its dependencies don't seem to have p5.16 subports yet.

Instead, every p5 port needs the perl5.branches line added to it

Once this is done, the default for perl5.branches needs to be removed from the perl5 portgroup.

Changed 7 years ago by BjarneDMat

Attachment: perl5branches.bash added

script to add the perl5.branches lines to all p5-* ports

Changed 7 years ago by BjarneDMat

Attachment: perl5branches.diff added

diff adding perl5.branches to all p5-* ports

comment:2 Changed 7 years ago by BjarneDMat

notes :

my perl5branches script can't at present see if a p5-* port has been replaced.

running the script followed by port sync will thus generate an error for all of those ports, and the perl5.branches line will have to be manually edited out againg

comment:3 Changed 7 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: newclosed

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

Cc: egall@… added

Cc Me!

Note: See TracTickets for help on using tickets.