On Wed, May 29, 2013 at 01:11:48PM -0700, Linda Walsh wrote: > > @@ -98,6 +98,7 @@ Preserves the whole environment, ie does not set > > .B USER > > nor > > .BR LOGNAME . > > +The option is ignored if the option \fB\-\-login\fR is specified. > --- > There are 2 problems > On the surface, this patch, alone, appears to remove desirable > functionality from su, and, it is a dangerous change in default Did you read the code? if (simulate_login) { ... } else { if (change_environment) { .... } } the patch does not change anything... Karel -- Karel Zak <kzak@xxxxxxxxxx> http://karelzak.blogspot.com -- To unsubscribe from this list: send the line "unsubscribe util-linux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html