After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 124064 - Missing check for ngettext in gnome-control-center
Missing check for ngettext in gnome-control-center
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: general
git master
Other other
: High normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-10-07 21:33 UTC by Danilo Segan
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch to check for ngettext in gnome-control-center/configure.in (387 bytes, patch)
2003-10-07 21:36 UTC, Danilo Segan
none Details | Review

Description Danilo Segan 2003-10-07 21:33:10 UTC
As Christophe Fergeau pointed out in bug 117997 (concerning DrWright and
ngettext usage), that change requires adding a check for ngettext to
gnome-control-center/configure.in

I'll attach a proposed patch that will halt if ngettext is not present.
(My autoconf/automake skills are limited, so be sure to review the patch: I
did test it, and it seems to work)

The error message configure would output is "ngettext is required" (please
give a better suggestion for the message).
Comment 1 Danilo Segan 2003-10-07 21:36:17 UTC
Created attachment 20550 [details] [review]
Proposed patch to check for ngettext in gnome-control-center/configure.in
Comment 2 Christophe Fergeau 2003-10-07 21:43:20 UTC
The error message should explain people how and where they can get
this "ngettext" (or at least give them a pointer on some web resource
where they can get more information about the issue)
Comment 3 Danilo Segan 2003-10-08 09:05:12 UTC
Ok, how about: "ngettext is required -- it's included with GNU gettext
(available from http://www.gnu.org/software/gettext/)"

Also, I've read in discussions regarding ngettext that it is available
on Solaris 9, and that there's a patch from Sun for Solaris 8. Can
anyone confirm that? If so, should a note about the patch be included
in the message too?
Comment 4 prakesh 2003-10-22 06:03:02 UTC
Added L10N keyword for easier searching.
Comment 5 Jody Goldberg 2004-02-17 19:48:12 UTC
Given that control-center pulls in gnome-vfs which already depends
ngettext this test is not terribly important in my mind.  However,
just to be thoroughly anal I've added the same test as gnome-vfs.