bug#18428: coreutils binary breaks coreutils documentation

[Pádraig Brady]

On 09/08/2014 07:12 PM, Andreas Schwab wrote:
> Bob Proulx <address@hidden> writes:
> 
>> For instance, in the touch(1) man page:
>>
>>   The full documentation for touch is maintained as a Texinfo manual.  If
>>   the info and touch programs are properly installed at  your  site,  the
>>   command
>>
>>          info coreutils 'touch invocation'
>>
>>   should give you access to the complete manual.
>>
>> This is now incorrect (as of 8.23?), because it gives the page:
> 
> "info touch" still works, which is equivalent to "info '(coreutils)touch
> invocation'".
> 
>> Note: Since the coreutils utility doesn't seem to exist in Debian, this
>> section could be removed, but this problem may reappear in the future.
>> So, it's better to use the capital letter C.
> 
> Having info dir entries only differing in case seems like a bad idea.

I agree.

We could rename the node, but the bracketed form works well
without the need for any extra quoting. I.E. this is unambiguous:

  info '(coreutils) stat invocation'

The following simple patch implements that.

thanks,
Pádraig.

diff --git a/src/system.h b/src/system.h
index 162446c..00180cb 100644
--- a/src/system.h
+++ b/src/system.h
@@ -582,7 +582,8 @@ emit_ancillary_info (void)
                 last_component (program_name));
     }
   printf (_("For complete documentation, run: "
-            "info coreutils '%s invocation'\n"), last_component 
(program_name));
+            "info '(coreutils) %s invocation'\n"),
+          last_component (program_name));
 }