Re: [update] devel/zeal use release tarballs

[Adam Wolk]

On Sun, Mar 04, 2018 at 01:27:48PM +, Stuart Henderson wrote:
> Please use a temporary DISTFILES rename (see fs2open) to avoid changing the
> distfile under the same name, bulk builds don't handle it well.
> 
> -- 
>  Sent from a phone, apologies for poor formatting.
> 

same diff, switched to using EXTRACT_SUFX= .tar.xz to avoid DITFILES mangling.

OK?
Index: Makefile
===
RCS file: /cvs/ports/devel/zeal/Makefile,v
retrieving revision 1.9
diff -u -p -r1.9 Makefile
--- Makefile28 Feb 2018 22:52:53 -  1.9
+++ Makefile4 Mar 2018 13:55:43 -
@@ -5,9 +5,9 @@ USE_WXNEEDED =  Yes
 
 COMMENT =  simple offline API documentation browser
 
-GH_ACCOUNT =   zealdocs
-GH_PROJECT =   zeal
-GH_TAGNAME =   v0.6.0
+V =0.6.0
+DISTNAME = zeal-${V}
+REVISION = 0
 
 CATEGORIES =   devel
 
@@ -23,9 +23,9 @@ WANTLIB += Qt5WebKitWidgets Qt5Widgets Q
 WANTLIB += archive c m sqlite3 xcb xcb-keysyms
 WANTLIB += ${COMPILER_LIBCXX}
 
-WANTLIB += ICE Qt5Concurrent Qt5Core Qt5Gui Qt5Network Qt5WebKit
-WANTLIB += Qt5WebKitWidgets Qt5Widgets Qt5X11Extras SM X11 Xext
-WANTLIB += archive c m sqlite3 xcb xcb-keysyms
+MASTER_SITES = https://github.com/zealdocs/zeal/releases/download/v${V}/
+
+EXTRACT_SUFX = .tar.xz
 
 MODULES =  devel/cmake x11/qt5
 BUILD_DEPENDS =devel/kf5/extra-cmake-modules
Index: distinfo
===
RCS file: /cvs/ports/devel/zeal/distinfo,v
retrieving revision 1.5
diff -u -p -r1.5 distinfo
--- distinfo28 Feb 2018 22:52:53 -  1.5
+++ distinfo4 Mar 2018 13:55:43 -
@@ -1,2 +1,2 @@
-SHA256 (zeal-0.6.0.tar.gz) = 7zB9OtTwPHb6X8JJ+OfgTeh5qnV0vsH/i+VI28LAKXM=
-SIZE (zeal-0.6.0.tar.gz) = 1062223
+SHA256 (zeal-0.6.0.tar.xz) = 2eBFhGAbxQRh2Z41RRbkaJO/Leu+N3yxc9FUHU/glYo=
+SIZE (zeal-0.6.0.tar.xz) = 670416

Re: [update] devel/zeal use release tarballs

[Stuart Henderson]

Please use a temporary DISTFILES rename (see fs2open) to avoid changing the 
distfile under the same name, bulk builds don't handle it well.


--
 Sent from a phone, apologies for poor formatting.



On 4 March 2018 12:36:13 Adam Wolk  wrote:


Hi ports@,

devel/zeal upstream started doing release tarballs:
https://github.com/zealdocs/zeal/issues/897

Here is a diff to switch our port to them, additionally
I noticed that I accidentally duplicated WANTLIB during
the upgrade to 0.4 so while here drop the duplicated
lines.

Feedback? OK's?

Regards,
Adam