Page MenuHomeFreeBSD

New Port: devel/mage
ClosedPublic

Authored by seanc on Apr 22 2018, 10:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 5, 12:58 AM
Unknown Object (File)
Fri, May 3, 3:45 AM
Unknown Object (File)
Fri, Apr 19, 3:13 PM
Unknown Object (File)
Mar 17 2024, 9:18 PM
Unknown Object (File)
Mar 9 2024, 3:14 AM
Unknown Object (File)
Jan 21 2024, 12:44 AM
Unknown Object (File)
Jan 18 2024, 2:24 PM
Unknown Object (File)
Dec 20 2023, 2:42 AM
Subscribers
None

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

This revision is now accepted and ready to land.Apr 23 2018, 12:01 AM
This revision was automatically updated to reflect the committed changes.
head/devel/mage/Makefile
5 ↗(On Diff #41739)

2.1 and remove GH_TAGNAME.

24–30 ↗(On Diff #41739)

Why two separate targets ?

Also, never, ever, ever, overwrite do-patch it is the target used by the framework to apply patches.

It would be better to do this the other way, remove your do-patch target, have the framework apply the patch with the %% placeholders, and replace them in post-patch.

seanc marked an inline comment as done.

Address feedback from @mat and use post-patch instead of pre-patch and do-patch.

seanc added inline comments.
head/devel/mage/Makefile
5 ↗(On Diff #41739)

When I do that, I'm unable to fetch the src tarball. Admittedly I haven't looked into why, but those two seem to be required atm:

% svn diff .
Index: Makefile
===================================================================
--- Makefile	(revision 468072)
+++ Makefile	(working copy)
@@ -2,7 +2,7 @@
 
 PORTNAME=	mage
 DISTVERSIONPREFIX=	v
-DISTVERSION=	2.1.0
+DISTVERSION=	2.1
 CATEGORIES=	devel
 
 MAINTAINER=	seanc@FreeBSD.org
@@ -16,7 +16,6 @@
 USE_GITHUB=	yes
 GH_ACCOUNT=	magefile
 GH_SUBDIR=	src/github.com/${GH_ACCOUNT}/${PORTNAME}
-GH_TAGNAME=	2.1
 GH_TAG_COMMIT=	771ebed
 
 PLIST_FILES=	bin/mage
% make makesum
=> magefile-mage-v2.1_GH0.tar.gz doesn't seem to exist in /usr/home/seanc/src/FreeBSD/ports/distfiles/.
=> Attempting to fetch https://codeload.github.com/magefile/mage/tar.gz/v2.1?dummy=/magefile-mage-v2.1_GH0.tar.gz
fetch: https://codeload.github.com/magefile/mage/tar.gz/v2.1?dummy=/magefile-mage-v2.1_GH0.tar.gz: Not Found
=> Attempting to fetch http://distcache.FreeBSD.org/ports-distfiles/magefile-mage-v2.1_GH0.tar.gz
fetch: http://distcache.FreeBSD.org/ports-distfiles/magefile-mage-v2.1_GH0.tar.gz: Not Found
=> Couldn't fetch it - please try to retrieve this
=> port manually into /usr/home/seanc/src/FreeBSD/ports/distfiles/ and try again.
*** Error code 1

Stop.
make: stopped in /usr/home/seanc/src/FreeBSD/ports/devel/mage
Exit 1

If I leave in GH_TAGNAME then the port can fetch correctly. This requires more investigation into the GH_* variables.

24–30 ↗(On Diff #41739)

The pre-patch step is prepping the supplied patch. In the do-patch step, I'm no longer preparing the patch to be applied, instead I'm now applying it. However I am explicitly not applying the patch found in ${FILESDIR} because I want to apply a patch using the processed patch. The reverse, however, is nicer. Updated.

This revision is now accepted and ready to land.May 9 2018, 12:15 PM
This revision was automatically updated to reflect the committed changes.