I just installed the latest release of gupload.py and it fails with the
following error:
[frank@cnc ~]$ gupload.py -v 1 pwd/2015-02-27_06-24-38_4_10.fit
Traceback (most recent call last):
File "/usr/bin/gupload.py", line 399, in <module>
g = gupload(myargs)
File "/usr/bin/gupload.py", line 95, in init
elif os.path.isfile(self.configCurrentDir):
AttributeError: gupload instance has no attribute 'configCurrentDir'
I did a lot of rerarranging from the prior version of this file. One of the goals was to make it easy to set up unit testing, but never got around to writing unit test, which presumably would have caught this error.
I'll get it patched and updated.
Thanks for the feedback.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Dave,
I just installed the latest release of gupload.py and it fails with the
following error:
[frank@cnc ~]$ gupload.py -v 1
pwd
/2015-02-27_06-24-38_4_10.fitTraceback (most recent call last):
File "/usr/bin/gupload.py", line 399, in <module>
g = gupload(myargs)
File "/usr/bin/gupload.py", line 95, in init
elif os.path.isfile(self.configCurrentDir):
AttributeError: gupload instance has no attribute 'configCurrentDir'
It looks like an easy fix, i.e.
--- gupload.py 2015-02-22 09:19:29.000000000 +1100
+++ gupload.py.fix 2015-02-28 17:58:34.061316947 +1100
@@ -92,7 +92,8 @@
self.msgLogger.debug('Using credentials from command line.')
self.username=myargs.l[0]
self.password=myargs.l[1]
- elif os.path.isfile(self.configCurrentDir):
+# elif os.path.isfile(self.configCurrentDir):
+ elif os.path.isfile(configCurrentDir):
self.msgLogger.debug('Using credentials from \'%s\'.' % configCurrentDir)
config=ConfigParser.RawConfigParser()
config.read(configCurrentDir)
Other than that, thanks for the effort on this useful little utility.
Regards
Frank
Frank,
I did a lot of rerarranging from the prior version of this file. One of the goals was to make it easy to set up unit testing, but never got around to writing unit test, which presumably would have caught this error.
I'll get it patched and updated.
Thanks for the feedback.