Bug 1310355

Summary: Review Request: spirv-headers - Khronos SPIR-V development headers
Product: [Fedora] Fedora Reporter: Igor Gnatenko <ignatenko>
Component: Package ReviewAssignee: Zbigniew Jędrzejewski-Szmek <zbyszek>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: airlied, ajax, package-review, zbyszek
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-08-22 07:31:24 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Igor Gnatenko 2016-02-20 21:55:46 UTC 
Spec URL: https://ignatenkobrain.fedorapeople.org/for-review/spirv-headers.spec
SRPM URL: https://ignatenkobrain.fedorapeople.org/for-review/spirv-headers-1.0-1.fc24.src.rpm
Description: Khronos SPIR-V development headers
Fedora Account System Username: ignatenkobrain
Comment 1 Zbigniew Jędrzejewski-Szmek 2016-03-11 15:18:22 UTC 
spirv-headers.src: W: no-%build-section
Better to add an empty one.

About the name:
upstream project is SPIR-V, the URL contains "spir-v" too. Shouldn't the name of the package be spir-v-headers?

I think you should also include https://www.khronos.org/registry/spir-v/api/spir-v.xml. Unfortunately the docs are non-redistributable. It would be nice to include them otherwise.
Comment 2 Igor Raits 2018-08-22 07:31:24 UTC 
Unfortunately I don't have time to work on these review requests anymore, sorry.