divyang4481 / firebreath

Automatically exported from code.google.com/p/firebreath
0 stars 0 forks source link

Prep scripts broken on XP x64 #100

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
 * WinXP Prof. x64
 * CMake 2.8.0
 * Occurs with dev, stable works fine.

When using "prep2005.cmd examples" (or one of the others):
CMake Error: The source directory "C:/dev/fb-dev/firebreath-dev"" does not 
appear to contain CMakeLists.txt.

Note the additional ".

When changing the cmake calling line in prep2005.cmd to:
cmake -G %GEN% ""-DPROJECTS_DIR=%PROJDIR%"" %* %FB_ROOT%

I get:
CMake Error at CMakeLists.txt:30 (message):
  The specified project directory C:/dev/fb-dev/firebreath-dev/examples" is
  not found

Note the additional ".

Changing it to:
cmake -G %GEN% "-DPROJECTS_DIR=%PROJDIR%" %* %FB_ROOT%

Makes it work again (but probably gives us the whitespace error again?).

Original issue reported on code.google.com by georg.fritzsche on 6 Nov 2010 at 9:05

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
I hate to say this, but if it's only broken on XP x64 I will not have time to 
fix it.  It's just not a high enough priority to make my list :-/

You can run cmake manually and still use it on XP x64, of course.

Original comment by taxilian on 8 Nov 2010 at 4:48

GoogleCodeExporter commented 8 years ago
I can verify that I'm seeing this on Windows XP SP3 (not X64) for the stable 
branch using the prep2010 script.  Georg's suggested modification solved the 
problem.

Original comment by physics...@gmail.com on 10 Nov 2010 at 1:04

GoogleCodeExporter commented 8 years ago
try it now.

Original comment by taxilian on 10 Nov 2010 at 1:42

GoogleCodeExporter commented 8 years ago
I just checked the latest stable and I'm still having the problem on XP SP3.

Is it possible that this depends on the version of cmake?  I'm running 2.8.2.
Changing common.cmd line 3 from:
   set FB_ROOT="%~dp0"
to:
   set FB_ROOT=%~dp0

fixed the problem for me -- but does this reintroduce the space problem?

Original comment by physics...@gmail.com on 10 Nov 2010 at 11:21

GoogleCodeExporter commented 8 years ago
hey, have you tried this lately?  Recent fixes to the prep scripts on windows 
may (hopefully) have fixed this...

Original comment by taxilian on 9 Dec 2010 at 7:35

GoogleCodeExporter commented 8 years ago
Not having heard anything about this in ages, I'm assuming either it's fixed or 
nobody cares.

Report back if this should be reopened.

Original comment by richarda...@gmail.com on 23 Jan 2011 at 2:32