Bug 268489 - graphics/vulkan-headers: invalid shebang for installed python script
Summary: graphics/vulkan-headers: invalid shebang for installed python script
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Jan Beich
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2022-12-20 17:46 UTC by Dmitry Marakasov
Modified: 2022-12-20 19:06 UTC (History)
0 users

See Also:
jbeich: maintainer-feedback+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dmitry Marakasov freebsd_committer freebsd_triage 2022-12-20 17:46:26 UTC 
====> Running Q/A tests (stage-qa)
Error: '/usr/bin/python3' is an invalid shebang you need USES=shebangfix for 'share/vulkan/registry/genvk.py'
Comment 1 Jan Beich freebsd_committer freebsd_triage 2022-12-20 18:03:27 UTC 
genvk.py always had such bogus shebang but stage-qa didn't complain before ports 93b836bc9e65 because it didn't have executable bit set. Adding USES=shebangfix would embed /usr/local/bin/python unless USES=python:build which would use /usr/local/bin/python3.9 which makes no sense for **installed** file without USES=python:run.
Comment 2 commit-hook freebsd_committer freebsd_triage 2022-12-20 18:49:08 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=c4b199a36a122b38b2235082fa176c41f7f13343

commit c4b199a36a122b38b2235082fa176c41f7f13343
Author:     Jan Beich <jbeich@FreeBSD.org>
AuthorDate: 2022-12-20 18:16:26 +0000
Commit:     Jan Beich <jbeich@FreeBSD.org>
CommitDate: 2022-12-20 18:48:43 +0000

    graphics/vulkan-headers: pacify stage-qa after 93b836bc9e65

    Error: '/usr/bin/python3' is an invalid shebang you need USES=shebangfix for 'share/vulkan/registry/genvk.py'

    genvk.py is used to generate include/vulkan/*.h but distfile already
    contains pregerenated version, so the shebang is unused. Regenerating
    headers isn't supported by CMake build, and requires docgenerator.py
    from Vulkan-Docs repo.

    PR:             268489
    Reported by:    amdmi3
    Regressed by:   https://github.com/KhronosGroup/Vulkan-Headers/commit/75a6b83f213d

 graphics/vulkan-headers/Makefile | 5 +++++
 1 file changed, 5 insertions(+)