Bug 1489983

Summary: Review Request: llvm4.0 - Compatibility package for LLVM 4.0
Product: [Fedora] Fedora Reporter: Tom Stellard <tstellar>
Component: Package ReviewAssignee: Jason Tibbitts <j>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: jistone, package-review
Target Milestone: ---Flags: j: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-02-01 22:22:30 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Comment 1 Josh Stone 2017-09-08 23:51:12 UTC
The main llvm package has a backport for ARM codegen which is important for rust, rust-lang-llvm-pr67.patch.  Any reason you didn't include this here?

Comment 2 Jason Tibbitts 2017-09-09 00:11:24 UTC
You don't need to go through the review process simply to import a parallel-installable version of an existing package.

https://fedoraproject.org/wiki/Package_Review_Process
https://fedoraproject.org/wiki/Packaging:ReviewGuidelines#Package_Review_Process

Yes, finding this is unpleasantly difficult, and unfortunately our new fedrepo-req process doesn't play well with things.  But it appears to me that the package is named properly, and is indeed a versioned import of the llvm package, so I'll just approve this and you can import it with this ticket as a reference.

Comment 3 Gwyn Ciesla 2017-10-16 13:38:54 UTC
(fedrepo-req-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/llvm4.0