[Bug 680936] Review Request: libssh2-python - Python bindings for the libssh2 library

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=680936

--- Comment #10 from Chris Lalancette <clalance@xxxxxxxxxx> 2011-07-08 09:20:31 EDT ---
(In reply to comment #8)
> Chris: 
> 
> Please increment release in your spec/srpm when making changes even if trivial,
> it makes it much easier on the reviewer when installing your SRPM. 
> 
> http://fedoraproject.org/wiki/Packaging:NamingGuidelines#Release_Tag
> 
> In your source URL is there a reason you can't use this as the source URL? 
> https://github.com/wallunit/ssh4py/zipball/0.7.1
> It would still have to be a commented troublesome source url, and it'd screw up
> the macros, but at least it permits upstream source to be verifiable, and less
> work for others. I don't see it as a blocker, but perhaps something to
> consider. 

You are right, I didn't know about the zipball thing.  I've updated the SPEC
now.

(In reply to comment #9)
> A quick other comment, the license field is incorrect: 
> 
> It's currently set as LGPLv2 when it should be LGPLv2+. The source files have
> the 'or (at your option) any later version.' clause in the license declaration 
> of the actual source.

Oh, right.  I've fixed this as well.

A new version of the SPEC and SRPM are available:

http://people.redhat.com/clalance/libssh2-python/libssh2-python-0.7.1-3.fc14.src.rpm
http://people.redhat.com/clalance/libssh2-python/libssh2-python.spec

-- 
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@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]