[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-10-31 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #39 from Fedora Update System --- llvm33-3.3-3.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-10-31 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|---

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-10-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #38 from Milan Bouchet-Valat --- I wouldn't worry too much about keeping them in sync: given the current upstream plans, F24 will be released with the next version of Julia, which will probably rely on LLVM 3.8. So llvm33 would onl

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-10-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #37 from Jens Petersen --- (In reply to Milan Bouchet-Valat from comment #36) > For -O3, I agree to drop it, but since it has worked for so long it doesn't > sound like > a serious issue. I'll fix that in the .spec file, but not w

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-10-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #36 from Milan Bouchet-Valat --- Ah, good catch, owning an empty dir doesn't make much sense. For -O3, I agree to drop it, but since it has worked for so long it doesn't sound like a serious issue. I'll fix that in the .spec file,

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-10-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #35 from Jens Petersen --- BTW minor thing - still not sure if llvm33 needs to own '%dir %{_datadir}/%{name}'? Keeping it in llvm33-devel looks okay to me. And I noticed that I dropped -O3 from llvm34 - maybe you want to too? I wi

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-10-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #34 from Fed

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-10-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #33 from Fedora Update System --- llvm33-3.3-3.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-c39b0b2f3b -- You are receiving this mail because: You are on the CC list for t

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-10-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-10-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 Jens Petersen changed: What|Removed |Added Flags||fedora-review+ --- Comment #32 from J

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-10-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #31 from Milan Bouchet-Valat --- Jens, would you finish your review? We should try to get both llvm3.3 and llvm3.5 in. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about chan

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-08-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #30 from Jens Petersen --- (In reply to Jens Petersen from comment #28) > some way to add a prefix. Sorry suffix. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-08-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #29 from Jens Petersen --- (In reply to Milan Bouchet-Valat from comment #27) > Ah, I didn't know that. I can review it if you want. Sounds like a logical > exchange. Okay if you have time that would be appreciated. -- You are r

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-08-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #28 from Jens Petersen --- Thanks for all your work on this. I am not sure which binary package should really correctly own the datadir but overall the package looks fine to me now. The only outstanding/embarrassing remaining iss

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-08-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #27 from Milan Bouchet-Valat --- Ah, I didn't know that. I can review it if you want. Sounds like a logical exchange. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about chang

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-08-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #26 from Jens Petersen --- I note in passing that my llvm35 review is still open too (bug 1223673). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-08-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #25 from Jens Petersen --- Thanks - will try to find time tomorrow to look at this again. -- 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 compo

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-08-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #24 from Milan Bouchet-Valat --- Bump! :-) -- 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 1109390] Review Request: llvm33 - Versioned LLVM

2015-08-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #23 from Milan Bouchet-Valat --- Sorry for the delay. I've synchronized llvm33.spec with your changes to llvm34.spec, and rebased the changelog and a few details on llvm.spec version 3.3. This one should be good to go -- and will a

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-07-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #22 from Jens Petersen --- Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated Issues: === - Header files in -devel subpackage, if present. Note: llvm33-doc : /usr/share/d

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-07-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #21 from Jens Petersen --- One thing, it also be good to compare with the old llvm.spec for 3.3 and also rebase the changelog off that. http://pkgs.fedoraproject.org/cgit/llvm.git/tree/llvm.spec?h=f19 -- You are receiving this m

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-07-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #20 from Jens Petersen --- http://pkgs.fedoraproject.org/cgit/llvm34.git/commit/?id=3c25e4ba6538f2f804a5b1f0ed67ea39e95f8c68 (I think it is good to try to keep the spec files for llvm33 <-> llvm34 <-> llvm35 <-> llvm36 in sync.)

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-07-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 --- Comment #19 from Jens Petersen --- (In reply to Milan Bouchet-Valat from comment #18) > OK, I've started from your llvm34 package, and with very few changes it > works. Thanks > %changelog > +* Sat Jul 04 2015 Milan Bouchet-Valat 3.3-1

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-07-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 Jens Petersen changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedo

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-07-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 Milan Bouchet-Valat changed: What|Removed |Added Blocks||1231163 Referenced Bugs: http

[Bug 1109390] Review Request: llvm33 - Versioned LLVM

2015-07-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1109390 Milan Bouchet-Valat changed: What|Removed |Added Summary|Review Request: llvm3.3 - |Review Request: llvm33 -