Messages in this thread |  | | | From | Oliver Neukum <> | | Subject | Re: [PATCH] driver core: Add the ability to unbind drivers to devices from userspace | | Date | Fri, 5 Aug 2005 20:01:11 +0200 |
| |
Am Freitag, 5. August 2005 15:32 schrieb Jon Smirl: > On 1/1/02, Pavel Machek <pavel@ucw.cz> wrote: > > Hi! > > > > > > > New, simplified version of the sysfs whitespace strip patch... > > > > > > > > Could you tell me why you don't just fail the operation if malformed > > > > input is supplied? > > > > > > Leading/trailing white space should be allowed. For example echo > > > appends '\n' unless you know to use -n. It is easier to fix the kernel > > > than to teach everyone to use -n. > > > > Please, NO! echo -n is the right thing to do, and users will eventually learn. > > We are not going to add such workarounds all over the kernel... > > It is not a work around. These are text attributes meant for human > use. Humans have a hard time cleaning up things they can't see. And > the failure mode for this is awful, your attribute won't set but > everything on the screen looks fine.
The average user has no place poking sysfs. Root should know when to use -n, as should shell scripts.
Regards Oliver - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |