tatrefthekiller Posted July 17, 2009 Report Share Posted July 17, 2009 (edited) Hi,I tried to compile the program, but i went into problem when running the command ./update-workspaces.shThe problem is, my working directory contains spaces, so the path is incorrect. I corrected the script as follow :#!/bin/sh# build/workspaces/start_dir=$(pwd)premake_dir=$(pwd)/../premakeworkspace_dir=$(pwd)/gccmkdir "$workspace_dir" 2>/dev/nullcd "$premake_dir"# build/premake/make -C srcHOSTTYPE=$HOSTTYPE ./premake --outpath "$workspace_dir" --atlas --collada "$@" --target gnu# These files need to be linked; premake makefiles assume that the# lua file is accessible from the makefile directorycd "$workspace_dir"ln -f -s "$premake_dir"/premake.lua "$premake_dir"/functions.lua .if [ -x "$premake_dir"/premake ]; then ln -f -s "$premake_dir"/premake .ficd "$start_dir"I just added some "" around paths.I hope it helps. Edited July 17, 2009 by tatrefthekiller Quote Link to comment Share on other sites More sharing options...
espes Posted July 17, 2009 Report Share Posted July 17, 2009 Make a diff and open up a ticket. I don't know how active devs are on this forum. Quote Link to comment Share on other sites More sharing options...
Belisarivs Posted July 17, 2009 Report Share Posted July 17, 2009 It is _bad_ habit to use spaces in dirs and filenames. Quote Link to comment Share on other sites More sharing options...
tatrefthekiller Posted July 17, 2009 Author Report Share Posted July 17, 2009 It's not a bad habit, Linux support filenames with spaces, I just use it.The compiler and tools are supposed to handle spaces too... Quote Link to comment Share on other sites More sharing options...
Ykkrosh Posted July 17, 2009 Report Share Posted July 17, 2009 Committed your fixed version to SVN. Thanks!Make a diff and open up a ticket. I don't know how active devs are on this forum.We're more active here than on Trac, though posting patches to Trac is probably a better idea since it reduces the chance of them slipping through cracks. Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.