#1473: Adding support for switching uid after startup for daemons
-------------------------+--------------------------------------------------
 Reporter:  cinderblock  |       Owner:       
     Type:  patch        |      Status:  new  
 Priority:  minor        |   Milestone:  1.3.2
Component:  core         |     Version:  1.3.1
 Keywords:               |  
-------------------------+--------------------------------------------------

Comment(by damoxc):

 Yup that'll be absolutely fine. Python won't execute the code inside the
 if when it's running on windows, and it will be as if that if wasn't there
 on other platforms.

 One mistake in the deluged-setuid.patch, true should be True, true will
 throw a name error in Python.

 Also it would be quite nice if the options were only added when not
 running on windows too, since otherwise there are pointless options added
 to the output, so wrapping those in an if deluge.common.windows_check()
 would be excellent.

-- 
Ticket URL: <http://dev.deluge-torrent.org/ticket/1473#comment:4>
Deluge <http://deluge-torrent.org/>
Deluge project

-- 
You received this message because you are subscribed to the Google Groups 
"Deluge Dev" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/deluge-dev?hl=en.

Reply via email to