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 117997 - Please use plural forms in drwright
Please use plural forms in drwright
Status: RESOLVED FIXED
Product: DrWright
Classification: Other
Component: general
master
Other All
: Normal normal
: ---
Assigned To: Richard Hult
Richard Hult
Depends on:
Blocks: 116236
 
 
Reported: 2003-07-21 15:55 UTC by Danilo Segan
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to provide plural forms with ngettext() (1.22 KB, patch)
2003-07-21 15:56 UTC, Danilo Segan
none Details | Review
Proposed patch to require ngettext in gnome-control-center (387 bytes, patch)
2003-10-07 21:05 UTC, Danilo Segan
none Details | Review

Description Danilo Segan 2003-07-21 15:55:31 UTC
Please read Christian's explanation of the problem and solution at
http://developer.gnome.org/doc/tutorials/gnome-i18n/developer.html#plurals

#: src/drwright.c:461
#, c-format
msgid "%d minutes until the next break"

#: src/drwright.c:464
msgid "One minute until the next break"

I'll attach a patch for this below, though it might not be the patch you'd
like ("One minute" will become "1 minute", feel free to change that).

If you don't mind this patch, I can commit this myself.
Comment 1 Danilo Segan 2003-07-21 15:56:49 UTC
Created attachment 18485 [details] [review]
Patch to provide plural forms with ngettext()
Comment 2 Christian Rose 2003-09-30 08:48:35 UTC
Can we have this fixed now for GNOME 2.5?
Comment 3 Danilo Segan 2003-09-30 11:00:26 UTC
I believe DrWright is *superseeded* by "Typing break" for both Gnome
2.4 and Gnome 2.5, and this module is only relevant to Gnome 2.2
installations.

Christian, are you able to change product and component for this bug
to "control-center" and "other capplets"/"keyboard" (I'm not really
sure what category this should go in), because all of this applies to
file gnome-control-center/typing-break/drwright.c instead -- and the
patch could probably be used unmodified.

If it's better to reopen a new bug, I'll do that.
Comment 4 Richard Hult 2003-09-30 13:47:46 UTC
Sounds good, if the control-center is branched for 2.6, I guess it
would be fine to commit.
Comment 5 Danilo Segan 2003-09-30 15:02:45 UTC
Okay, commited to gnome-control-center HEAD CVS.

Marking as resolved and fixed.
Comment 6 Christophe Fergeau 2003-10-07 18:03:55 UTC
configure.in doesn't seem to properly check for ngettext availability.
I thought it was decided that for now, each module using ngettext had
to check for its availability by itself?
Comment 7 Danilo Segan 2003-10-07 21:05:37 UTC
Created attachment 20549 [details] [review]
Proposed patch to require ngettext in gnome-control-center
Comment 8 Danilo Segan 2003-10-07 21:08:29 UTC
Okay, this is a patch that would stop in configure if ngettext is not
available with error message "ngettext is required" (please give a
better suggestion for the message).

Is this good enough at this time?
Comment 9 Richard Hult 2003-10-07 21:13:12 UTC
This is for control-center, not drwright. Maybe a separate bug for
that module is better?
Comment 10 Danilo Segan 2003-10-07 21:37:04 UTC
I believe you're right, this has nothing to do with DrWright itself.

I've filed a bug 124064 for control-center (I've added Christophe to
CC field). Everyone agrees that it's ok to close this bug again?