Re: CVS commit: pkgsrc/www/py-autobahn

[Thomas Klausner <wiz%netbsd.org@localhost>]

Hi!

This fixes the build on NetBSD (in the bulk builds, and for me), but
breaks it on SmartOS and Darwin:

https://reports.pkgci.org/SmartOS/upstream/trunk/20251214.2248/py311-autobahn-25.12.1nb1/install.log
https://reports.pkgci.org/Darwin/14.5/arm64/20251215.0224/py311-autobahn-25.12.1nb1/install.log

=> Checking file-check results for py311-autobahn-25.12.1nb1
ERROR: ************************************************************
ERROR: The following files are in /home/pbulk/build/www/py-autobahn/work/.destdir/opt/pkg but not in the PLIST:
ERROR:         /home/pbulk/build/www/py-autobahn/work/.destdir/opt/pkg/lib/python3.11/site-packages/autobahn/nvx/_nvx_utf8validator.c
ERROR:         /home/pbulk/build/www/py-autobahn/work/.destdir/opt/pkg/lib/python3.11/site-packages/autobahn/nvx/_nvx_utf8validator.o
ERROR:         /home/pbulk/build/www/py-autobahn/work/.destdir/opt/pkg/lib/python3.11/site-packages/autobahn/nvx/_nvx_xormasker.c
ERROR:         /home/pbulk/build/www/py-autobahn/work/.destdir/opt/pkg/lib/python3.11/site-packages/autobahn/nvx/_nvx_xormasker.o
*** Error code 1

I looked a bit but I don't see why those files are installed - only
the *.so files are needed, this is the source code and an unused
object file. So I think they should not be installed.

We could, as a workaround, '${RM} -f' them in post-install, but
perhaps someone has an idea where they come from?

Cheers,
 Thomas


On Sat, Dec 13, 2025 at 10:24:37AM +0100, David Brownlee wrote:
> Module Name:	pkgsrc
> Committed By:	abs
> Date:		Sat Dec 13 10:24:37 UTC 2025
> 
> Modified Files:
> 	pkgsrc/www/py-autobahn: Makefile PLIST
> 
> Log Message:
> Fix PLIST, bump PKGREVISION
> 
> 
> To generate a diff of this commit:
> cvs rdiff -u -r1.44 -r1.45 pkgsrc/www/py-autobahn/Makefile
> cvs rdiff -u -r1.22 -r1.23 pkgsrc/www/py-autobahn/PLIST
> 
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
> 

> Modified files:
> 
> Index: pkgsrc/www/py-autobahn/Makefile
> diff -u pkgsrc/www/py-autobahn/Makefile:1.44 pkgsrc/www/py-autobahn/Makefile:1.45
> --- pkgsrc/www/py-autobahn/Makefile:1.44	Thu Dec 11 15:13:51 2025
> +++ pkgsrc/www/py-autobahn/Makefile	Sat Dec 13 10:24:37 2025
> @@ -1,7 +1,8 @@
> -# $NetBSD: Makefile,v 1.44 2025/12/11 15:13:51 adam Exp $
> +# $NetBSD: Makefile,v 1.45 2025/12/13 10:24:37 abs Exp $
>  
>  DISTNAME=	autobahn-25.12.1
>  PKGNAME=	${PYPKGPREFIX}-${DISTNAME}
> +PKREVISION=	1
>  CATEGORIES=	www python
>  MASTER_SITES=	${MASTER_SITE_PYPI:=a/autobahn/}
>  
> 
> Index: pkgsrc/www/py-autobahn/PLIST
> diff -u pkgsrc/www/py-autobahn/PLIST:1.22 pkgsrc/www/py-autobahn/PLIST:1.23
> --- pkgsrc/www/py-autobahn/PLIST:1.22	Thu Dec 11 15:13:51 2025
> +++ pkgsrc/www/py-autobahn/PLIST	Sat Dec 13 10:24:37 2025
> @@ -1,4 +1,4 @@
> -@comment $NetBSD: PLIST,v 1.22 2025/12/11 15:13:51 adam Exp $
> +@comment $NetBSD: PLIST,v 1.23 2025/12/13 10:24:37 abs Exp $
>  bin/wamp-${PYVERSSUFFIX}
>  ${PYSITELIB}/${WHEEL_INFODIR}/METADATA
>  ${PYSITELIB}/${WHEEL_INFODIR}/RECORD
> @@ -55,10 +55,6 @@ ${PYSITELIB}/autobahn/nvx/__init__.pyo
>  ${PYSITELIB}/autobahn/nvx/_compile_args.py
>  ${PYSITELIB}/autobahn/nvx/_compile_args.pyc
>  ${PYSITELIB}/autobahn/nvx/_compile_args.pyo
> -${PYSITELIB}/autobahn/nvx/_nvx_utf8validator.c
> -${PYSITELIB}/autobahn/nvx/_nvx_utf8validator.o
> -${PYSITELIB}/autobahn/nvx/_nvx_xormasker.c
> -${PYSITELIB}/autobahn/nvx/_nvx_xormasker.o
>  ${PYSITELIB}/autobahn/nvx/_utf8validator.c
>  ${PYSITELIB}/autobahn/nvx/_utf8validator.py
>  ${PYSITELIB}/autobahn/nvx/_utf8validator.pyc
>