Opened 10 years ago
Closed 10 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 10 years ago by dbevans (David B. Evans)
Cc: | mojca@… added |
---|
comment:2 Changed 10 years ago by mf2k (Frank Schima)
Type: | request → enhancement |
---|
comment:3 follow-up: 4 Changed 10 years ago by danielluke (Daniel J. Luke)
comment:4 follow-up: 5 Changed 10 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 follow-up: 6 Changed 10 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 follow-up: 7 Changed 10 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 Changed 10 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 follow-up: 9 Changed 10 years ago by mojca (Mojca Miklavec)
What about something like the following (which can probably be simplified further)?
-
Portfile
18 18 19 19 homepage http://subversion.apache.org/ 20 20 21 set perl5.branches {5. 8 5.10 5.12 5.14 5.16 5.18 5.20}21 set perl5.branches {5.16 5.18 5.20} 22 22 foreach branch ${perl5.branches} { 23 23 set p_idx [lsearch ${perl5.branches} ${branch}] 24 24 set other_perls [lreplace ${perl5.branches} $p_idx $p_idx] … … 30 30 } 31 31 } 32 32 } 33 set perl5.old_branches {5.8 5.10 5.12 5.14} 34 foreach branch ${perl5.old_branches} { 35 subport ${name}-${branch} { 36 replaced_by ${name}-5.16 37 PortGroup obsolete 1.0 38 revision 1 39 } 40 } 33 41 34 42 if {${subport} eq ${name}} { 35 43 … … 49 57 livecheck.url http://svn.apache.org/repos/asf/subversion/tags/ 50 58 livecheck.regex "(\\d+\\.\\d+\\.\\d+)/" 51 59 60 } elseif {${subport} eq "${name}-5.8" || ${subport} eq "${name}-5.10" || ${subport} eq "${name}-5.12" || ${subport} eq "${name}-5.14"} { 52 61 } else { 53 62 54 63 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 follow-up: 11 Changed 10 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 10 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 follow-up: 12 Changed 10 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 follow-up: 13 Changed 10 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
inp5-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 Changed 10 years ago by larryv (Lawrence Velázquez)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Replying to dluke@…:
Replying to mojca@…:
Can you please explain what you meant with that? I don't understand how exactly
subversion-perlbinding
inp5-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.
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?