From: Omar Polo <op@omarpolo.com>
Subject: Re: [new] multimedia/mat2
To: Jag Talon <jag@aangat.lahat.computer>
Cc: ports@openbsd.org
Date: Thu, 18 Jul 2024 13:00:19 +0200

On 2024/07/17 09:51:49 -0400, Jag Talon <jag@aangat.lahat.computer> wrote:
> Ping! Does this look ok to everyone?

with RUN_DEPENDS sorted and a tiny formatting change to DESCR it's OK
op@ to import.

Just a curiosity, why don't call it mat2 instead of py3-mat2?  I
understand that it also provides a library, but I guess the main
entrypoint is the binary bin/mat2 and so the package could be called
like that?

I'm attaching a diff on top of your tarball just to show what I mean.
Only the first hunk is for my proposal to call it "mat2".


--- Makefile.orig	Thu Jul 18 12:54:07 2024
+++ Makefile	Thu Jul 18 12:55:39 2024
@@ -2,7 +2,6 @@
 
 MODPY_EGG_VERSION =	0.13.4
 DISTNAME =		mat2-${MODPY_EGG_VERSION}
-PKGNAME =		py-${DISTNAME}
 
 CATEGORIES =		multimedia
 
@@ -20,14 +19,14 @@
 FLAVORS =		python3
 FLAVOR =		python3
 
-RUN_DEPENDS =		graphics/libimagequant \
-			x11/gnome/librsvg \
-			graphics/gdk-pixbuf2 \
-			print/poppler \
-			audio/py-mutagen${MODPY_FLAVOR} \
+RUN_DEPENDS =		audio/py-mutagen${MODPY_FLAVOR} \
 			devel/py-gobject3${MODPY_FLAVOR} \
+			graphics/ffmpeg \
+			graphics/gdk-pixbuf2 \
+			graphics/libimagequant \
 			graphics/p5-Image-ExifTool \
-			graphics/ffmpeg
+			print/poppler \
+			x11/gnome/librsvg
 
 # Test files aren't available in PyPI
 NO_TEST =		Yes
--- pkg/DESCR.orig	Thu Jul 18 12:53:44 2024
+++ pkg/DESCR	Thu Jul 18 12:53:49 2024
@@ -1,5 +1,6 @@
 mat2 is a metadata removal tool, supporting a wide range of commonly
-used file formats. It provides a Python library and a command-line interface.
+used file formats. It provides a Python library and a command-line
+interface.
 
 Supported formats include: avi, bmp, css, epub/ncx, flac, gif, jpeg,
 m4a/mp2/mp3, mp4, odc/odf/odg/odi/odp/ods/odt, off/opus/oga/spx, pdf,