Differential D885

Cleanup @dirrm* from lang/perl5.*
ClosedPublic
Actions

Authored by mat on Oct 2 2014, 9:10 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Oct 23, 1:18 AM

	Unknown Object (File)
	Sat, Oct 18, 6:58 PM

	Unknown Object (File)
	Thu, Oct 16, 3:32 AM

	Unknown Object (File)
	Mon, Oct 6, 5:26 AM

	Unknown Object (File)
	Mon, Oct 6, 3:55 AM

	Unknown Object (File)
	Mon, Oct 6, 12:43 AM

	Unknown Object (File)
	Sun, Oct 5, 10:22 PM

	Unknown Object (File)
	Sun, Oct 5, 9:43 PM

Subscribers

Details

Reviewers

Group Reviewers

Commits

rP369892: Cleanup @dirrm*.

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

mat updated this revision to Diff 1867.Oct 2 2014, 9:10 PM

mat retitled this revision from to Cleanup @dirrm* from lang/perl5.*.

mat updated this object.

mat edited the test plan for this revision. (Show Details)

mat added a reviewer: Perl.

Would love someone to test this, otherwise, I'll just

Macro shipit:

bapt added a subscriber: bapt.Oct 2 2014, 10:22 PM

bapt added inline comments.

lang/perl5.14/pkg-plist
3002 ↗	(On Diff #1867)	What is the point of this entry? I'm not sure about the below one either only @dir %%SITE_PERL%% seem interesting
3007 ↗	(On Diff #1867)	This would need an alternative as well but that is not the point, I'll bring an alternative soon :)
lang/perl5.20/pkg-plist
2261 ↗	(On Diff #1867)	@rmtry?
2262 ↗	(On Diff #1867)	god this is dirty, I'd be interested in what other os do here

mat added inline comments.Oct 3 2014, 11:45 AM

lang/perl5.14/pkg-plist
3002 ↗	(On Diff #1867)	The point is to have the empty directory exist. Perl modules expect it to be there.
lang/perl5.20/pkg-plist
2261 ↗	(On Diff #1867)	Ah, yes, forgot about that one :-)

use @rmtry sort better.

bapt accepted this revision.Oct 3 2014, 11:51 AM

bapt added a reviewer: bapt.

This revision is now accepted and ready to land.Oct 3 2014, 11:51 AM

mat added inline comments.Oct 3 2014, 12:24 PM

lang/perl5.14/pkg-plist
3007 ↗	(On Diff #1867)	About that, I've been pondering removing this and doing it in post-install, but I would need to generate that bit of plist, and I don't like doing that :-)

Closed by commit rP369892 (authored by @mat).

Revision Contents
Changeset List

Path

Size

head/

lang/

perl5.14/

366 lines

perl5.16/

359 lines

perl5.18/

360 lines

perl5.20/

332 lines

Diff 1871

head/lang/perl5.14/pkg-plist

Loading...

head/lang/perl5.16/pkg-plist

Loading...

head/lang/perl5.18/pkg-plist

Loading...

head/lang/perl5.20/pkg-plist

Loading...