Opened 9 years ago

Closed 9 years ago

#46379 closed enhancement (fixed)

subversion-perlbindings: remove support for obsolete perl versions

Reported by: dbevans (David B. Evans) Owned by: danielluke (Daniel J. Luke)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: blair (Blair Zajac), mojca (Mojca Miklavec)
Port: subversion-perlbindings

Description

The following are the only remaining dependents of obsolete perl5.8 through perl5.14

subversion-perlbindings-5.8
subversion-perlbindings-5.10
subversion-perlbindings-5.12
subversion-perlbindings-5.14

Dependents p5-svn-mirror and p5-svn-simple now only support perl5.16 through perl5.20.

Change History (13)

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

Cc: mojca@… added

comment:2 Changed 9 years ago by mf2k (Frank Schima)

Type: requestenhancement

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

Do we have a recipe for this? are we just removing the older perls from perl5.branches, or are we trying to provide some sort of transition for people who may have an older p5 port installed?

comment:4 in reply to:  3 ; Changed 9 years ago by larryv (Lawrence Velázquez)

We’re generally using replaced_by to “upgrade” people using older Perl ports to the 5.16 versions (see #46322).

comment:5 in reply to:  4 ; Changed 9 years ago by danielluke (Daniel J. Luke)

Replying to larryv@…:

We’re generally using replaced_by to “upgrade” people using older Perl ports to the 5.16 versions (see #46322).

Ok, so I need to add subversion-perlbindings to the obsolete p5-graveyard port and then remove the older perls from p5.branches in the subversion-perlbindings port itself?

comment:6 in reply to:  5 ; Changed 9 years ago by danielluke (Daniel J. Luke)

Replying to dluke@…:

Replying to larryv@…:

We’re generally using replaced_by to “upgrade” people using older Perl ports to the 5.16 versions (see #46322).

Ok, so I need to add subversion-perlbindings to the obsolete p5-graveyard port and then remove the older perls from p5.branches in the subversion-perlbindings port itself?

actually, will this just not work because p5-graveyard expects the subports to be p5.X and the subversion-perlbindings port don't match that pattern?

comment:7 in reply to:  6 Changed 9 years ago by larryv (Lawrence Velázquez)

Replying to dluke@…:

actually, will this just not work because p5-graveyard expects the subports to be ^p5.X and the subversion-perlbindings port don't match that pattern?

It won’t work in the usual way, but I can shoehorn it in. You can go ahead and remove the subports from subversion-perlbindings, and I’ll add them to the graveyard afterwards.

comment:8 Changed 9 years ago by mojca (Mojca Miklavec)

What about something like the following (which can probably be simplified further)?

  • Portfile

     
    1818
    1919homepage            http://subversion.apache.org/
    2020
    21 set perl5.branches {5.8 5.10 5.12 5.14 5.16 5.18 5.20}
     21set perl5.branches {5.16 5.18 5.20}
    2222foreach branch ${perl5.branches} {
    2323    set p_idx [lsearch ${perl5.branches} ${branch}]
    2424    set other_perls [lreplace ${perl5.branches} $p_idx $p_idx]
     
    3030        }
    3131    }
    3232}
     33set perl5.old_branches {5.8 5.10 5.12 5.14}
     34foreach branch ${perl5.old_branches} {
     35    subport ${name}-${branch} {
     36        replaced_by ${name}-5.16
     37        PortGroup   obsolete 1.0
     38        revision    1
     39    }
     40}
    3341
    3442if {${subport} eq ${name}} {
    3543
     
    4957    livecheck.url   http://svn.apache.org/repos/asf/subversion/tags/
    5058    livecheck.regex "(\\d+\\.\\d+\\.\\d+)/"
    5159
     60} elseif {${subport} eq "${name}-5.8" || ${subport} eq "${name}-5.10" || ${subport} eq "${name}-5.12" || ${subport} eq "${name}-5.14"} {
    5261} else {
    5362
    5463    master_sites    apache:subversion

I don't see the need to try to hack the p5-graveyard port any further. It does its job to handle thousands of standard ports, but I don't see why this simple exception couldn't be handled inside the subversion-perlbindings port itself.

comment:9 in reply to:  8 ; Changed 9 years ago by danielluke (Daniel J. Luke)

Replying to mojca@…:

What about something like the following (which can probably be simplified further)?

besides the ugly elsif that's there, presumably putting it in p5-graveyard makes things better when we finally make the macports default perl the current one?

I don't really care either way.

comment:10 Changed 9 years ago by danielluke (Daniel J. Luke)

Removed older perls in r131061, I can add them back with replaced_by foo if we decide that that's what's best (or larryv can add them to p5-graveyard).

comment:11 in reply to:  9 ; Changed 9 years ago by mojca (Mojca Miklavec)

Replying to dluke@…:

Replying to mojca@…:

What about something like the following (which can probably be simplified further)?

besides the ugly elsif that's there,

Yes, I agree that the "elsif" is a tiny bit ugly, but I was hoping for a hacker to come up with a nicer solution. I was merely pointing to the direction that I would pick.

presumably putting it in p5-graveyard makes things better when we finally make the macports default perl the current one?

Can you please explain what you meant with that? I don't understand how exactly subversion-perlbinding in p5-graveyard would make anything better, cleaner or easier in the future. And what did you mean with "make the macports default perl the current one"? The switch to 5.20?

comment:12 in reply to:  11 ; Changed 9 years ago by danielluke (Daniel J. Luke)

Replying to mojca@…:

Can you please explain what you meant with that? I don't understand how exactly subversion-perlbinding in p5-graveyard would make anything better, cleaner or easier in the future.

fewer places to deprecate future perl versions

And what did you mean with "make the macports default perl the current one"? The switch to 5.20?

yes (although it might not be 5.20 by the time we get around to it). Perhaps we'll get read of the p5.xx foo at that point and just have p5- ports that work with the current perl by then, though (one can hope).

comment:13 in reply to:  12 Changed 9 years ago by larryv (Lawrence Velázquez)

Resolution: fixed
Status: newclosed

Replying to dluke@…:

Replying to mojca@…:

Can you please explain what you meant with that? I don't understand how exactly subversion-perlbinding in p5-graveyard would make anything better, cleaner or easier in the future.

fewer places to deprecate future perl versions

It also prevents future changes to subversion-perlbindings from causing spurious Buildbot failures. This is the actual purpose of the graveyard metaports.

Anyway, r131075.

Note: See TracTickets for help on using tickets.