Skip to content

Make 'path too long' error more descriptive #722

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

Closed
Normalised opened this issue Apr 20, 2017 · 6 comments
Closed

Make 'path too long' error more descriptive #722

Normalised opened this issue Apr 20, 2017 · 6 comments

Comments

@Normalised
Copy link

Just updated to 4.0.1.201704171301 from 4.0.1.201704020116 and I now have an error :

The path C:/dev/apps/eclipse/cpp-neon/eclipse/arduinoPlugin is to long

Which prevents the Upload Sketch command from starting.

(I've reverted my install so I can keep working)

@jantje
Copy link
Member

jantje commented Apr 20, 2017

This is not a issue but a design decision #705

@jantje jantje closed this as completed Apr 20, 2017
@Normalised
Copy link
Author

Ok, but leaving a previously working install in a broken state with no explanation doesn't seem like the best way of implementing this design decision.

At least a dialog or warning explaining that 'The installation path is too long and the plugin will no longer function correctly' would go a long way to helping.

@jantje
Copy link
Member

jantje commented Apr 20, 2017

Didn't you get a dialog box?

@Normalised
Copy link
Author

Yes, but it just says there's an error and that path is too long. Not friendly or helpful.

I suggest:

Due to issues with long pathnames on Windows, the plugin installation path must less than 40 characters. Your current path: PLUGIN_PATH is too long and the plugin will no longer function correctly
Please visit #705 for details.

@jantje jantje reopened this Apr 20, 2017
@jantje jantje changed the title Update to 4.0.1.201704171301 caused a 'path too long' error Make 'path too long' error more descriptive Apr 20, 2017
jantje pushed a commit that referenced this issue Apr 20, 2017
jantje pushed a commit that referenced this issue Apr 20, 2017
@jantje
Copy link
Member

jantje commented Apr 20, 2017

I hope you like the new message more

@Normalised
Copy link
Author

Fantastic 👍

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

No branches or pull requests

2 participants