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 663338 - [PATCH] typo in messages
[PATCH] typo in messages
Status: RESOLVED FIXED
Product: gdm
Classification: Core
Component: general
unspecified
Other All
: Normal trivial
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2011-11-03 17:40 UTC by OKANO Takayoshi
Modified: 2011-11-03 18:08 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
daemon: correct XDMCP misspelling (1.23 KB, patch)
2011-11-03 18:08 UTC, Ray Strode [halfline]
committed Details | Review
daemon: remove translation markers from xdmcp warnings (14.16 KB, patch)
2011-11-03 18:08 UTC, Ray Strode [halfline]
committed Details | Review

Description OKANO Takayoshi 2011-11-03 17:40:24 UTC
There are typo "XMDCP" which should be "XDMCP".

patch follows:

diff --git a/daemon/gdm-xdmcp-display-factory.c b/daemon/gdm-xdmcp-display-factory.c
index d5ac32b..37f97e4 100644
--- a/daemon/gdm-xdmcp-display-factory.c
+++ b/daemon/gdm-xdmcp-display-factory.c
@@ -2905,13 +2905,13 @@ decode_packet (GIOChannel             *source,
 
         if G_UNLIKELY (header.version != XDM_PROTOCOL_VERSION &&
                        header.version != GDM_XDMCP_PROTOCOL_VERSION) {
-                g_warning (_("XMDCP: Incorrect XDMCP version!"));
+                g_warning (_("XDMCP: Incorrect XDMCP version!"));
                 return TRUE;
         }
 
         address = gdm_address_new_from_sockaddr ((struct sockaddr *) &clnt_ss, ss_len);
         if (address == NULL) {
-                g_warning (_("XMDCP: Unable to parse address"));
+                g_warning (_("XDMCP: Unable to parse address"));
                 return TRUE;
         }
 
diff --git a/gui/simple-chooser/gdm-host-chooser-widget.c b/gui/simple-chooser/gdm-host-chooser-widget.c
index 3e8a2e2..a1b2751 100644
--- a/gui/simple-chooser/gdm-host-chooser-widget.c
+++ b/gui/simple-chooser/gdm-host-chooser-widget.c
@@ -224,13 +224,13 @@ decode_packet (GIOChannel           *source,
 
         if G_UNLIKELY (header.version != XDM_PROTOCOL_VERSION &&
                        header.version != GDM_XDMCP_PROTOCOL_VERSION) {
-                g_warning (_("XMDCP: Incorrect XDMCP version!"));
+                g_warning (_("XDMCP: Incorrect XDMCP version!"));
                 return TRUE;
         }
 
         address = gdm_address_new_from_sockaddr ((struct sockaddr *) &clnt_ss, ss_len);
         if (address == NULL) {
-                g_warning (_("XMDCP: Unable to parse address"));
+                g_warning (_("XDMCP: Unable to parse address"));
                 return TRUE;
         }
Comment 1 Ray Strode [halfline] 2011-11-03 18:01:28 UTC
warnings should probably in english so they're googable
Comment 2 Ray Strode [halfline] 2011-11-03 18:08:36 UTC
The following fixes have been pushed:
df50577 daemon: correct XDMCP misspelling
7c4b7c1 daemon: remove translation markers from xdmcp warnings
Comment 3 Ray Strode [halfline] 2011-11-03 18:08:38 UTC
Created attachment 200632 [details] [review]
daemon: correct XDMCP misspelling

It's spelled XDMCP not XMDCP
Comment 4 Ray Strode [halfline] 2011-11-03 18:08:41 UTC
Created attachment 200633 [details] [review]
daemon: remove translation markers from xdmcp warnings

They're warnings the user won't see without snooping.
They are hardly intelligible as it is and shouldn't
be translated, so they're easy to google.