[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2012-10-18 Thread bugzilla 
https://bugzilla.redhat.com/show_bug.cgi?id=728662

Ankur Sinha (FranciscoD) sanjay.an...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2012-10-18 19:03:21

--- Comment #11 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
Package in repos now. Closing bug.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-10 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

--- Comment #6 from Praveen Kumar kumarpraveen.nit...@gmail.com 2011-08-10 
08:22:58 EDT ---
Ah sorry I have only 32 bit system so didn't see it but now fixed.
Thanks for suggestion 
Spec URL: http://kumarpraveen.fedorapeople.org/pylibmc/python-pylibmc.spec
SRPM URL:
http://kumarpraveen.fedorapeople.org/pylibmc/python-pylibmc-1.2.0-3.20110805gitf01c31.fc15.src.rpm

koji build : http://koji.fedoraproject.org/koji/taskinfo?taskID=3263828

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-10 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

Ankur Sinha sanjay.an...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Ankur Sinha sanjay.an...@gmail.com 2011-08-10 09:00:12 
EDT ---
Hello,

Issues are fixed. This package is XXX APPROVED XXX

Please build it and we can jump to the django-pylibmc review :D

Thanks,
Ankur

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-10 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

--- Comment #8 from Praveen Kumar kumarpraveen.nit...@gmail.com 2011-08-10 
09:16:24 EDT ---
New Package SCM Request
===
Package Name: python-pylibmc
Short Description: Memcached client for Python
Owners: sundaram pjp kumarpraveen
Branches: el6

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-10 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

--- Comment #9 from Praveen Kumar kumarpraveen.nit...@gmail.com 2011-08-10 
09:19:41 EDT ---
New Package SCM Request
===
Package Name: python-pylibmc
Short Description: Memcached client for Python
Owners: sundaram pjp kumarpraveen
Branches: f14 f15 el6

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-10 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

Praveen Kumar kumarpraveen.nit...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-10 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

--- Comment #10 from Jon Ciesla l...@jcomserv.net 2011-08-10 11:13:51 EDT ---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-09 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

Rahul Sundaram methe...@gmail.com changed:

   What|Removed |Added

 CC||methe...@gmail.com

--- Comment #4 from Rahul Sundaram methe...@gmail.com 2011-08-09 03:34:05 EDT 
---
Just a quick note

https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Addon_Packages_.28python_modules.29

There is an exception to this rule. If the upstream source has py (or Py)
in its name, you can use that name for the package. So, for example, pygtk is
acceptable

So this package can just be called pylibmc.  It doesn't recommend that naming. 
Hence I am fine with it, either way

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-09 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

--- Comment #5 from Ankur Sinha sanjay.an...@gmail.com 2011-08-09 13:24:39 
EDT ---
Praveen,

Please correct the permissions for the soname:

From rpmlint:
python-pylibmc.x86_64: E: non-standard-executable-perm
/usr/lib64/python2.7/site-packages/_pylibmc.so 0775L

Thanks,
Ankur

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-08 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

--- Comment #3 from Praveen Kumar kumarpraveen.nit...@gmail.com 2011-08-08 
21:08:47 EDT ---
Thanks for suggestion 
Spec URL: http://kumarpraveen.fedorapeople.org/pylibmc/python-pylibmc.spec
SRPM URL:
http://kumarpraveen.fedorapeople.org/pylibmc/python-pylibmc-1.2.0-2.20110805gitf01c31.fc15.src.rpm

koji Build : http://koji.fedoraproject.org/koji/taskinfo?taskID=3260715

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-07 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

Ankur Sinha sanjay.an...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-07 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

Ankur Sinha sanjay.an...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||sanjay.an...@gmail.com
 AssignedTo|nob...@fedoraproject.org|sanjay.an...@gmail.com
   Flag|fedora-review?  |

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-07 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

--- Comment #1 from Ankur Sinha sanjay.an...@gmail.com 2011-08-07 13:49:01 
EDT ---
Review:

+ OK
- NA
? ISSUE

+ Package meets naming and packaging guidelines
+ Spec file matches base package name.
+ Spec has consistant macro usage.
+ Meets Packaging Guidelines.
+ License
+ License field in spec matches
+ License file included in package
+ Spec in American English
+ Spec is legible.
+ Sources match upstream md5sum:
^^ Git check out, checked using diff.

- Package needs ExcludeArch
+ BuildRequires correct
- Spec handles locales/find_lang
- Package is relocatable and has a reason to be.
+ Package is code or permissible content.
- Doc subpackage needed/used.
+ Packages %doc files don't affect runtime.

- Headers/static libs in -devel subpackage.
- Spec has needed ldconfig in post and postun
- .pc files in -devel subpackage/requires pkgconfig
- .so files in -devel subpackage.
- -devel package Requires: %{name} = %{version}-%{release}
- .la files are removed.

- Package is a GUI app and has a .desktop file

+ Package compiles and builds on at least one arch.
+ Package has no duplicate files in %files.
+ Package doesn't own any directories other packages own.
+ Package owns all the directories it creates.
+ No rpmlint output.
^^
[ankur@ankur SRPMS]$ rpmlint /var/lib/mock/fedora-rawhide-i386/result/*.rpm
../SPECS/python-pylibmc.spec
python-pylibmc-1.2.0-1.20110805gitf01c31.fc15.src.rpm 
python-pylibmc.noarch: W: spelling-error Summary(en_US) Memcached - Schemed
python-pylibmc.noarch: W: spelling-error %description -l en_US memcached -
schemed
python-pylibmc.noarch: W: spelling-error %description -l en_US libmemcached -
semidetached
python-pylibmc.src: W: spelling-error Summary(en_US) Memcached - Schemed
python-pylibmc.src: W: spelling-error %description -l en_US memcached -
schemed
python-pylibmc.src: W: spelling-error %description -l en_US libmemcached -
semidetached
python-pylibmc.src: W: invalid-url Source0:
python-pylibmc-1.2.0.20110805gitf01c31.tar.xz
../SPECS/python-pylibmc.spec: W: invalid-url Source0:
python-pylibmc-1.2.0.20110805gitf01c31.tar.xz
python-pylibmc.src: W: spelling-error Summary(en_US) Memcached - Schemed
python-pylibmc.src: W: spelling-error %description -l en_US memcached -
schemed
python-pylibmc.src: W: spelling-error %description -l en_US libmemcached -
semidetached
python-pylibmc.src: W: invalid-url Source0:
python-pylibmc-1.2.0.20110805gitf01c31.tar.xz
3 packages and 1 specfiles checked; 0 errors, 12 warnings.


MINOR SPELLING ERRORS ONLY. OK.

+ final provides and requires are sane:
[ankur@ankur result]$ review-req-check 
== python-pylibmc-1.2.0-1.20110805gitf01c31.fc17.noarch.rpm ==
Provides:
python-pylibmc = 1.2.0-1.20110805gitf01c31.fc17

Requires:
python(abi) = 2.7

== python-pylibmc-1.2.0-1.20110805gitf01c31.fc17.src.rpm ==
Provides:

Requires:
python-devel  
python-setuptools  
libmemcached-devel  
zlib-devel  


SHOULD Items:

+ Should build in mock.
+ Should build on all supported archs
- Should function as described.
- Should have sane scriptlets.
- Should have subpackages require base package with fully versioned depend.
+ Should have dist tag
+ Should package latest version
- check for outstanding bugs on package. (For core merge reviews)

Issues:

1. It looks like a good package. I see you've removed some sonames. Why is
this? Please clarify and I shall approve the package :D

Thanks,
Ankur

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-07 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

--- Comment #2 from Ankur Sinha sanjay.an...@gmail.com 2011-08-07 13:54:24 
EDT ---
One suggestion though: In the setup macro, you needn't use such a complex name.
You're creating the directory to extract into, even just %{name} will suffice.
Only a cosmetic suggestion.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-07 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

Ankur Sinha sanjay.an...@gmail.com changed:

   What|Removed |Added

 Blocks||728664
 Depends on|728664  |

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-06 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

Praveen Kumar kumarpraveen.nit...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-06 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=728662

Praveen Kumar kumarpraveen.nit...@gmail.com changed:

   What|Removed |Added

 Depends on||728664

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review