#68362 closed defect (fixed)

wireshark{3,4} doesn't respect +python variant when building

Reported by: hexane360 (Colin Gilgenbach) Owned by: ghosthound
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: wireshark4

Description

Wireshark has +python variants, but currently those just ensure *a* python is installed, not that the specified python is actually used.

The following diff seems to fix this, but it hasn't been tested rigorously:

--- macports-ports/net/wireshark4/Portfile	2023-10-04 14:47:14
+++ ports/net/wireshark4/Portfile	2023-10-05 11:51:44
@@ -172,6 +172,8 @@
     set py_conflicts [lreplace ${pythons_ports} ${i} ${i}]
     variant ${py_port} description "Use python ${py_ver} during build" conflicts {*}${py_conflicts} "
         depends_build-append      port:${py_port}
+        configure.args-append     -DPython3_ROOT_DIR=\"${frameworks_dir}/Python.framework/Versions/${py_ver}/bin\"
+        configure.args-append     -DPython3_FIND_STRATEGY=LOCATION
     "
 }

Change History (3)

comment:1 Changed 13 months ago by jmroot (Joshua Root)

Owner: set to ghosthound
Status: newassigned

comment:2 in reply to:  description Changed 13 months ago by ryandesign (Ryan Carsten Schmidt)

Keywords: python cmake removed

Replying to hexane360:

+        configure.args-append     -DPython3_ROOT_DIR=\"${frameworks_dir}/Python.framework/Versions/${py_ver}/bin\"

Why would these quotes need to be preceded with backslashes?

Version 0, edited 13 months ago by ryandesign (Ryan Carsten Schmidt) (next)

comment:3 Changed 13 months ago by i0ntempest

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.