Skip to content

Commit 8808286

Browse files
committed
check if we have required parameters for install-printerdriver (Fedora issue #1754204)
1 parent 10015a7 commit 8808286

2 files changed

Lines changed: 13 additions & 2 deletions

File tree

NEWS

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
- remove deprecated SIGNAL_RUN_LAST (pull/134)
2121
- use remote ppd for CUPS shared queues (pull/137)
2222
- get rid of warnings in applet (Fedora issue #1732890)
23+
- check if we have required parameters in install-printerdriver (Fedora issue #1754204)
2324

2425
1.5.10 changes
2526
--------------

install-printerdriver.py

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,18 @@ def progress(progress, type, user_data):
1717

1818
set_debugging (True)
1919

20-
package = sys.argv[1]
21-
repo = sys.argv[2]
20+
try:
21+
package = sys.argv[1]
22+
except:
23+
debugprint("Missing package name to install.")
24+
sys.exit(1)
25+
26+
try:
27+
repo = sys.argv[2]
28+
except:
29+
debugprint("Missing name of repo.")
30+
sys.exit(1)
31+
2232
try:
2333
repo_gpg_id = sys.argv[3]
2434
except:

0 commit comments

Comments
 (0)