-
-
Notifications
You must be signed in to change notification settings - Fork 7k
adding arduino-linux-setup.sh script #6053
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
Conversation
a3cc5fb
to
30a5395
Compare
You should rather provide a proper debian package for debian/ubuntu users and not install the files directly inside the filesystem. This was never a good idea. #5827 |
Hmm. I don't know if it was a good idea to automatically uninstall any packages on the system. This is what dependencies and conflicting packages are for. This does not seem useful to me. Maybe you should print some advice that if modemmanager is installed, the user should take care of it. |
Manually merged, the autorun of the script from |
Why was this merged? As I said software should not automate any installation or even uninstall software. A proper distribution package should be used instead. Otherwise people with less knowledge will mess up their system and be unable to undo all the changes. I think it was not a wise idea to follow this "concept". ArduinoCC itself should think of providing debian packages via PPA, as its the most often used distribution at the moment (debian/ubuntu) for end consumer. ArchLinux already has proper packages. |
@NicoHood this has been merged since it condenses a lot of FAQs into a "simple" bash script that the user MAY launch if needed. The script is not being launched automatically during installation so it doesn't hurt anyone 😄 |
Those bash commands have to be put into a readme with instruction details and reasons. Users will just try to run it, possibly break their system, dont know what they did and stop. Just write one good doc, but dont automate this. Its evil. But still your opinion. |
Actually there are a number of things wrong with arduino-linux-setup.sh!
Lastly - there is no documentation regarding arduino-linux-setup.sh! Nothing that tells the user whether or not to run the scrip and if so when. Personally I don't think this file should be part of the distribution at all. |
@sk8nfool you are right in all your points. |
It kind of depends on how it's fixed. If the final result has benefit then
keep it and document it. If not toss it.
…On Thu, Oct 11, 2018 at 3:06 AM Martino Facchin ***@***.***> wrote:
@sk8nfool <https://github.com/sk8nfool> you are right in all your points.
Now we only need to understand if leaving the file there (and fixing its
shortcomings) is worth the effort or if we'd better remove it entirely.
Poll time 🙂 ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6053 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ALc1imL-jqDcccijjiK8t9VbtbJLbC4Wks5uju3VgaJpZM4MZTAG>
.
|
The shell script helps the linux user to quickly configure the OS environment to work with the Arduino IDE. It mainly performs these tasks :
the script "call" is embedded in the install.sh script :
or you can launch it as standalone with :