Date: Tue, 27 Jan 2009 22:09:21 -0600 From: "Jeremy Messenger" <mezz7@cox.net> To: "Victor Popov" <v.a.popov@gmail.com> Cc: ports@freebsd.org Subject: Re: [patch] net-p2p/deluge port improvement - startup scripts + update to 1.1.1 Message-ID: <op.uof8ijrq9aq2h7@localhost> In-Reply-To: <497CA7D7.6020903@gmail.com> References: <497BE96E.7050004@gmail.com> <497CA7D7.6020903@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, 25 Jan 2009 11:56:39 -0600, Victor Popov <v.a.popov@gmail.com>
wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Hello everyone,
>
> Replying to myself, sorry.
>
> Victor Popov wrote:
>
> | Deluge bittorrent client is great, and has a flexible architecture,
> which
> | allows user to run a downloading daemon, only occasionally launching an
> | user interface to manage downloads; all the other time daemon can work
> | unattended.
> | net-p2p/deluge port is missing startup scripts for daemon, and this is
> | what I want to be fixed.
> |
> | The attached patch adds two rc scripts, for launching the daemon,
> deluged,
> | and for launching deluge in webui mode. Ideally, this should be several
> | separate ports, like net-p2p/transmission-*, but deluge is installed by
> | own installer, so dividing port into several would require a lot of
> work.
> |
> | Some comments about why I wrote these startup scripts in this way:
> | - First of all, I don't at all like the idea of running p2p client as
> | usual desktop user for security reasons - desktop users may store
> | sensitive information in their mailboxes, browser configs and other
> | places, so it is better to dedicate different uid for peering task.
> This
> | is why _user and _home configuration variables are introduced.
> | I wonder, should we assign an uid in the UIDs file? What default
> homedir
> | should this user have? I've set default to /home/deluge, and maybe
> there
> | are better places?
> | - Minor bug in launching command-line: it should be
> --logfile=${logfile},
> | not a redirection, but unfortunately it does not work now. Could not
> | investigate and fix it, sorry.
> Fixed in new version and handled in updated patch.
>
> | - run_rc_command is called with environment variable HOME set to
> ${home},
> | because deluged can't determine homedir from user, and it does not
> accept
> | "-c" switch.
> | - Bug in deluge-webui script: when it is time to stop deluge, script
> has
> | to find process due to lack of pidfile. If at that time you are running
> | deluge with gtk UI, script will find your instance, and will try to
> kill
> | it. If deluge_webui_user is not you, it's ok, your deluge is safe, but
> | after that script will not be able to start webui, because it would
> think
> | it is already running. This is because deluge program does not accept
> | "--pidfile" parameter. There is already a feature-request in deluge
> | bug-tracker for adding pidfile
> [http://dev.deluge-torrent.org/ticket/758],
> | so maybe soon it would be possible to improve this script.
> |
> |
> | Can this patch be commited, please? If something is wrong, comments
> would
> | be very appreciated.
>
> I've updated my patch to deliver new version of deluge - 1.1.1. In
> addition to portversion bump and distfile changes,
I have committed 1.1.1, thanks for patch. But I didn't commit the startup
script yet. It's still undone, because it still need
pkg-install/pkg-deinstall and probably a few more. Take a look at
sysutils/hal or/and x11/gdm for example how to do with add user/group. I
have never work with like that, so I will need time to learn about it that
is more like weekend. Unless you want to do it, be my guest. :-)
> I've changed
> MASTER_SITES to "official" download site, although it has the same IP
> now,
> it can change in future. Also, some plist fixes related to new version.
I didn't commit these. The plist part of %%PORTVERSION%% is not need
because I usually create a new plist at the every version anyway.
Cheers,
Mezz
> - --
> Best regards, Victor Popov mailto:v.a.popov@gmail.com
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v2.0.10 (FreeBSD)
>
> iEYEARECAAYFAkl8p8oACgkQtKisLOtMPvlKGwCguSKaE0BwjqqkPpAexYo+UsaS
> emAAoNb+LHfj45BNbU9bAljm1x8WwAev
> =+WiO
> -----END PGP SIGNATURE-----
--
mezz7@cox.net - mezz@FreeBSD.org
FreeBSD GNOME Team
http://www.FreeBSD.org/gnome/ - gnome@FreeBSD.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?op.uof8ijrq9aq2h7>