43e2606efa67a3e0d688df05ab7718f1635a5e60
[gentoo.git] /
1 From 601e1362d63293ca567295f63cb52bcfeb5d9959 Mon Sep 17 00:00:00 2001
2 From: Mike Frysinger <vapier@chromium.org>
3 Date: Tue, 5 May 2015 23:19:57 -0400
4 Subject: [PATCH 1/3] pwclient: handle missing ~/.pwclientrc better
5
6 The upgrade path will crash if ~/.pwclientrc doesn't exist, so add an
7 explicit check for the file before we try to "upgrade" it.
8
9 The default error messages don't mention the config file, so it can be
10 a bit confusing what pwclient is complaining about when running.
11
12 Signed-off-by: Mike Frysinger <vapier@chromium.org>
13 ---
14  apps/patchwork/bin/pwclient | 6 +++---
15  1 file changed, 3 insertions(+), 3 deletions(-)
16
17 diff --git a/apps/patchwork/bin/pwclient b/apps/patchwork/bin/pwclient
18 index 2a80981..56aa909 100755
19 --- a/apps/patchwork/bin/pwclient
20 +++ b/apps/patchwork/bin/pwclient
21 @@ -552,7 +552,7 @@ def main():
22      config = ConfigParser.ConfigParser()
23      config.read([CONFIG_FILE])
24  
25 -    if not config.has_section('options'):
26 +    if not config.has_section('options') and os.path.exists(CONFIG_FILE):
27          sys.stderr.write('~/.pwclientrc is in the old format. Migrating it...')
28  
29          old_project = config.get('base','project')
30 @@ -590,10 +590,10 @@ def main():
31              sys.exit(1)
32  
33      if not config.has_section(project_str):
34 -        sys.stderr.write("No section for project %s\n" % project_str)
35 +        sys.stderr.write('No section for project %s in ~/.pwclientrc\n' % project_str)
36          sys.exit(1)
37      if not config.has_option(project_str, 'url'):
38 -        sys.stderr.write("No URL for project %s\n" % project_str)
39 +        sys.stderr.write('No URL for project %s in ~/.pwclientrc\n' % project_str)
40          sys.exit(1)
41      if not do_signoff and config.has_option('options', 'signoff'):
42          do_signoff = config.getboolean('options', 'signoff')
43 -- 
44 2.4.0
45