From: Bruno Haible <bruno@clisp.org>
Date: Tue, 22 Jan 2002 20:42:10 +0000 (+0000)
Subject: Special hack for SunOS4 grep.
X-Git-Tag: v0.11~31
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c95830650cd97b85e5797d415f6a93c6df12502f;p=thirdparty%2Fgettext.git

Special hack for SunOS4 grep.
---

diff --git a/src/ChangeLog b/src/ChangeLog
index c5c5f9cee..0e0ddaf71 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,7 @@
+2002-01-21  Bruno Haible  <bruno@clisp.org>
+
+	* msggrep.c (main): On SunOS4, don't pass option "-q" to grep.
+
 2002-01-19  Bruno Haible  <bruno@clisp.org>
 
 	* msgfmt.c (check_header_entry): Emit error messages including the
diff --git a/src/msggrep.c b/src/msggrep.c
index 661269ff2..ad338b8ee 100644
--- a/src/msggrep.c
+++ b/src/msggrep.c
@@ -134,7 +134,6 @@ main (argc, argv)
   msgdomain_list_ty *result;
   bool sort_by_filepos = false;
   bool sort_by_msgid = false;
-  size_t i;
 
   /* Set program name for messages.  */
   set_program_name (argv[0]);
@@ -328,14 +327,28 @@ warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.\n\
     if (grep_args[grep_pass]->nitems > 0)
       {
 	string_list_ty *args = grep_args[grep_pass];
+	size_t option_q;
+	size_t i, j;
+
+	/* We use "grep -q" because it is slightly more efficient than
+	   "grep".  We pipe grep's output to /dev/null anyway.  But
+	   SunOS4's grep program doesn't understand the -q option.  */
+#if (defined (sun) || defined (__sun)) && !defined (__SVR4)
+	option_q = 0;
+#else
+	option_q = 1;
+#endif
 
 	grep_argv[grep_pass] =
-	  (char **) xmalloc ((2 + args->nitems + 1) * sizeof (char *));
+	  (char **) xmalloc ((1 + option_q + args->nitems + 1)
+			     * sizeof (char *));
 	grep_argv[grep_pass][0] = (char *) grep_path;
-	grep_argv[grep_pass][1] = "-q";
-	for (i = 2; i <= args->nitems + 1; i++)
-	  grep_argv[grep_pass][i] = (char *) args->item[i - 2];
-	grep_argv[grep_pass][i] = NULL;
+	j = 1;
+	if (option_q)
+	  grep_argv[grep_pass][j++] = "-q";
+	for (i = 0; i < args->nitems; i++)
+	  grep_argv[grep_pass][j++] = (char *) args->item[i];
+	grep_argv[grep_pass][j] = NULL;
       }
 
   /* Select the messages.  */