On 02/24/2013 11:09 AM, Pádraig Brady wrote: > On 02/24/2013 05:39 PM, Sami Kerola wrote: >> Hello, >> >> When using file type indicator to a root, e.g., >> >> $ ls -Fd / >> // >> >> two slashes are printed. To me that feels wrong. Patch below is an >> attempt to make root print out to be a single slash. Perhaps there is >> more elegant way to do the same. Or maybe I am mistaking, and double >> slash is correct, although I could not find earlier posting about >> that. > > I agree it looks a bit awkward, but I think it's more consistent > to keep the trailing /. Consider a script that filters on and > strips a trailing /
I kind of like the idea of omitting the trailing slash for roots (note the plural); since '/' and '//' are not necessarily the same root, and listing '//' for '/' or '///' for '//' is confusing. However, since the proposed patch does not special case '//', it is not an appropriate patch in its current form. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
