Opened 10 months ago

Closed 9 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:

  1. depends_build       port:python311 
    
  1. 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 Changed 10 months ago by ryandesign (Ryan Carsten Schmidt)

Owner: set to ryandesign
Status: newaccepted

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.

comment:2 in reply to:  1 ; Changed 10 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 in reply to:  2 Changed 10 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:

ref https://github.com/macports/macports-ports/pull/19728

comment:4 Changed 9 months ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: acceptedclosed

In afc36d10a3aa94fafac46a0ee34f633e313980c1/macports-ports (master):

kerberos5: Update to 1.21.2; don't look for python

Python is only used for the tests, which aren't enabled in the port, so
remove the python dependency and tell the configure script not to search
for python.

Closes: #67889
Closes: https://github.com/macports/macports-ports/pull/19728

Note: See TracTickets for help on using tickets.