git-core - make the Portfile independent of the installed version of Perl

[BjarneDMat]

I had upgraded completely to Perl 5.14 & removed every trace of Perl 5.12 from my computers

However, Perl 5.12 is hardcoded into the Portfile for git-core.

The attached patch-Portfile makes git-core independent of which version of Perl is installed.

[BjarneDMat]

patch to make git-core independen of installed version of Perl

[ryandesign (Ryan Carsten Schmidt)]

There are tons of ports that depend specifically on perl 5.12 modules.

The changes proposed in this patch look unusual to me; I'm not aware of us doing anything similar in any other ports.

[BjarneDMat]

p5-dbd-mysql is already using this system, which is where I originally got the idea to this from ;-)

I've proposed this change for sql-grey too in #34357

In my opinion - unless it's explicitly required of the port - then ports ought to be independent of which version of Perl is the major one.

[cmconnelly (Claire Connelly)]

Replying to macintosh@…:

In my opinion - unless it's explicitly required of the port - then ports ought to be independent of which version of Perl is the major one.

Completely agree. I don't need multiple versions of Perl lying around, especially if there's no actual dependency on a specific version of Perl. Same goes for Python, Ruby, and other languages.

[nonstop.server@…]

Cc Me!

[jmroot (Joshua Root)]

This doesn't work right; you need to set a variant to distinguish which version of perl is used.

[BjarneDMat]

Replying to jmr@…:

This doesn't work right; you need to set a variant to distinguish which version of perl is used.

I'm sorry, but your statement is not correct.
It works perfectly in all instances I've used it.

Having said that, this ticket has to bee marked 'wont-fix' or 'duplicate' as it's superceeded by #34617

[ci42]

Replying to macintosh@…:

Having said that, this ticket has to bee marked 'wont-fix' or 'duplicate' as it's superceeded by #34617

I don't see that your proposed patch has been accepted.

However, If I have some spare time I will provide variants for perl5.14/perl5.16. Until I'll have implemented these variants this ticket remains open.

[jmroot (Joshua Root)]

Replying to macintosh@…:

I'm sorry, but your statement is not correct.
It works perfectly in all instances I've used it.

Then your testing has been insufficient. Example: Build an archive of git-core on a machine with only perl5.12 installed, then install it on a machine with only perl5.14 installed. None of the perl-using bits will work.

This can happen because "git-core-1.7.11.5_0.darwin_11.x86_64.tbz2" is produced in both cases. If there was a variant controlling which perl to use, it would be e.g. "git-core-1.7.11.5_0+perl512.darwin_11.x86_64.tbz2" vs "git-core-1.7.11.5_0+perl514.darwin_11.x86_64.tbz2", and only the matching archive would be installed.

[cooljeanius (Eric Gallager)]

Cc Me!

[ddissett (daniel dissett)]

Portfile diffs add perl_12|14|16 variants

[ddissett (daniel dissett)]

Added a patch to add perl_12, perl_14, and perl_16 variants to the git-core Portfile. Most of the Portfile changes were cut and pasted from the intltool Portfile.

Test builds worked for me with several variant combinations.

Adding it here if its useful to anyone since the last discussion on this ticket was 14 months ago and I'm apparently on a crusade to keep all but one perl5_16 out my new macports install.

[ci42]

Variants for perl 5.12 (default), 5.14 and 5.16 added in r114984.