Euphoria
Ticket #594:
Cannot use euc in paths with spaces
-
Reported by
euphoric
Jan 14, 2011
A space in the filepath when using euc with a resource file (e.g., "-rc-file myres") causes an error.
Details
1. Comment by mattlewis
Jan 14, 2011
As a temporary workaround, try using the -makefile option. Then, you should be able to edit the makefile to add proper quoting or whatever it needs.
2. Comment by euphoric
Jan 14, 2011
I have added the proper quotations to buildsys.e, but it's still failing. I'll try the -makefile option...
3. Comment by euphoric
Jan 14, 2011
I got around this by doing these steps manually:
euc myprog.exw
wrc -r myres.rc
wrc -q -ad myres.res myprog.exe
Maybe buildsys.e or the makefile can be changed to try to do it like this since Watcom's wrc doesn't seem to like spaces in paths... that is, use relative paths instead of the current method of using full paths.
I modified buildsys.e so that it puts quotes around the paths, but that doesn't help.
I can do nothing more about wrc's inability to handle spaces in our source.
4. Comment by SDPringle
Feb 13, 2011
I already have a subroutine written that will fix this. It is in the tip of the 4.0 branch. It returns the path as short names when available. I have to look for it somewhere but then it is just a matter of calling the routine where buildsys adds the resource. grep the source for 'work-around' and 'WATCOM' (case insensitive). If not, I'll get around to this at the end of the week.
5. Comment by mattlewis
Feb 14, 2011
Based on Shawn's comment, moving this to "Fixed, Please Confirm." I believe this change has also been merged into the trunk.
6. Comment by SDPringle
Feb 16, 2011
I've created a test case:
mkdir 'Hello World'
cd "Hello World"
copy %EUDIR%\eubw.res .
Now inside this directory I put hello.exw:
include std/win32/msgbox.e
atom ur = message_box("Hello World", "Info", MB_OK)
The ICON and manifest includes in the resource file always caused an error. I removed these lines to simplify the problem.
I was able to reproduce the problem with this test case and now I have a fix and I am using the unit tests to make sure nothing breaks.
7. Comment by SDPringle
Feb 16, 2011
The fix is now in the 4.0 branch.
8. Comment by jimcbrown
Feb 16, 2011
I'm reopening this ticket because the wrong version control system is being used.
svn:4273 was made before this ticket was opened and fixes an entirely unrelated issue.
If this has been fixed, please enter the full nodeid of the hg revision that the fix was made in.
9. Comment by SDPringle
Feb 16, 2011
The fix is in 4270:71e4b1210cf0
10. Comment by euphoric
Apr 04, 2011
I was able to translate wxEDB.exw as found in C:\Program Files\wxEDB\ without an issue.
GitHub