Re: CVS commit: pkgsrc/lang/python311

[Adam <adam%netbsd.org@localhost>]

> Modified Files:
> pkgsrc/lang/python311: Makefile PLIST
> 
> Log Message:
> Remove dependency on devel/gettext-lib, we don`t need it
> 
> 
> To generate a diff of this commit:
> cvs rdiff -u -r1.26 -r1.27 pkgsrc/lang/python311/Makefile
> cvs rdiff -u -r1.12 -r1.13 pkgsrc/lang/python311/PLIST
> 
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
> 
> Modified files:
> 
> Index: pkgsrc/lang/python311/Makefile
> diff -u pkgsrc/lang/python311/Makefile:1.26 pkgsrc/lang/python311/Makefile:1.27
> --- pkgsrc/lang/python311/Makefile:1.26 Tue Apr 30 17:01:15 2024
> +++ pkgsrc/lang/python311/Makefile Tue Apr 30 17:13:55 2024
> @@ -1,4 +1,4 @@
> -# $NetBSD: Makefile,v 1.26 2024/04/30 17:01:15 cheusov Exp $
> +# $NetBSD: Makefile,v 1.27 2024/04/30 17:13:55 cheusov Exp $
> 
> .include "dist.mk"
> 
> @@ -239,7 +239,6 @@ setuptools-preinstall:
> .include "../../archivers/bzip2/buildlink3.mk"
> .include "../../archivers/xz/buildlink3.mk"
> .include "../../databases/sqlite3/buildlink3.mk"
> -.include "../../devel/gettext-lib/buildlink3.mk"
> .if ${USE_BUILTIN.gettext:U:tl} == no
> CONFIGURE_ENV+= ac_cv_lib_intl_textdomain=yes
> .endif
> 
> Index: pkgsrc/lang/python311/PLIST
> diff -u pkgsrc/lang/python311/PLIST:1.12 pkgsrc/lang/python311/PLIST:1.13
> --- pkgsrc/lang/python311/PLIST:1.12 Tue Apr 30 17:04:13 2024
> +++ pkgsrc/lang/python311/PLIST Tue Apr 30 17:13:55 2024
> @@ -1,4 +1,4 @@
> -@comment $NetBSD: PLIST,v 1.12 2024/04/30 17:04:13 cheusov Exp $
> +@comment $NetBSD: PLIST,v 1.13 2024/04/30 17:13:55 cheusov Exp $
> bin/2to3-${PY_VER_SUFFIX}
> bin/pydoc${PY_VER_SUFFIX}
> bin/python${PY_VER_SUFFIX}
> @@ -195,6 +195,7 @@ include/python${PY_VER_SUFFIX}/warnings.
> include/python${PY_VER_SUFFIX}/weakrefobject.h
> lib/libpython${PY_VER_SUFFIX}.so
> lib/libpython${PY_VER_SUFFIX}.so.1.0
> +lib/python3.11/lib-dynload/_dbm.so
> lib/pkgconfig/python-${PY_VER_SUFFIX}-embed.pc
> lib/pkgconfig/python-${PY_VER_SUFFIX}.pc
> lib/python${PY_VER_SUFFIX}/LICENSE.txt

This is wrong. It should be:

lib/python${PY_VER_SUFFIX}/lib-dynload/_dbm.so

but there is already this entry in PLIST:

${PLIST.dbm}lib/python${PY_VER_SUFFIX}/lib-dynload/_dbm.so

so you must deal with PLIST.dbm=yes in Makefile.


Kind regards,
Adam