Re: audio/beets: update to 2.1.0

2024-11-22 Thread Stuart Henderson
On 2024/11/22 12:29, Kirill A. Korinsky wrote:
> On Fri, 22 Nov 2024 12:08:01 +0100,
> Stuart Henderson  wrote:
> > 
> > On 2024/11/22 11:55, Kirill A. Korinsky wrote:
> > > ports@,
> > > 
> > > I'd like to update audio/beets to 2.1.0.
> > > 
> > > Here a diff which update beets, adds new optional dependencies which and
> > > drops man and tests. Yeah, here no more tests in pypi, and no man, see:
> > > https://github.com/beetbox/beets/releases/tag/v2.1.0
> > > 
> > > Tested on -curren/amd64.
> > 
> > Missing an RDEP on munkres? (not yet in ports, used in
> > beets/autotag/match.py)
> >
> 
> Are you sure? It has RDEP devel/py-algorithm-munkres

Ah I see, a badly named port then :)

> > py-typing-extensions shouldn't be needed since we have py 3.11.
> > 
> > Not required but I would recommend sorting deps by the order in
> > pyproject.toml, it makes it easier to check them.
> > 
> > README could do with an update, py-acoustid is now an RDEP so users
> > don't need telling to install it.
> 
> Thanks, here an updated diff:

thanks, OK.



Re: audio/beets: update to 2.1.0

2024-11-22 Thread Kirill A . Korinsky
On Fri, 22 Nov 2024 12:08:01 +0100,
Stuart Henderson  wrote:
> 
> On 2024/11/22 11:55, Kirill A. Korinsky wrote:
> > ports@,
> > 
> > I'd like to update audio/beets to 2.1.0.
> > 
> > Here a diff which update beets, adds new optional dependencies which and
> > drops man and tests. Yeah, here no more tests in pypi, and no man, see:
> > https://github.com/beetbox/beets/releases/tag/v2.1.0
> > 
> > Tested on -curren/amd64.
> 
> Missing an RDEP on munkres? (not yet in ports, used in
> beets/autotag/match.py)
>

Are you sure? It has RDEP devel/py-algorithm-munkres

> py-typing-extensions shouldn't be needed since we have py 3.11.
> 
> Not required but I would recommend sorting deps by the order in
> pyproject.toml, it makes it easier to check them.
> 
> README could do with an update, py-acoustid is now an RDEP so users
> don't need telling to install it.

Thanks, here an updated diff:

Index: Makefile
===
RCS file: /home/cvs/ports/audio/beets/Makefile,v
diff -u -p -r1.64 Makefile
--- Makefile4 Jun 2024 12:24:44 -   1.64
+++ Makefile22 Nov 2024 11:27:39 -
@@ -1,6 +1,6 @@
 COMMENT=   CLI tools to manage music collections
 
-MODPY_EGG_VERSION= 2.0.0
+MODPY_EGG_VERSION= 2.1.0
 DISTNAME=  beets-${MODPY_EGG_VERSION}
 CATEGORIES=audio
 
@@ -12,37 +12,30 @@ MAINTAINER= Kirill A. Korinsky 
Index: distinfo
===
RCS file: /home/cvs/ports/audio/beets/distinfo,v
diff -u -p -r1.24 distinfo
--- distinfo4 Jun 2024 12:24:44 -   1.24
+++ distinfo22 Nov 2024 10:46:40 -
@@ -1,2 +1,2 @@
-SHA256 (beets-2.0.0.tar.gz) = OxFytbw3KeM6bqRon30CNmgr+CjGcZa2omDwOJyx+M8=
-SIZE (beets-2.0.0.tar.gz) = 2194709
+SHA256 (beets-2.1.0.tar.gz) = oKvN+V1fifz0awfdwrW5Lh3Vq+aPTMciLyGv8t/8R4k=
+SIZE (beets-2.1.0.tar.gz) = 478597
Index: patches/patch-test_test_files_py
===
RCS file: patches/patch-test_test_files_py
diff -N patches/patch-test_test_files_py
--- patches/patch-test_test_files_py4 Jun 2024 12:24:44 -   1.1
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,18 +0,0 @@
-https://github.com/beetbox/beets/pull/5272
-
-Index: test/test_files.py
 test/test_files.py.orig
