Re: [Bug 294348] Re: ubuntu directing users to coreutils mailing list for general problems

[Jim Meyering]

Eric Blake <address@hidden> wrote:

> According to Jim Meyering on 11/8/2008 11:51 AM:
>> -  /* TRANSLATORS: The placeholder indicates the bug-reporting address
>> -     for this package.  Please add _another line_ saying
>> -     "Report translation bugs to <...>\n" with the address for translation
>> +  /* TRANSLATORS: The second placeholder indicates the bug-reporting
>> +     address for this package.  Please add _another line_ saying
>> +     "Report %s translation bugs to <...>\n" with the address for 
>> translation
>>       bugs (typically your translation team's web or email address).  */
>> -  printf (_("\nReport bugs to <%s>.\n"), PACKAGE_BUGREPORT);
>> +  printf (_("\nReport %s bugs to <%s>.\n"), last_component (program_name),
>> +      PACKAGE_BUGREPORT);
>
> And where would the second line's %s be filled in from?  You would need to
> add another last_component call as an additional printf argument before
> you could suggest that.
>
> Doesn't it just suffice to have the text read:
>
> Report ls bugs to <address@hidden>.
> Report translation bugs to <...>.

Good catch.
I started to write the first patch below,
but then saw that very few translators have heeded the instructions.

diff --git a/src/system.h b/src/system.h
index 8577d44..e839d7e 100644
--- a/src/system.h
+++ b/src/system.h
@@ -614,10 +614,13 @@ ptr_align (void const *ptr, size_t alignment)
 static inline void
 emit_bug_reporting_address (void)
 {
-  /* TRANSLATORS: The second placeholder indicates the bug-reporting
-     address for this package.  Please add _another line_ saying
-     "Report %s translation bugs to <...>\n" with the address for translation
-     bugs (typically your translation team's web or email address).  */
+  /* TRANSLATORS: The second placeholder indicates the bug-reporting address
+     for this package.  Please append a line saying "Report translation bugs
+     to <...>\n" with the address for translation bugs (typically your
+     translation team's web or email address).  E.g., in French,
+     ls --help will print the translation of these two lines:
+     Report ls bugs to <address@hidden>.
+     Report translation bugs to <http://translationproject.org/team/af.html>.
   printf (_("\nReport %s bugs to <%s>.\n"), last_component (program_name),
          PACKAGE_BUGREPORT);
 }



So here's a slightly more invasive change:

    Print the "Report translation bugs to <...>." diagnostic
    for all but the C/POSIX locale.

Though I'm not sure I want to print such a line in an English locale.
In spite of that, I have a slight preference for this approach.
Note: if I go that route, then I'll have to remove all other
hard-locale.h inclusions to get past make syntax-check.

Opinions?

diff --git a/src/system.h b/src/system.h
index ee0d25c..6e4be48 100644
--- a/src/system.h
+++ b/src/system.h
@@ -611,15 +611,19 @@ ptr_align (void const *ptr, size_t alignment)
     ? false : (((Accum) = (Accum) * 10 + (Digit_val)), true))          \
   )

+#include "hard-locale.h"
 static inline void
 emit_bug_reporting_address (void)
 {
-  /* TRANSLATORS: The second placeholder indicates the bug-reporting
-     address for this package.  Please add _another line_ saying
-     "Report %s translation bugs to <...>\n" with the address for translation
-     bugs (typically your translation team's web or email address).  */
   printf (_("\nReport %s bugs to <%s>.\n"), last_component (program_name),
          PACKAGE_BUGREPORT);
+  /* TRANSLATORS: Replace LANG_CODE in this URL with your language code
+     <http://translationproject.org/team/LANG_CODE.html> to form one of
+     the URLs at http://translationproject.org/team/.  Otherwise, replace
+     the entire URL with your team's email address.  */
+  if (hard_locale (LC_MESSAGES))
+    fputs (_("Report translation bugs to "
+            "<http://translationproject.org/team/>\n"), stdout);
 }

 #include "inttostr.h"