[Bug 1352215] Review Request: llvm37 -- Versioned LLVM 3.7

2016-08-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352215



--- Comment #7 from Milan Bouchet-Valat  ---
Actually, I wanted to follow that naming convention too, but in the initial
review I was asked to use llvm33 instead. Glad to see the guidelines have been
made more explicit. I've changed it back to llvm3.7.

I've just requested the creation of a new git repo.

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352215] Review Request: llvm37 -- Versioned LLVM 3.7

2016-08-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352215



--- Comment #6 from Jason Tibbitts  ---
Sorry, what request do you need to be validated?  Have you requested your
package in the package database?  You can use this ticket as the reference.  I
can process the request, or if John does it first then he'll see this ticket.

However, it is important to note that this package is not correctly named, and
you only get the review exemption if you actually follow the naming guidelines.

The name should be "llvm3.7".  Unless it really is version thirty-seven.  See
https://fedoraproject.org/wiki/Packaging:Naming  The fact that llvm33 and
certainly other packages are doing it wrong isn't a good reason for failing to
do it correctly now.

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352215] Review Request: llvm37 -- Versioned LLVM 3.7

2016-08-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352215



--- Comment #5 from Milan Bouchet-Valat  ---
I still need somebody to support my request in order to get the git repos set
up. :-)

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352215] Review Request: llvm37 -- Versioned LLVM 3.7

2016-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352215



--- Comment #4 from Jason Tibbitts  ---
Yeah, I didn't see the point in reviewing something that had already been
reviewed.  Though I'm sure it could be cleaned up, there is probably utility in
keeping the specs and organization similar between versions.

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352215] Review Request: llvm37 -- Versioned LLVM 3.7

2016-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352215



--- Comment #3 from Milan Bouchet-Valat  ---
Thanks! As per the latest guidelines, it seems that we don't actually have to
follow the standard review process since LLVM is already packaged in Fedora:
https://fedoraproject.org/wiki/Packaging:ReviewGuidelines#Package_Review_Process

I'd still appreciate your comments, and of course I need somebody other than me
to validate the 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 component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352215] Review Request: llvm37 -- Versioned LLVM 3.7

2016-07-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352215

Jens Petersen  changed:

   What|Removed |Added

  Alias||llvm37



-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352215] Review Request: llvm37 -- Versioned LLVM 3.7

2016-07-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352215



--- Comment #2 from Jens Petersen  ---
I am taking a look.

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352215] Review Request: llvm37 -- Versioned LLVM 3.7

2016-07-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352215

Jens Petersen  changed:

   What|Removed |Added

 Blocks||1206852 (ghc-8.0.1)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1206852
[Bug 1206852] ghc-8.0.1 is available
-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352215] Review Request: llvm37 -- Versioned LLVM 3.7

2016-07-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352215

Milan Bouchet-Valat  changed:

   What|Removed |Added

 CC||or...@cora.nwra.com,
   ||peter...@redhat.com
 Blocks||505154 (FE-SCITECH)



--- Comment #1 from Milan Bouchet-Valat  ---
Jens, Orion: Since you've followed the review for llvm33, you're probably in
the best position to review this new one.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=505154
[Bug 505154] Tracker: Review Requests for Science and Technology related
packages
-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org