Page MenuHomeFreeBSD

Enable separate flavors for different python versions.
ClosedPublic

Authored by jhb on Dec 12 2018, 9:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 28, 2:24 PM
Unknown Object (File)
Sun, Sep 28, 2:14 PM
Unknown Object (File)
Sun, Sep 28, 2:14 PM
Unknown Object (File)
Fri, Sep 26, 3:12 PM
Unknown Object (File)
Thu, Sep 25, 2:20 AM
Unknown Object (File)
Thu, Sep 25, 2:16 AM
Unknown Object (File)
Thu, Sep 25, 2:14 AM
Unknown Object (File)
Sep 18 2025, 8:57 PM
Subscribers

Details

Summary

Since python is enabled by default, this keeps the version with the
default python version as 'gdb' and adds a package suffix for other
python versions.

Test Plan
  • poudriere testport of both flavors and build of devel/gdb@all built both

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

devel/gdb/Makefile
88–92 ↗(On Diff #51933)

No.

PKGNAMESUFFIX=  ${PYTHON_PKGNAMESUFFIX}

And I would really rather it be non optional.

jhb marked an inline comment as done.Dec 12 2018, 11:15 PM
jhb added inline comments.
devel/gdb/Makefile
88–92 ↗(On Diff #51933)

See my thread on ports@ for why this negatively impacts the user experience. If it has to be non-optional I'll just drop the flavors instead.

I agree with John, about the suffix only for not default python version.
I need a bit of time, to properly test it.

Maybe it is my english not being correct, but I thought "I would rather" meant something that would be nice but not required.

The other change was, on the other hand, required.

Use PYTHON_PKGNAMESUFFIX.

jhb marked an inline comment as done.Dec 21 2018, 6:03 PM

Would it be better to use 'USES_PYTHON=optsuffix' (even though it's marked deprecated) instead of explicitly setting PKGNAMESUFFIX?

LGTM
totally agree with John, about package naming: by default the package should be called gdb. The gdb-py36 should be used for the special case only.

This revision is now accepted and ready to land.Dec 29 2018, 3:52 PM
This revision was automatically updated to reflect the committed changes.