Re: textproc/py27-expat requires python27-2.7.16

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

On Tue, Jun 04, 2019 at 06:56:29AM -0400, Greg Troxel wrote:
> "Sijmen J. Mulder" <ik%sjmulder.nl@localhost> writes:
> 
>   Index: lang/python/srcdist.mk
>   ===================================================================
>   RCS file: /cvsroot/pkgsrc/lang/python/srcdist.mk,v
>   retrieving revision 1.37
>   diff -u -r1.37 srcdist.mk
>   --- lang/python/srcdist.mk	3 Jul 2018 03:55:40 -0000	1.37
>   +++ lang/python/srcdist.mk	4 Jun 2019 06:25:20 -0000
>   @@ -15,6 +15,10 @@
>    EXTRACT_ELEMENTS+=	${PYSUBDIR}/Modules/clinic
>    .  endif
> 
>   +# Standard modules depend on their own version
>   +BUILDLINK_API_DEPENDS.${PYPACKAGE}+=	${PYPACKAGE}>=${PY_DISTVERSION}
>   +BUILDLINK_ABI_DEPENDS.${PYPACKAGE}+=	${PYPACKAGE}>=${PY_DISTVERSION}
>   +
>    python-std-patchsetup:
>           ${SED} ${PY_SETUP_SUBST:S/=/@!/:S/$/!g/:S/^/ -e s!@/} \
>                   <${FILESDIR}/setup.py >${WRKSRC}/setup.py
> 
> 
> I wonder if it should be =, not >=, but otherwise that seems to avoid

How would you do "=" considering PKGREVISION?

You'd need <${PY_DISTVERSION+1} with a magic +1 operator that bumps
the last part of the version number :)
 Thomas

> the issue about bumping requirements for random other packages.
> I'd say if you don't get a good explanation of why it's wrong in 24h, go
> for it.
>