Fixed build problem when path has spaces. #34

Merged
merged 2 commits into from Jul 10, 2013

Projects

None yet

2 participants

@ralfbiedert

No description provided.

@jdf jdf commented on an outdated diff Jul 10, 2013
@@ -41,7 +41,7 @@
<attribute name="target"/>
<sequential>
<exec executable="/bin/rm" failonerror="true">
- <arg line="-rf ${basedir}/@{target}"/>
+ <arg line="-rf '${basedir}/@{target}'"/>
@jdf
jdf Jul 10, 2013 owner

I think a better solution would be to replace the offending line with

  <arg value="-rf"/>
  <arg value="${basedir}/@{target}"/>

This would handle not only spaces, but also single-quote characters!

@ralfbiedert

Can you try to paste that again? I think your command was eaten by markdown.

@jdf
Owner
jdf commented Jul 10, 2013

I've updated the comment.

@ralfbiedert

Done.

@jdf jdf merged commit 9a59933 into jdf:master Jul 10, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment