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.

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

Incorrect file path separator in Processing GRASS tools #26076

Closed
qgib opened this issue Feb 23, 2018 · 3 comments
Closed

Incorrect file path separator in Processing GRASS tools #26076

qgib opened this issue Feb 23, 2018 · 3 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Easy fix GRASS

Comments

@qgib
Copy link
Contributor

qgib commented Feb 23, 2018

Author Name: Andy Harfoot (Andy Harfoot)
Original Redmine Issue: 18185
Affected QGIS version: 2.18.16
Redmine category:processing/grass
Assignee: Victor Olaya


When running i.maxlik from the GRASS7 tools in the Processing toolbox, the command fails when copying the signature file to the temporary GRASS mapset. The failure is due to the default Windows file path separator () being altered to a Linux one (/) by the tool script.

This has been previously reported here:
https://gis.stackexchange.com/questions/244245/osgeo4w-qgis-2-18-9-grass-i-maxlik-cant-find-signature-on-windows-10-network

A fix is supplied in the form of a patch to the copyFile function in the GRASS i. Processing utils module here:
\apps\qgis\python\plugins\processing\algs\grass7\ext\i.py

However the alteration of the path occurs in the getParameterValue function.


@qgib
Copy link
Contributor Author

qgib commented Jan 20, 2019

Author Name: Jürgen Fischer (@jef-n)


Please test with QGIS 3.4 - QGIS 2.18 reached it's end of life.


  • status_id was changed from Open to Feedback

@qgib
Copy link
Contributor Author

qgib commented Jan 21, 2019

Author Name: Andy Harfoot (Andy Harfoot)


This bug has been fixed in QGIS3.

@qgib
Copy link
Contributor Author

qgib commented Jan 21, 2019

Author Name: Nyall Dawson (@nyalldawson)


  • resolution was changed from to fixed/implemented
  • status_id was changed from Feedback to Closed

@qgib qgib closed this as completed Jan 21, 2019
@qgib qgib added Bug Either a bug report, or a bug fix. Let's hope for the latter! GRASS Easy fix labels May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Easy fix GRASS
Projects
None yet
Development

No branches or pull requests

1 participant