Opened 16 months ago
Closed 15 months ago
#67889 closed defect (fixed)
kerberos5: @1.21.1_1 - deviation in Portfile & Python311 dependency
Reported by: | Zweihorn (@Zweihorn) | Owned by: | ryandesign (Ryan Carsten Schmidt) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.8.1 |
Keywords: | Cc: | ||
Port: | kerberos5 |
Description
Please note deviation in Portfile as:
-
depends_build port:python311
-
configure.python ${prefix}/bin/python3.10
Maybe (2.) should be changed to "${prefix}/bin/python3.11" at maintainer's deliberation?
Change History (4)
comment:1 follow-up: 2 Changed 16 months ago by ryandesign (Ryan Carsten Schmidt)
Owner: | set to ryandesign |
---|---|
Status: | new → accepted |
comment:2 follow-up: 3 Changed 16 months ago by Zweihorn (@Zweihorn)
Replying to ryandesign:
I will want to contact the developers to report that the configure script should not allow a wrong python value, if I can determine how to do so.
Thank you for the insight and I appreciate your concern.
Just as an idea and I tested this. Maybe an intermediate fix in the Portfile could be:
set python_branch 3.11 set python_version [strsed ${python_branch} {s|\.||}] . . . depends_build port:python${python_version} configure.python ${prefix}/bin/python${python_branch}
I derived this from the MacPorts Guide ref https://guide.macports.org/index.html#reference.portgroup.python.variables
However, a "PortGroup Python" would be an overkill or not helpful at all, I guess. That is why I introduced the a.m. local variables and schematic.
Hope this helps.
comment:3 Changed 16 months ago by Zweihorn (@Zweihorn)
Replying to Zweihorn:
Replying to ryandesign:
I will want to contact the developers to report that the configure script should not allow a wrong python value, if I can determine how to do so.
Thank you for the insight and I appreciate your concern.
Just as an idea and I tested this. Maybe an intermediate fix in the Portfile could be:
comment:4 Changed 15 months ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
I agree, that's wrong. And yet the checks of the PR that made this change succeeded. I fixed the problem locally and did a new build and diffed the contents with the previous build and there was no difference (aside from expected differences in all binary files). Reading configure.ac, it says python is used by some of the tests but the Portfile doesn't even enable the test suite. I will see if I can remove the python dependency without upsetting the configure script too much.
I will want to contact the developers to report that the configure script should not allow a wrong python value, if I can determine how to do so.