[Bug 543566] Review Request: tsocks - Library to allow transparent SOCKS proxying

[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=543566





--- Comment #3 from Jean-Francois Saucier <jfsaucier@xxxxxxxxxxxx>  2009-12-05 15:35:45 EDT ---
Spec URL: http://jfsaucier.fedorapeople.org/packages/tsocks.spec
SRPM URL:
http://jfsaucier.fedorapeople.org/packages/tsocks-1.8-0.3.beta5.fc12.src.rpm


Thanks a lot for taking your time reviewing my package. I tried to solve most
bugs, see below for transcript :

* rpmlint :
rpmlint is silent on the srpm and spec file. I fixed most of the
errors/warnings on the rpm file. Here is the remaining warnings for the rpm
file :

- tsocks.i686: W: no-soname /usr/lib/libtsocks.so.1.8
I checked to solve this one, but I'm not sure how to do it cleanly and what it
affect. Can you enlight me on this?

- tsocks.i686: W: shared-lib-calls-exit /usr/lib/libtsocks.so.1.8
exit@xxxxxxxxx
I could patch the program, but I'm not sure it's worth it.

- tsocks.i686: W: devel-file-in-non-devel-package /usr/lib/libtsocks.so
tsocks need this file and it's not a devel-file.



* I submited scratch build for f11 and f12 on i386, x86_64, ppc and ppc64 and
everything is good :
http://koji.fedoraproject.org/koji/taskinfo?taskID=1851702
http://koji.fedoraproject.org/koji/taskinfo?taskID=1851708


* I patch the Makefile to remove the need of static library linking.


* I think that I fixed my version and release string. I must have misread the
guidelines ;)


* You are right, I changed the group because tsocks is more a library with a
wrapper script than an application.


* Description, man pages, add worthy files, ldconfig and changelog all fixed!

-- 
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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

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