Another port predating GH_DISTFILE which screams to get out of sync with
bsd.port.mk ;)

This port also demonstrates another use case we currently don't support:
adding to EXTRACT_ONLY with DISTFILES patterns.

I came here to look for potential fallout/fixes in case of the
MASTER_SITES_GITHUB changes, but this EXTRACT_ONLY issue(?) already
exists completely on its own.

So I'll put this in a different mail, trying to improve on EXTRACT_SUFX.


In the meantime, this diff cleans up the port and makes sure that it
would keep working with my proposed patch.

OK?

Index: Makefile
===================================================================
RCS file: /cvs/ports/x11/gnome/gdm/Makefile,v
retrieving revision 1.315
diff -u -p -r1.315 Makefile
--- Makefile    17 Apr 2022 10:28:58 -0000      1.315
+++ Makefile    13 Jul 2022 15:26:48 -0000
@@ -9,9 +9,10 @@ GH_PROJECT=            gdm
 GH_COMMIT=             864881c9ff9629711d782071fa959f76f2d8c3a7
 EXTRACT_SUFX=          .tar.gz
 
-# bsd.port.mk:1288
-DISTFILES=             
${DISTNAME}-${GH_COMMIT:C/(........).*/\1/}${EXTRACT_SUFX}{${GH_COMMIT}${EXTRACT_SUFX}}
 \
+DISTFILES=             ${GH_DISTFILE} \
                        gdm-puffy.png:0
+# XXX can't use GH_DISTFILE since EXTRACT_ONLY expects filenames instead of
+# DISTFILES filename{url}sufx patterns
 EXTRACT_ONLY=          
${DISTNAME}-${GH_COMMIT:C/(........).*/\1/}${EXTRACT_SUFX}
 
 MASTER_SITES=          ${MASTER_SITES_GITHUB}

Reply via email to