Project

General

Profile

Bug #508

od doesn't set text domain

Added by Garrett D'Amore almost 10 years ago. Updated almost 10 years ago.

Status:
Resolved
Priority:
Normal
Category:
cmd - userland programs
Start date:
2010-12-12
Due date:
% Done:

100%

Estimated time:
Difficulty:
Tags:
Gerrit CR:

Description

The "od(1)" utility needs to set the text domain by calling "textdomain()".

#1

Updated by Garrett D'Amore almost 10 years ago

Fix is nearly trivial.

diff -r f1dd13f1ef7d usr/src/cmd/od/od.c
--- a/usr/src/cmd/od/od.c    Sun Dec 05 19:54:51 2010 -0800
+++ b/usr/src/cmd/od/od.c    Sun Dec 12 21:03:28 2010 -0800
@@ -39,6 +39,11 @@

 #define    _(x)    gettext(x)

+
+#ifndef TEXT_DOMAIN
+#define    TEXT_DOMAIN    "SYS_TEST" 
+#endif
+
 /* address format */
 static char *afmt  =    "%07llo";
 static char *cfmt  =    "       ";
@@ -551,6 +556,7 @@
     input = stdin;

     (void) setlocale(LC_ALL, "");
+    (void) textdomain(TEXT_DOMAIN);

     while ((c = getopt(argc, argv, "A:bCcdDfFj:N:oOsSxXvt:")) != EOF) {
         switch (c) {
#2

Updated by Garrett D'Amore almost 10 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Resolved last night with

changeset: 13257:d3807abc6720
user: Garrett D'Amore <>
date: Wed Dec 15 17:12:35 2010 -0800
description:
508 od doesn't set text domain
509 need an open source sed(1) replacement
516 libc regex misses \< and \> word delimiters
Reviewed by:
Reviewed by:
Approved by:

Also available in: Atom PDF