On Thu, 24 Aug 2023 12:48:22 +0200
Omar Polo <[email protected]> wrote:
> while I can't test on a !rust arch I did a few 'simulation' and it
> seems to package correctly both with and without rsvg.
>
> not super-familiar with plist fragments but it seems fine. ok op@
semarie@ showed me how to conditionally comment a file in PLIST.
This is easier than a plist fragment for a small number of files
(here, only svg.so).
The diff now sets COMMENT_SVG to an empty string or "@comment ".
I will commit it after I finish checking it.
Index: Makefile
===================================================================
RCS file: /cvs/ports/graphics/imlib2/Makefile,v
retrieving revision 1.49
diff -u -p -r1.49 Makefile
--- Makefile 7 May 2023 06:54:21 -0000 1.49
+++ Makefile 26 Aug 2023 21:53:59 -0000
@@ -1,6 +1,7 @@
COMMENT= image manipulation library
DISTNAME= imlib2-1.11.0
+REVISION= 0
SHARED_LIBS= Imlib2 8.0 # 12.0
CATEGORIES= graphics
@@ -11,14 +12,11 @@ PERMIT_PACKAGE= Yes
MASTER_SITES= ${MASTER_SITE_SOURCEFORGE:=enlightenment/}
-WANTLIB += ${COMPILER_LIBCXX} X11 X11-xcb Xau Xdmcp Xext Xrender
-WANTLIB += aom brotlicommon brotlidec brotlienc bz2 c cairo cairo-gobject
-WANTLIB += dav1d de265 expat ffi fontconfig freetype gdk_pixbuf-2.0
-WANTLIB += gif gio-2.0 glib-2.0 gmodule-2.0 gobject-2.0 harfbuzz
-WANTLIB += heif hwy iconv id3tag intl jpeg jxl jxl_threads lcms2
-WANTLIB += lzma m openjp2 pango-1.0 pangocairo-1.0 pangoft2-1.0
-WANTLIB += pcre2-8 pixman-1 png rsvg-2 tiff webp webpdemux x265
-WANTLIB += xcb xcb-render xcb-shm xml2 z zstd
+WANTLIB += ${COMPILER_LIBCXX} X11 X11-xcb Xau Xdmcp Xext aom brotlicommon
+WANTLIB += brotlidec brotlienc bz2 c dav1d de265 freetype gif
+WANTLIB += heif hwy id3tag jpeg jxl jxl_threads lcms2 lzma m openjp2
+WANTLIB += png sharpyuv tiff webp webpdemux x265 xcb xcb-shm z
+WANTLIB += zstd
COMPILER= base-clang ports-gcc
@@ -31,14 +29,29 @@ LIB_DEPENDS+= archivers/bzip2 \
graphics/openjp2 \
graphics/png \
graphics/tiff \
- multimedia/libheif \
- x11/gnome/librsvg
+ multimedia/libheif
SEPARATE_BUILD= Yes
CONFIGURE_STYLE= gnu
.if ${MACHINE_ARCH} == "i386"
CONFIGURE_ARGS+= --disable-mmx
.endif
+
+.include <bsd.port.arch.mk>
+.if ${PROPERTIES:Mrust}
+# We have librsvg-2.0 >= 2.46 only for rust archs.
+WANTLIB += Xrender cairo cairo-gobject expat ffi fontconfig gdk_pixbuf-2.0
+WANTLIB += gio-2.0 glib-2.0 gmodule-2.0 gobject-2.0 harfbuzz iconv
+WANTLIB += intl pango-1.0 pangocairo-1.0 pangoft2-1.0 pcre2-8
+WANTLIB += pixman-1 rsvg-2 xcb-render xml2
+LIB_DEPENDS+= x11/gnome/librsvg
+COMMENT_SVG=
+.else
+CONFIGURE_ARGS+= --without-svg
+COMMENT_SVG= "@comment "
+.endif
+SUBST_VARS+= COMMENT_SVG
+UPDATE_PLIST_ARGS+= -c COMMENT_SVG
CONFIGURE_ARGS+= --without-ps
Index: pkg/PLIST
===================================================================
RCS file: /cvs/ports/graphics/imlib2/pkg/PLIST,v
retrieving revision 1.12
diff -u -p -r1.12 PLIST
--- pkg/PLIST 29 Apr 2023 08:10:43 -0000 1.12
+++ pkg/PLIST 26 Aug 2023 21:53:59 -0000
@@ -33,7 +33,7 @@ lib/imlib2/loaders/
@so lib/imlib2/loaders/lzma.so
@so lib/imlib2/loaders/png.so
@so lib/imlib2/loaders/pnm.so
-@so lib/imlib2/loaders/svg.so
+${COMMENT_SVG}@so lib/imlib2/loaders/svg.so
@so lib/imlib2/loaders/tga.so
@so lib/imlib2/loaders/tiff.so
@so lib/imlib2/loaders/webp.so