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 317479 - Comma splices
Comma splices
Status: RESOLVED FIXED
Product: gdm
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2005-09-29 01:37 UTC by Adam Weinberger
Modified: 2005-09-29 21:08 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Adam Weinberger 2005-09-29 01:37:06 UTC
A new set of strings with comma splices has been introduced. Any time that you combine two distinct 
sentences with a comma, it is a grammar error.

#: gui/gdmflexiserver.c:697
msgid "Start new flexible session, do not show popup"

These are two distinct sentences. I'd replace the comma with a semicolon.
"Start new flexible session; do not show popup"

#: gui/greeter/greeter_item_ulist.c:196
msgid "Theme broken, must have pam-message label!"

These are two distinct sentences. I'd replace the comma with a colon.
"Theme broken: must have pam-message label."
Comment 1 Brian Cameron 2005-09-29 21:01:54 UTC
Thanks, fixed in CVS head and 2.12 branch.  It's okay fixing these strings in the
2.12 branch since these are new messages added after the official 2.12 release.
I was agreeable to adding them since the first one adds the --startnew feature
which is intended to be used only by the screensaver to support starting a new
login from the screensaver, and the second is a syslog message that reports a
useful error.