-+++ test/test_files.py
-@@ -88,10 +88,12 @@ class MoveTest(_common.TestCase):
- self.i.move(operation=MoveOperation.COPY)
- self.assertExists(self.path)
- 
-+@unittest.skipUnless(_common.HAVE_REFLINK, "need reflink")
- def test_reflink_arrives(self):
- self.i.move(operation=MoveOperation.REFLINK_AUTO)
- self.assertExists(self.dest)
- 
-+@unittest.skipUnless(_common.HAVE_REFLINK, "need reflink")
- def test_reflink_does_not_depart(self):
- self.i.move(operation=MoveOperation.REFLINK_AUTO)
- self.assertExists(self.path)
Index: pkg/PLIST
===
RCS file: /home/cvs/ports/audio/beets/pkg/PLIST,v
diff -u -p -r1.25 PLIST
--- pkg/PLIST   4 Jun 2024 12:24:45 -   1.25
+++ pkg/PLIST   22 Nov 2024 10:44:14 -
@@ -6,7 +6,6 @@ lib/python${MODPY_VERSION}/site-packages
 
lib/python${MODPY_VERSION}/site-packages/beets-${MODPY_EGG_VERSION}.dist-info/RECORD
 
lib/python${MODPY_VERSION}/site-packages/beets-${MODPY_EGG_VERSION}.dist-info/WHEEL
 
lib/python${MODPY_VERSION}/site-packages/beets-${MODPY_EGG_VERSION}.dist-info/entry_points.txt
-lib/python${MODPY_VERSION}/site-packages/beets-${MODPY_EGG_VERSION}.dist-info/top_level.txt
 lib/python${MODPY_VERSION}/site-packages/beets/__init__.py
 lib/python${MODPY_VERSION}/site-packages/beets/__main__.py
 
${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/beets/${MODPY_PYCACHE}/
@@ -98,10 +97,6 @@ lib/python${MODPY_VERSION}/site-packages
 
lib/python${MODPY_VERSION}/site-packages/beets/util/${MODPY_PYCACHE}artresizer.${MODPY_PYC_MAGIC_TAG}pyc
 
lib/python${MODPY_VERSION}/site-packages/beets/util/${MODPY_PYCACHE}bluelet.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
 
lib/python${MODPY_VERSION}/site-packages/beets/util/${MODPY_PYCACHE}bluelet.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/beets/util/${MODPY_PYCACHE}confit.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
-lib/python${MODPY_VERSION}/site-packages/beets/util/${MODPY_PYCACHE}confit.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/beets/util/${MODPY_PYCACHE}enumeration.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
-lib/python${MODPY_VERSION}/site-packages/beets/util/${MODPY_PYCACHE}enumeration.${MODPY_PYC_MAGIC_TAG}pyc
 
lib/python${MODPY_VERSION}/site-packages/beets/util/${MODPY_PYCACHE}functemplate.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
 
lib/python${MODPY_VERSION}/site-packages/beets/util/${MODPY_PYCACHE}functemplate.${MODPY_PYC_MAGIC_TAG}pyc
 
lib/python${MODPY_VERSION}/site-packages/beets/util/${MODPY_PYCACHE}hidden.${MODPY_PYC_MAGIC_TAG}${MODPY_PYOEXTENSION}
@@ -114,8

Re: audio/beets: update to 2.1.0

2024-11-22 Thread Stuart Henderson
On 2024/11/22 11:55, Kirill A. Korinsky wrote:
> ports@,
> 
> I'd like to update audio/beets to 2.1.0.
> 
> Here a diff which update beets, adds new optional dependencies which and
> drops man and tests. Yeah, here no more tests in pypi, and no man, see:
> https://github.com/beetbox/beets/releases/tag/v2.1.0
> 
> Tested on -curren/amd64.

Missing an RDEP on munkres? (not yet in ports, used in
beets/autotag/match.py)

py-typing-extensions shouldn't be needed since we have py 3.11.

Not required but I would recommend sorting deps by the order in
pyproject.toml, it makes it easier to check them.

README could do with an update, py-acoustid is now an RDEP so users
don't need telling to install it.