FS#55834 - [vulkan-headers] /usr/src/vulkan.hpp includes out of date VK_HEADER_VERSION check (57 vs 61)

Attached to Project: Arch Linux
Opened by Lakin Wecker (lakin.wecker) - Sunday, 01 October 2017, 23:54 GMT
Last edited by Laurent Carlier (lordheavy) - Wednesday, 04 October 2017, 16:32 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Laurent Carlier (lordheavy)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description: /usr/src/vulkan.hpp includes an out-dated version check for version 57. The version in /usr/src/vulkan.h is 61 (which is the latest version.


Additional info:
version 1:1.0.61-1

Steps to reproduce:
➜ ack "VK_HEADER_VERSION" /usr/include/vulkan/vulkan.h
#define VK_HEADER_VERSION 61

~
➜ ack "VK_HEADER_VERSION" /usr/include/vulkan/vulkan.hpp
static_assert( VK_HEADER_VERSION == 57 , "Wrong VK_HEADER_VERSION!" );

~
➜ pacman -Qo /usr/include/vulkan/vulkan.hpp
/usr/include/vulkan/vulkan.hpp is owned by vulkan-headers 1:1.0.61-1

~
➜ pacman -Qo /usr/include/vulkan/vulkan.h
/usr/include/vulkan/vulkan.h is owned by vulkan-headers 1:1.0.61-1


I'm leaving the severity at low even though it makes it impossible to use the .hpp version. I'm doing this because I'm unsure what the arch severity levels should really mean and this is the default.
This task depends upon

Closed by  Laurent Carlier (lordheavy)
Wednesday, 04 October 2017, 16:32 GMT
Reason for closing:  Fixed
Additional comments about closing:  vulkan-headers-1:1.0.61-2
Comment by Lakin Wecker (lakin.wecker) - Monday, 02 October 2017, 00:00 GMT
It looks like you've pinned the commit for Vulkan-HPP to daae0b6 which is for the 57 version: https://raw.githubusercontent.com/KhronosGroup/Vulkan-Hpp/daae0b6194a197c4b119c7ee7efd31b501662bb8/vulkan/vulkan.hpp

https://raw.githubusercontent.com/KhronosGroup/Vulkan-Hpp/master/vulkan/vulkan.hpp has the appropriate version check.

Comment by Laurent Carlier (lordheavy) - Wednesday, 04 October 2017, 16:32 GMT
vulkan-headers-1:1.0.61-2 will not include anymore vulkan.hpp header as often it doesn't build.

Loading...