Bill Allombert wrote:

>> The attached patch should fix this.
> 
> Not that it matter much, but your patch is not against the upstream version:
> sudo is the default instead of su.
> I do not see why ubuntu make that change since they could just add
> a file /etc/su-to-rootrc with
> SU_TO_ROOT_SU=sudo
> to achieve the same effect without patching su-to-root and make it 
> interface-incompatible with upstream.

Sorry, it was in fact me, who did this change yesterday and I've not
taken care enough to not include it in the patch.

I've though about using the conffile for this, but when I've noticed
that there isn't shipped one by default, I've thought it would be better
to change it in the source directly.

Apparently I've been wrong, so I'll change it to use another default in
/etc/su-to-rootrc instead (and note that instead in the man page).

Thank you for pointing this out.


Do you think the patch is correct otherwise? I'd like to include the fix
for this in the next upload then, too.


Cheers.

-- 
http://daniel.hahler.de/



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to