summaryrefslogtreecommitdiffstats
path: root/source3/script
diff options
context:
space:
mode:
authorGünther Deschner <gd@samba.org>2010-02-18 11:20:49 +0100
committerGünther Deschner <gd@samba.org>2010-02-18 11:23:59 +0100
commitb6f58e25b5bbce25d4e8825e680afe3986f10e8c (patch)
tree1959c96fc12eceaf751aef9625fb94b24d381a62 /source3/script
parentc42d9c4ec410e205091784cd97cbceb5572609d8 (diff)
downloadsamba-b6f58e25b5bbce25d4e8825e680afe3986f10e8c.tar.gz
samba-b6f58e25b5bbce25d4e8825e680afe3986f10e8c.tar.xz
samba-b6f58e25b5bbce25d4e8825e680afe3986f10e8c.zip
s3-selftest: fix return code for modprinter.pl -a.
Background is: the SetPrinter level 2 calls "addprinter command" an fails if a share already existed (and the addprinter command returned a non-0 return code). Removing the non-0 return code is fine, as in AddPrinter{Ex}, we have checks to see if a share already exists before calling out the addprinter command. Maybe one day, we need to have a "changeprinter command"... Guenther
Diffstat (limited to 'source3/script')
-rwxr-xr-xsource3/script/tests/printing/modprinter.pl3
1 files changed, 2 insertions, 1 deletions
diff --git a/source3/script/tests/printing/modprinter.pl b/source3/script/tests/printing/modprinter.pl
index 406484188da..9e5e3292c6c 100755
--- a/source3/script/tests/printing/modprinter.pl
+++ b/source3/script/tests/printing/modprinter.pl
@@ -95,7 +95,8 @@ while (<CONFIGFILE>) {
if ($section eq $share_name) {
$found_section = 1;
if ($opt_add) {
- die("share $share_name already exists\n");
+ exit 0;
+# die("share $share_name already exists\n");
}
if ($opt_delete) {
$within_section = 1;