Skip to content

wxGUI: do not import from pygrass on gui startup#2202

Merged
petrasovaa merged 1 commit into
OSGeo:mainfrom
petrasovaa:SignatureSelect-decode
Feb 13, 2022
Merged

wxGUI: do not import from pygrass on gui startup#2202
petrasovaa merged 1 commit into
OSGeo:mainfrom
petrasovaa:SignatureSelect-decode

Conversation

@petrasovaa
Copy link
Copy Markdown
Contributor

Replaces pygrass.utils.decode as not needed here, grass.utils.decode was called anyway.
This should prevent that gui crashes when loading C libraries fails, see #2192.

Replaces pygrass.utils.decode as not needed here, grass.utils.decode was called anyway
@petrasovaa petrasovaa added this to the 8.0.1 milestone Feb 13, 2022
@petrasovaa petrasovaa merged commit 8e66baa into OSGeo:main Feb 13, 2022
@petrasovaa petrasovaa deleted the SignatureSelect-decode branch February 13, 2022 20:22
@petrasovaa
Copy link
Copy Markdown
Contributor Author

I believe this is safe to backport.

petrasovaa added a commit that referenced this pull request Feb 13, 2022
Replaces pygrass.utils.decode as not needed here, grass.utils.decode was internally called anyway.
This should prevent that gui crashes when loading C libraries fails, see #2192.
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Oct 26, 2022
Replaces pygrass.utils.decode as not needed here, grass.utils.decode was internally called anyway.
This should prevent that gui crashes when loading C libraries fails, see OSGeo#2192.
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Feb 17, 2023
Replaces pygrass.utils.decode as not needed here, grass.utils.decode was internally called anyway.
This should prevent that gui crashes when loading C libraries fails, see OSGeo#2192.
neteler pushed a commit to nilason/grass that referenced this pull request Nov 7, 2023
Replaces pygrass.utils.decode as not needed here, grass.utils.decode was internally called anyway.
This should prevent that gui crashes when loading C libraries fails, see OSGeo#2192.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GUI wxGUI related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant