Bug 5633 - --pidfile option for rsync daemon
Summary: --pidfile option for rsync daemon
Status: RESOLVED FIXED
Alias: None
Product: rsync
Classification: Unclassified
Component: core (show other bugs)
Version: 3.1.0
Hardware: Other Linux
: P3 enhancement (vote)
Target Milestone: ---
Assignee: Wayne Davison
QA Contact: Rsync QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-07-24 18:14 UTC by Arkadiusz Miskiewicz
Modified: 2008-07-28 03:59 UTC (History)
0 users

See Also:


Attachments
The --dparam (-M) daemon option for 3.0.4. (7.96 KB, patch)
2008-07-28 03:27 UTC, Wayne Davison
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Arkadiusz Miskiewicz 2008-07-24 18:14:58 UTC
This is feature request. 

Please add ability to specify different pid file at cmdline time so it will be able to run multiple daemons with different options specified only at cmdline time and the same config file (so no need to duplicate config file just to change some options.

Something like --pidfile /tmp/other.pid
Comment 1 Wayne Davison 2008-07-27 18:37:54 UTC
The latest code in the git repository now has a general option that allows the user to specify config overrides via the command line.  This lets you specify the pidfile like this:

rsync --daemon --dparam pidfile=/tmp/other.pid

or

rsync --daemon -M pidfile=/tmp/other.pid

You can adjust any global or default value using that option, and it will supersede a global/default value in the config file (but will not supersede an explicitly specified module value in a module's definition).
Comment 2 Arkadiusz Miskiewicz 2008-07-28 02:10:27 UTC
Could this be backported to 3.0.x (3.0.4) ?
Comment 3 Wayne Davison 2008-07-28 03:27:09 UTC
Created attachment 3425 [details]
The --dparam (-M) daemon option for 3.0.4.

The 3.0.x branch is only getting bugfixes, so it won't be getting this new feature.  It is easy to turn the recent git check-in into a patch that can be manually applied to 3.0.4pre1, though (it just needed some minor tweaks).
Comment 4 Arkadiusz Miskiewicz 2008-07-28 03:59:50 UTC
Thanks, backport patch works well for me.