Opened 12 years ago

Closed 12 years ago

Last modified 10 years ago

#34784 closed enhancement (fixed)

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

Reported by: ryandesign (Ryan Carsten 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 12 years ago.
script to add the perl5.branches lines to all p5-* ports
perl5branches.diff (372.6 KB) - added by BjarneDMat 12 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 12 years ago by ryandesign (Ryan Carsten 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 12 years ago by BjarneDMat

Attachment: perl5branches.bash added

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

Changed 12 years ago by BjarneDMat

Attachment: perl5branches.diff added

diff adding perl5.branches to all p5-* ports

comment:2 Changed 12 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 12 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: newclosed

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

Cc: egall@… added

Cc Me!

Note: See TracTickets for help on using tickets.