From: justdave%syndicomm.com <>
Date: Mon, 11 Jun 2001 00:15:56 +0000 (+0000)
Subject: Re-fix for bug 21253: fix calling conventions for syncshadowdb so that a phony parame... 
X-Git-Tag: bugzilla-2.14~89
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=808408770704c74994a322632c334d248dbc189b;p=thirdparty%2Fbugzilla.git

Re-fix for bug 21253: fix calling conventions for syncshadowdb so that a phony parameter to get past the multi-param system() test in tinderbox actually works.
r= tara
---

diff --git a/globals.pl b/globals.pl
index 0f91aeb9ec..2b3456959f 100644
--- a/globals.pl
+++ b/globals.pl
@@ -117,10 +117,11 @@ sub SyncAnyPendingShadowChanges {
                 return;
             } elsif (defined $pid) {
                 # child process code runs here
-                exec("./syncshadowdb",[]) or die "Unable to exec syncshadowdb: $!";
-                # passing the empty list as a second parameter tricks it into
-                # using execvp instead of running a shell, but still doesn't
-                # pass any parameters to syncshadowdb
+                exec("./syncshadowdb","--") or die "Unable to exec syncshadowdb: $!";
+                # the idea was that passing the second parameter tricks it into
+                # using execvp instead of running a shell. Not really necessary since
+                # there are no shell meta-characters, but it passes our tinderbox
+                # test that way. :) http://bugzilla.mozilla.org/show_bug.cgi?id=21253
             } elsif ($! =~ /No more process/) {
                 # recoverable fork error, try again in 5 seconds
                 sleep 5;
diff --git a/syncshadowdb b/syncshadowdb
index 7dce870380..471b6d6ad1 100755
--- a/syncshadowdb
+++ b/syncshadowdb
@@ -51,6 +51,9 @@ foreach my $opt (@ARGV) {
     } elsif ($opt eq '-syncall') {
         $syncall = 1;
         $verbose = 1;
+    } elsif ($opt eq '--') {
+        # do nothing - null parameter so we can use
+        # multi-param system() call in globals.pl
     } else {
         Usage();
     }