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)
Jan 31 2024, 9:48 AM
Unknown Object (File)
Jan 5 2024, 10:01 PM
Unknown Object (File)
Dec 20 2023, 6:58 AM
Unknown Object (File)
Nov 15 2023, 2:09 AM
Unknown Object (File)
Oct 30 2023, 9:39 PM
Unknown Object (File)
Oct 14 2023, 1:10 AM
Unknown Object (File)
Sep 28 2023, 9:55 PM
Unknown Object (File)
Aug 12 2023, 1:22 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
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 21522
Build 20835: arc lint + arc unit

Event Timeline

devel/gdb/Makefile
88–92

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

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.