[Bug 175433] Review Request: tor - Anonymizing overlay network for TCP (The onion router)

[bugzilla at redhat.com]

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: tor - Anonymizing overlay network for TCP (The onion router)


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





------- Additional Comments From paul at xtdnet.nl  2006-01-08 15:14 EST -------
You are proposing a package for FE, not "my devel box". There are also no
provisions for Suse or Mandrake type startup sequences. IMHO, splitting tor in
tor-lsb is just going to confuse users who are running this on FE, and not some
devel box who want to install "tor" and not "tor-lsb". This is not "bloat" on FE
systems, for which you are submitting a package. 

other comments:

cat <<EOF >>src/config/torrc.sample.in

I personally dislike creating files from the spec file. They should be included
as  a SOURCE file. (some of these constucts fail in a non-interactive shell as well)

tor is a very common name and often already exists on Scandinavian boxes. This
will cause tor to fail during install. This is why my tor rpm/spec uses _tor
instead of tor as user/group. Alternative torproxy or tordaemon could be used.

tor uses libevent, but (wrongly) staticly links this. It should not do this and
depend on the libevent package, which is part of FE.

SOURCE1 is defined but not used. It should be removed.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.