grep: /var/lib/locales/supported.d/*[^~]: No such file or directory

Bug #44016 reported by Ian Jackson
6
Affects Status Importance Assigned to Milestone
firefox (Ubuntu)
Fix Released
Undecided
Mozilla Bugs

Bug Description

-anarres:~> dapper-firefox&
[4] 28601
-anarres:~> (dapper) DISPLAY=anarres.relativity.greenend.org.uk:0.0 firefox
grep: /var/lib/locales/supported.d/*[^~]: No such file or directory

Revision history for this message
Simon Law (sfllaw) wrote :

Looks like a missing 2>/dev/null at /usr/bin/firefox:284.

Changed in firefox:
status: Unconfirmed → Confirmed
Revision history for this message
era (era) wrote :

Is this fixed already? On my copy the line numbers are way different but here's the only match on /var/lib/locales:

vnix$ egrep -nC2 /var/lib/locales /usr/bin/firefox /dev/null
/usr/bin/firefox-63-if [ "x${MOZ_DISABLE_PANGO}" = x ]; then
/usr/bin/firefox-64- if egrep '^(bn|gu|hi|kn|ml|mr|ne|pa|ta|te)_' \
/usr/bin/firefox:65: /var/lib/locales/supported.d/*[^~] >/dev/null 2>&1; then/usr/bin/firefox-66- MOZ_DISABLE_PANGO=0
/usr/bin/firefox-67- else

Anyway, the proper way to do that is with egrep -qs instead of throwing away error messages competely, isn't it?

Revision history for this message
Alexander Sack (asac) wrote :

so what negative side effects does this bug have, other than just an error message to the console? Gets pango disabled though it should not?

Changed in firefox:
assignee: nobody → mozillateam
importance: Medium → Undecided
Revision history for this message
Alexander Sack (asac) wrote :

we need a proper summary/description in order to process this bug.

Changed in firefox:
status: Confirmed → Needs Info
David Farning (dfarning)
Changed in firefox:
assignee: mozillateam → mozilla-bugs
Revision history for this message
Alexander Sack (asac) wrote :

in what cases, does this fail? What locale do i need to reproduce?

Revision history for this message
era (era) wrote :

It would seem to me that the script needs to take into account the possibility that /var/lib/locales/supported.d/ is empty. Mine contains "en" and "local", of which "en" is indicated as being installed by "language-pack-en-base" (and I guess "local" gets updated dynamically when you install a language pack, maybe ...?)

Unless this case is caused by something more sinister, I would still simply suggest that you add the options "-qs" to the egrep invocation in order to avoid warnings, as well as spurious output when you do get a match; you can then remove the redirections to >/dev/null which are kind of overkill anyway.

Revision history for this message
Alexander Sack (asac) wrote :

we don't use debians start-script in gutsy anymore.

Changed in firefox:
status: Incomplete → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.