Possible +x bit missing on /usr/share/vulkan/registry/*.py

[jengelh]

Possible +x bit missing on /usr/share/vulkan/registry/*.py

[jengelh]

Nah, rpmlint is a more of a distro matter. Thanks everyone here and for #337.

[oddhack]

I don't know who is constructing the package you're installing, but in both the Vulkan-Docs repo and Vulkan-Headers, genvk.py has the exec bit set, so it sounds like something is being lost - I suggest reporting to to whomever provides vulkan-headers.noarch. The other scripts are all imported and would not even potentially be invoked directly. N.b. genvk.py can be invoked directly in arcane debugging circumstances, but normally is just called as 'python3 genvk.py' from makefiles and build scripts.

If that answers the question, please close. I don't think there's anything we can do in this repo.

[oddhack]

(I should perhaps add, genvk.py has the exec bit set in my clone of Vulkan-Headers - I can't quite tell from the github UI whether that's true in the upstream but I don't know why it wouldn't be.)

[MarijnS95]

$ git ls-files -s registry/genvk.py
100755 4cd362f02e343694c465f62f354d6e4773851a1c 0	registry/genvk.py

+x is set for ugo.

[jengelh]

though genvk.py has the +x bit, cmake install does not transfer the bit over.

install(DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/registry" DESTINATION ${CMAKE_INSTALL_DATADIR}/vulkan)

I'm not versed in cmake, but systems like automake for example make a distinction between "data" and "scripts" (one gets installed -x, but other +x), and perhaps cmake has something similar you need to use.

apiconventions.py is -x in git (but still has #! line). This is where the question was aimed. The #! looks to serve just as a syntax hint, so me (as the packager) added an ignore directive.

[juan-lunarg]

I made a PR partially address this issue.
#337

Note this only fixes the problem for genvk.py. The other python files will need to be updated to not have the shebang. Since it looks like they aren't meant to be standalone python scripts (they look more like python modules).

Those files would need to be updated in the Vulkan-Docs repository:
https://github.com/KhronosGroup/Vulkan-Docs

[juan-lunarg]

Closing this issue since there is nothing more that can be done in this repository now that CMake preserves the file permissions as of #337

To fix the remaining rpmlint issues, that will need to addressed in https://github.com/KhronosGroup/Vulkan-Docs