Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Error message when a temporary file failed to be removed on Ubuntu, related to #150 #218

Closed
2 of 8 tasks
joseph2rs opened this issue Dec 19, 2016 · 2 comments
Closed
2 of 8 tasks

Error message when a temporary file failed to be removed on Ubuntu, related to #150 #218

joseph2rs opened this issue Dec 19, 2016 · 2 comments
Milestone
4.2.0

Comments

@joseph2rs
Copy link

Description

I'have an error that should be a consequence of some modifications made for #150 :

Traceback (most recent call last):
  File "/home/afsy/jrs/Documents/geometwps/srvgeomet27/src/pywps-master/pywps/app/Process.py", line 227, in _run_process
    100, STATUS.DONE_STATUS, clean=self.async)
  File "/home/afsy/jrs/Documents/geometwps/srvgeomet27/src/pywps-master/pywps/app/WPSResponse.py", line 70, in update_status
    self.write_response_doc(self.doc, clean)
  File "/home/afsy/jrs/Documents/geometwps/srvgeomet27/src/pywps-master/pywps/app/WPSResponse.py", line 84, in write_response_doc
    self.process.clean()
  File "/home/afsy/jrs/Documents/geometwps/srvgeomet27/src/pywps-master/pywps/app/Process.py", line 281, in clean
    except WindowsError as err:
NameError: global name 'WindowsError' is not defined

Environment

  • operating system: Ubuntu 14.04, 16.04
  • Python version: 2.7
  • PyWPS version: 4, latest
  • source/distribution
  • git clone
  • Debian
  • PyPI
  • zip/tar.gz
  • other (please specify):
  • web server
  • Apache/mod_wsgi
  • CGI
  • other (please specify): Flask

Steps to Reproduce

Lock on the tmp dir

Additional Information
jachym added a commit to jachym/PyWPS that referenced this issue Jan 2, 2017
@jachym
Fixing geopython#218 WindowsError on non-windows systems
00e5b11
@jachym
Copy link
Member

jachym commented Jan 2, 2017

Hi,

the error is due to missing WindwsError class on non-Windows system

I've prepared fix for that jachym@00e5b11#diff-bddcbbd3f8d8f4182a33f03a1f9a847dL278

Quick solution: just remove the two lines in Process.py

-        except WindowsError as err:
-                LOGGER.error('Windows Error: %s', err)

@jachym jachym mentioned this issue Jan 2, 2017
Merged
1 task
jachym added a commit that referenced this issue Feb 18, 2017
@jachym
Fixing #218 WindowsError on non-windows systems (#222)
418c92e
@ldesousa ldesousa added this to the 4.2.0 milestone Mar 22, 2018
@ldesousa
Copy link
Contributor

Fixed in PR #222

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
4.2.0
Development

No branches or pull requests
3 participants
@ldesousa @jachym @joseph2rs