[Bug 196591] Review Request: bitlbee

bugzilla at redhat.com bugzilla at redhat.com
Sun Sep 3 23:48:59 UTC 2006


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: bitlbee


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





------- Additional Comments From paul at xtdnet.nl  2006-09-03 19:48 EST -------
I'll happilly take it on for approval. It does not matter much to me who
maintains the package.

I didn't use __install or __rm because that's the FE preferred method. It is the
same reason I use %configure, and not ./configure, despite bitlbee not
supporting all options.  make vs %{__make} I am not sure what the official FE
policy is, I find both being used.

I noticed my package did not properly install the help files. They are not
available. Your package does correctly install the help file. 

I am not sure why you Require: xmlto, as it seems to be this is not used at all.
I cannot find any calls to xmlto/xmlif in the building, and the xmlto package
only provides two binaries, no libraries.

I am not sure I understand your comments on install/make and breaking/not fixing
things.

My spec file also contained the post/postun/preun entries, which you should
copy. It properly adds the bitlbee service so that programs as 'ntsysv' can
enable/disable these (xinetd) services properly. Also add the appropriate
Requires (see my spec file).

As for which user to use. I guess neither daemon nor nobody is supposed to own
files. We could add a user bitlbee to run as instead? This user would not need
Fedora registration, as it is completely irrelevant what uid/gid it would get.

The accept() issue has been reported by me to upstream:
http://bugs.bitlbee.org/bitlbee/ticket/200

If you create a new package, I will do some more testing again.
I noticed my version did not correctly support any of the proxy settings, but I
am not sure if that is a building issue (missing certain libraries?) or whether
this is a generic upstream issue.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list