We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 1574

Summary: updaterev.sh fails when path contains spaces
Product: SDL Reporter: Andreas Schiffler <aschiffler>
Component: buildAssignee: Andreas Schiffler <aschiffler>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: blocker    
Priority: P2    
Version: HG 2.0   
Hardware: x86_64   
OS: Windows 7   
Attachments: Fixed updaterev.sh

Description Andreas Schiffler 2012-08-19 09:06:33 UTC
Created attachment 930 [details]
Fixed updaterev.sh

Building from 

 /cygdrive/c/Users/Andreas Schiffler/Desktop/Sources/SDL/build

on Windows using cygwin fails with

$ make
Warning, configure.in is out of date
#(cd .. && sh autogen.sh && sh configure)
/bin/sh ../build-scripts/updaterev.sh
mv: target `Schiffler/Desktop/Sources/SDL/build/include/SDL_revision.h' is not a directory
Makefile:113: recipe for target `update-revision' failed
make: *** [update-revision] Error 1

The script build-scripts/updaterev.sh needs to be updated to support paths with spaces in it by wrapping all occurrences of $header into "..".

Patch of fix attached.
Comment 1 Andreas Schiffler 2013-03-10 16:56:06 UTC
Assigning to myself to apply the fix.
Comment 2 Andreas Schiffler 2013-03-10 19:31:53 UTC
Patch applied and verified using this procedure
(cd SDL/build && make && make install)   
... works
(tar cvf SDL.tar && mkdir "Folder with Spaces" && cd "Folder with Spaces" && tar cvf ../SDL.tar && cd SDL/build && make clean && make && make install) 
... still works.