Skip to content

Files required for integration with a Linux distribution #1572

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
wants to merge 16 commits into from

Conversation

mavit
Copy link
Contributor

@mavit mavit commented Sep 11, 2013

Here are various files required to integrate an application with the Linux desktop. If these files could be included with the Arduino source, Linux distributors wouldn't have to each maintain their own copies.

@cmaglie
Copy link
Member

cmaglie commented Dec 9, 2013

@mavit
most of your commits was manually merged with #1727 into ide-1.5.x branch, I think that we miss only the startup script but that seems to be Debian specific? may we close this pull request?

Also the executable flag was fixed in a past commit by @ffissore if I remember well.

C

@mavit
Copy link
Contributor Author

mavit commented Dec 11, 2013

Thanks!

The startup script changes were actually written for Fedora, and were picked up by Debian/Ubuntu, to prompt the user that they need additional permissions to access the USB port. The exact mechanism is likely to vary a little from distribution to distribution, though, so you perhaps wouldn't want to include this functionality in the binary Arduino download.

@mavit
Copy link
Contributor Author

mavit commented Dec 11, 2013

There's one more file that hasn't been merged, the .appdata.xml file, which contains data required to list Arduino in new "app stores" for Linux (see, e.g., https://wiki.gnome.org/GnomeGoals/AppDataGnomeSoftware). It's proposed by the people working on these app stores that this file should go upstream if possible, because the upstream developers are those best-placed to describe their application.

@matthijskooijman
Copy link
Collaborator

Oh, overlooked that one. I added it in #1730.

@cmaglie
Copy link
Member

cmaglie commented Dec 13, 2013

Merged also #1730 thanks!

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

Successfully merging this pull request may close these issues.

4 participants