Bug 725570 - Review Request: gcc-python-plugin - GCC plugin that embeds Python
Summary: Review Request: gcc-python-plugin - GCC plugin that embeds Python
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tom "spot" Callaway
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-07-25 21:10 UTC by Dave Malcolm
Modified: 2011-07-26 16:10 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-07-26 16:10:49 UTC
Type: ---
tcallawa: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Dave Malcolm 2011-07-25 21:10:43 UTC
Spec URL:
http://fedorapeople.org/~dmalcolm/python-packaging/gcc-python-plugin.spec

SRPM URL: 
http://fedorapeople.org/~dmalcolm/python-packaging/gcc-python-plugin-0.3-1.fc15.src.rpm

Description:
Plugins for embedding various versions of Python within GCC

(it's actually built against all 4 versions of python we ship in Fedora, giving 4 gcc plugins, as 4 sub-packages)

rpmlint output on local build is clean:
$ rpmlint ~/rpmbuild/RPMS/x86_64/*-0.3-1* gcc-python-plugin.spec 
6 packages and 1 specfiles checked; 0 errors, 0 warnings.

Successful scratch build into dist-f16 here:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3228670

Comment 1 Dave Malcolm 2011-07-25 21:11:54 UTC
This relates to this Fedora 16 feature:
  http://fedoraproject.org/wiki/Features/GccPythonPlugin

Comment 2 Tom "spot" Callaway 2011-07-26 14:25:53 UTC
There are some minor items which are not necessary unless you're building for EPEL, specifically:

* BuildRoot
* rm -rf $RPM_BUILD_ROOT at the beginning of %install
* %clean section that simply does rm -rf $RPM_BUILD_ROOT
* %defattr(-,root,root,-) lines in %files sections

All of these are the implicit defaults now (except in EL-4 and EL-5). Consider removing them before import if you do not plan on supporting those targets.

== Review ==

Good:

- rpmlint checks return clean
- package meets naming guidelines
- package meets packaging guidelines
- license (GPLv3+) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream bfc888fc457b3f1b5a6442393eb157a22080059a2f90dff0564a62f1fdd43868
- package compiles on devel (koji)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- macro use consistent
- code, not content
- docs subpackage okay
- nothing in %doc affects runtime
- no need for .desktop file 

APPROVED.

Comment 3 Dave Malcolm 2011-07-26 15:24:36 UTC
New Package SCM Request
=======================
Package Name: gcc-python-plugin
Short Description: Plugins for embedding various versions of Python within GCC
Owners: dmalcolm
Branches: 
InitialCC:

Comment 4 Gwyn Ciesla 2011-07-26 15:35:00 UTC
Git done (by process-git-requests).

Spot, you didn't take ownership of the BZ. <wristslap>

Comment 5 Dave Malcolm 2011-07-26 16:10:49 UTC
(In reply to comment #4)
> Git done (by process-git-requests).
Thanks!

> Spot, you didn't take ownership of the BZ. <wristslap
Setting ownership of this bug to Spot.

Thanks for the review.

Imported as:
http://pkgs.fedoraproject.org/gitweb/?p=gcc-python-plugin.git;a=commitdiff;h=0c8191f4f62f3736da8b1f8e27aabec1a6354968

and I've successfully built gcc-python-plugin-0.3-1.fc16 into rawhide (F16 still, it seems) as:
  http://koji.fedoraproject.org/koji/taskinfo?taskID=3231737

Closing NEXTRELEASE.


Note You need to log in before you can comment on or make changes to this bug.