[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 Milan Bouchet-Valat changed: What|Removed |Added Status|ASSIGNED|CLOSED

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #29 from Orion Poplawski --- I think I'm good, so go ahead. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #28 from Milan Bouchet-Valat --- Thanks. I've pushed my patches. Do you need anything more, or should we trigger a build? -- You are receiving this mail because: You are on the CC list for the bug. You are

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #27 from Orion Poplawski --- Done. Thanks. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #26 from Milan Bouchet-Valat --- Sorry for the delay. I don't really care about where things are installed as long as I can build Julia with the package. Can you upload your version of the package to git? Then

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #25 from Orion Poplawski --- My updated install_dirs.patch doesn't quite work with your current -2 llvm package. I get: CMake Error at /usr/lib64/cmake/llvm3.9/LLVMConfig.cmake:165 (include): include could

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #24 from Orion Poplawski --- Here's my current version: https://www.cora.nwra.com/~orion/fedora/llvm3.9.spec https://www.cora.nwra.com/~orion/fedora/llvm3.9-3.9.1-4.el7.src.rpm * Fri Feb 10 2017 Orion

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #23 from Orion Poplawski --- It's really not that different from how we handle MPI packages: http://fedoraproject.org/wiki/Packaging:MPI What would really be good to do is to have clang's CMakeLists.txt do:

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #22 from Milan Bouchet-Valat --- I don't like the idea of putting everything under /lib64/llvm3.9. That probably doesn't follow the FHS, even if we add symlinks. It took me quite some work to improve LLVM's

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #21 from Orion Poplawski --- Here is my current spec that installs into prefix: https://www.cora.nwra.com/~orion/fedora/llvm3.9.spec I've also kept the versioned binaries for everything except llvm-config.

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #20 from Milan Bouchet-Valat --- I haven't had the time to look in depth to this, but it looks like the incorrect LLVMConfing.cmake path should be fixed upstream. I think there's even a CMake variable for this,

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #19 from Orion Poplawski --- Well, my attempt at installing into will not work with current llvm-config if we want to keep the llvm-%{major_version} names in /usr/bin. Multi-lib requires those names to the

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #18 from Orion Poplawski --- And now looking at 3.9 - /usr/lib64/cmake/llvm3.9/LLVMExports.cmake:set(_IMPORT_PREFIX "/usr") leads to errors like: CMake Error at

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #17 from Orion Poplawski --- Here are the issues I've encountered trying to use llvm3.X, including building clang3.X with llvm3.X: - clang wants to find 'llvm-config', but currently this is

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #16 from Milan Bouchet-Valat --- I don't understand what you mean. We don't currently move libraries around, we just pass the needed paths to configure/cmake. My patch against the CMake system ensures

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #15 from Orion Poplawski --- My bad - I got confused with too many different versions at the same time. I've been playing around with trying to build a versioned clang with the versioned llvm packages (you've

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #14 from Milan Bouchet-Valat --- Are you sure? I just downloaded the SRPM again, and the .spec file calls %cmake, not ./configure. Update on progress: the patch is still being reviewed upstream. If that takes

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #13 from Orion Poplawski --- I don't think the content of https://nalimilan.fedorapeople.org/llvm3.9-3.9.1-2.fc24.src.rpm is correct. -- You are receiving this mail because: You are on the CC list for the

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #12 from Orion Poplawski --- This package doesn't build for me: + ./configure --build=x86_64-redhat-linux-gnu --host=x86_64-redhat-linux-gnu --program-prefix= --disable-dependency-tracking --prefix=/usr

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #11 from Milan Bouchet-Valat --- No, it doesn't contain clang. Indeed it's a separate package since 3.7 AFAICT, so somebody will have to package it separately. Note that since the port to CMake, versioned

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 Jeremy Newton changed: What|Removed |Added CC|

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #9 from Kevin Fenzi --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/llvm3.9 -- You are receiving this mail because: You are on the CC list for the bug. You are always

[Bug 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 Kevin Fenzi changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug 1409363] Review Request: llvm3.9 -- Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #8 from Milan Bouchet-Valat --- Yes, for now rawhide has 3.9.0, but I expect F26 to ship with 4.0, which will be released in February. So indeed for the next few weeks it's going to be redundant, but I figured

[Bug 1409363] Review Request: llvm3.9 -- Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #7 from Jens Petersen --- Just for the record llvm 3.9.0 is already in Rawhide since end of Nov. Or maybe do you still need llvm3.9 for current releases? -- You are receiving this mail because: You are on

[Bug 1409363] Review Request: llvm3.9 -- Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #6 from Milan Bouchet-Valat --- No worries. Here's the Koji build on all arches: https://koji.fedoraproject.org/koji/taskinfo?taskID=17155580 -- You are receiving this mail because: You are on the CC list for

[Bug 1409363] Review Request: llvm3.9 -- Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 Kevin Fenzi changed: What|Removed |Added CC||ke...@scrye.com ---

[Bug 1409363] Review Request: llvm3.9 -- Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #4 from Milan Bouchet-Valat --- Kevin: I'm fine with going through a review, but I was told the contrary for llvm3.7, and the rules at

[Bug 1409363] Review Request: llvm3.9 -- Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #3 from Orion Poplawski --- llvm3.9.spec:35: W: mixed-use-of-spaces-and-tabs (spaces: line 35, tab: line 10) I think -static is missing: Requires: %{name}-devel%{?_isa} = %{version}-%{release} No

[Bug 1409363] Review Request: llvm3.9 -- Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #2 from Kevin Fenzi --- Package request has been denied with the reason: The package must be reviewed and fedora-review flag set to + before you can request the package. -- You are receiving this mail because:

[Bug 1409363] Review Request: llvm3.9 -- Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 --- Comment #1 from Milan Bouchet-Valat --- I've just filed an SCM admin request. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and

[Bug 1409363] Review Request: llvm3.9 -- Versioned LLVM 3.9

https://bugzilla.redhat.com/show_bug.cgi?id=1409363 Milan Bouchet-Valat changed: What|Removed |Added CC|