MantisBT - Doomseeker
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0002323Doomseeker[All Projects] Bugpublic2015-06-21 21:042018-09-29 14:35
ReporterZalewa 
Assigned ToZalewa 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformMicrosoftOSWindowsOS VersionXP/Vista/7
Product Version1.0 
Target Version1.1Fixed in Version1.1 
Summary0002323: Use _wgetenv() to fetch environment variables on Windows
DescriptionWe use getenv() currently and it will fail with non-latin characters. _wgetenv() should be used.
Steps To Reproduce
Additional Information'https://msdn.microsoft.com/en-us/library/tehxacec.aspx [^]'
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-06-21 21:04ZalewaNew Issue
2015-06-22 00:38Blzut3Note Added: 0012773
2015-07-07 17:46ZalewaNote Added: 0012838
2015-07-07 17:46ZalewaAssigned To => Zalewa
2015-07-07 17:46ZalewaStatusnew => needs review
2015-07-31 19:02ZalewaNote Added: 0013023
2015-07-31 19:02ZalewaStatusneeds review => resolved
2015-07-31 19:02ZalewaFixed in Version => 1.1
2015-07-31 19:02ZalewaResolutionopen => fixed
2015-07-31 19:03ZalewaNote Edited: 0013023bug_revision_view_page.php?bugnote_id=13023#r7780
2018-09-29 14:25WubTheCaptainStatusresolved => closed
2018-09-29 14:35WubTheCaptainTarget Version => 1.1

Notes
(0012773)
Blzut3   
2015-06-22 00:38   
Looks like there's also QProcessEnvironment to get the environment variables straight from Qt.
(0012838)
Zalewa   
2015-07-07 17:46   
QProcessEnvironment it is, then:
'https://bitbucket.org/Blzut3/doomseeker/commits/6edf7ebc53de2236699098131f37e69be3ba89f7 [^]'
(0013023)
Zalewa   
2015-07-31 19:02   
(edited on: 2015-07-31 19:03)
Update with this change is online in beta update channel. I don't expect anyone to bother testing if it works correctly, but if anyone wants to give a shot here are the instructions:

1. Close Doomseeker.
2. Delete Zandronum configuration from doomseeker.ini.
3. Change name of your C:\Program Files directory to something that contains characters outside ASCII range (cyrillic will do fine). Ensure your Windows knows this by ensuring that ProgramFiles env vars point to the renamed directory. I think that doing this just to test a change like this is a bit too extreme so it's best to stop at this point and not continue any further.
4. Install Zandronum to your new Progrдm Files directory.
5. Make sure Doomseeker's directory is put as far away from any Zandronum installation as possible.
6. Restart Doomseeker.
7. See if path to Zandronum installed in Progrдm files directory was detected correctly by Doomseeker.