Errors in ExamineSnap

The last commit has introduced some errors (Linux environment) when calling pyroSAR.snap.geocode:

Traceback (most recent call last):
  File "/usr/local/bin/geocode.py", line 13, in <module>
    from pyroSAR.snap import geocode
  File "/usr/local/lib/python2.7/dist-packages/pyroSAR/snap/__init__.py", line 4, in <module>
    snap_config = ExamineSnap()
  File "/usr/local/lib/python2.7/dist-packages/pyroSAR/snap/auxil.py", line 250, in __init__
    self.__get_etc()
  File "/usr/local/lib/python2.7/dist-packages/pyroSAR/snap/auxil.py", line 262, in __get_etc
    except WindowsError:
NameError: global name 'WindowsError' is not defined

If I adjust this manually to OSError than the following error occurs:

Traceback (most recent call last):
  File "/usr/local/bin/geocode.py", line 13, in <module>
    from pyroSAR.snap import geocode
  File "/usr/local/lib/python2.7/dist-packages/pyroSAR/snap/__init__.py", line 4, in <module>
    snap_config = ExamineSnap()
  File "/usr/local/lib/python2.7/dist-packages/pyroSAR/snap/auxil.py", line 250, in __init__
    self.__get_etc()
  File "/usr/local/lib/python2.7/dist-packages/pyroSAR/snap/auxil.py", line 260, in __get_etc
    self.config_path = os.path.join(self.etc, [s for s in self.auxdata if "snap.auxdata.properties" in s][0])
IndexError: list index out of range

I also do not understand why the “etc” directory is related to the path of the executable (self.path), see line 258 of pyroSAR/snap/auxil.py: self.etc = os.path.join(os.path.dirname(os.path.dirname(self.path)), 'etc')

On my system I have an “etc” directory within HOME/.snap with configuration files.

[jonas-eberle]

In this case I use SNAP 5. The path variable (self.path) is set to /usr/local/bin/snap, which is a link: lrwxrwxrwx 1 root staff 18 Mar 31 14:13 /usr/local/bin/snap -> /opt/snap/bin/snap

The etc variable (https://github.com/johntruckenbrodt/pyroSAR/blob/master/pyroSAR/snap/auxil.py#L258) is then /usr/local/etc but it should be /opt/snap/etc. So it seems that the link generates the error.

The config files are indeed located in /opt/snap/etc.

Edit: You could use os.path.islink(path) to check for symlinks and os.path.realpath(path) to resolve the symlinks:

>>> os.path.realpath('/usr/local/bin/snap')
'/opt/snap/bin/snap'

[ibaris]

Roger that! I delete the comma. Exactly as you said the comma convert the dict into a tuple. I will push the changes in to my forked repository and delete this issue here. If you find any other bugs we can generate a new issue.

Thanks again for your message, advices and suggestions.