-
-
Notifications
You must be signed in to change notification settings - Fork 3k
-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
crash when creating new GRASS location on Windows #11104
Comments
Author Name: Marco Pasetti - (Marco Pasetti -) I think that the only possible solution is to introduce a name check to prevent this error, telling that the location Bug already exists and repromp for a new name. Marco |
Author Name: Maciej Sieczka - (Maciej Sieczka -) Bug confirmed under WINE 1.0.0 configured to emulate Win XP, on amd64 Debian testing using Marco's "QGIS 0.11.0 for Windows pre-release testing installer". Indeed it seems the culprit is Windows is not case-sensitive as GNU/Linux is. |
Author Name: Marco Pasetti - (Marco Pasetti -) Replying to [comment:4 msieczka]:
Absolutely! Windows is NOT case sensitive. As I said two months ago, there would be a simple solution: introduce a name check function to prevent this error, checking if the location's name (that is the folder) already exixts and, if yes, telling that the location Bug already exists and reprompt for a new name. I donn't know if that should be done in GRASS code; actually I think that we could do that in one of the src\qgis-dev\src\plugins\grass\ cpp files Jurgen? |
Author Name: Tom Elwertowski (Tom Elwertowski) This also happens on a Mac. There's already a duplicate name check in qgsgrassnewmapset.cpp. It needs to be made case insensitive for Mac and Win. |
Author Name: Tom Elwertowski (Tom Elwertowski) Fixed by 47dbaf2 (SVN r9288).
|
Author Name: cgsbob - (cgsbob -) Replying to [comment:7 telwertowski]:
The complete compiler output can be found here: [http://rafb.net/p/SAY4QJ37.html]
|
Author Name: Maciej Sieczka - (Maciej Sieczka -) And I get a foloowing error in 98c3309 (SVN r9291):
|
Author Name: Tom Elwertowski (Tom Elwertowski)
|
Author Name: Maciej Sieczka - (Maciej Sieczka -) Yet another error, in 2624cde (SVN r9292):
|
Author Name: Tom Elwertowski (Tom Elwertowski) Add the line #include after #include <math.h> and try compiling again. I will wait for any other issues before updating svn. What OS are you using? I will also add a comment that this file is needed for a particular OS so it doesn't get removed again.
|
Author Name: Tom Elwertowski (Tom Elwertowski) The last issue is fixed by gcc 4.3 compatibility changes in d049698 (SVN r9293).
|
Author Name: Maciej Sieczka - (Maciej Sieczka -) Buids fine now. I'm on Debian testing amd64, gcc 4.3.1. |
Author Name: Anónimo (Anónimo) Milestone Version 1.0.0 deleted |
Author Name: Redmine Admin (Redmine Admin)
Original Redmine Issue: 1044
Redmine category:grass
Assignee: Tom Elwertowski
QGIS (Windows version only) crashes if you are trying to create location with almost same name as exist (the only difference is character case).
Steps to reproduce:
The problem is that Linux is case sensitive (bug and Bug are two different strings) but in Windows there is impossible to create two files named "bug" and "Bug".
The text was updated successfully, but these errors were encountered: