New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drake fails to build on OSX if inside a folder with parenthesis (example ~/Dropbox (Personal)/drake-distro #394

Closed
enriquefernandez opened this Issue Sep 11, 2014 · 4 comments

Comments

Projects
None yet
5 participants
@enriquefernandez

enriquefernandez commented Sep 11, 2014

My drake-distro was installed in:

~/Dropbox (Personal)/Courses/Underactuated/drake-distro

When executing:

make

it looks like things build properly until it gets to the MATLAB compilation of drake.
The compilation of the first item fails and reports:
bash: syntax error near unexpected token `('

This happens because the classpath passed to java is not escaped correctly.
Apparently, when you use '(' in bash, you need to enclose the path with " or ', but that's not what the compilation is doing and therefore it fails.

Note that Dropbox folders with parenthesis in it are common if you have also enterprise Dropbox accounts (such as the MIT ones we get now).

Installation worked properly, once I moved the drake-distro to a folder with no parenthesis in it.

If this can't be fixed, I suggest that it's at least mentioned in the installation instructions.

@RussTedrake

This comment has been minimized.

Show comment
Hide comment
@RussTedrake

RussTedrake Sep 13, 2014

Contributor

I took a look at this today, and while it's certainly possible, it will be hard to fix. The classpath that you see the error on is generated by UseJava.cmake (not my code), and there are a number of other places in the code where things are going to break similarly. I've added a note to the installation instructions, and will leave this bug open til it's resolved.

Contributor

RussTedrake commented Sep 13, 2014

I took a look at this today, and while it's certainly possible, it will be hard to fix. The classpath that you see the error on is generated by UseJava.cmake (not my code), and there are a number of other places in the code where things are going to break similarly. I've added a note to the installation instructions, and will leave this bug open til it's resolved.

@enriquefernandez

This comment has been minimized.

Show comment
Hide comment
@enriquefernandez

enriquefernandez Sep 15, 2014

Hi Russ

I imagined this one would be hard to fix. Anyway, people normally don't have folders with parenthesis on them. Unfortunately, Dropbox made the choice of automatically changing old personal Dropbox folders to 'Dropbox (Personal)' when adding an enterprise account (such as the one we now get free at MIT).

Thank you.

enriquefernandez commented Sep 15, 2014

Hi Russ

I imagined this one would be hard to fix. Anyway, people normally don't have folders with parenthesis on them. Unfortunately, Dropbox made the choice of automatically changing old personal Dropbox folders to 'Dropbox (Personal)' when adding an enterprise account (such as the one we now get free at MIT).

Thank you.

@RussTedrake

This comment has been minimized.

Show comment
Hide comment
@RussTedrake

RussTedrake Sep 15, 2014

Contributor

totally understand. i’ll leave this open as low-priority for now.

On Sep 15, 2014, at 2:02 PM, Enrique Fernández notifications@github.com wrote:

Hi Russ

I imagined this one would be hard to fix. Anyway, people normally don't have folders with parenthesis on them. Unfortunately, Dropbox made the choice of automatically changing old personal Dropbox folders to 'Dropbox (Personal)' when adding an enterprise account (such as the one we now get free at MIT).

Thank you.


Reply to this email directly or view it on GitHub.

Contributor

RussTedrake commented Sep 15, 2014

totally understand. i’ll leave this open as low-priority for now.

On Sep 15, 2014, at 2:02 PM, Enrique Fernández notifications@github.com wrote:

Hi Russ

I imagined this one would be hard to fix. Anyway, people normally don't have folders with parenthesis on them. Unfortunately, Dropbox made the choice of automatically changing old personal Dropbox folders to 'Dropbox (Personal)' when adding an enterprise account (such as the one we now get free at MIT).

Thank you.


Reply to this email directly or view it on GitHub.

@david-german-tri

This comment has been minimized.

Show comment
Hide comment
@david-german-tri

david-german-tri Jan 13, 2017

Contributor

I'm going to close this due to low importance and years of inactivity.

Contributor

david-german-tri commented Jan 13, 2017

I'm going to close this due to low importance and years of inactivity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment