-
Notifications
You must be signed in to change notification settings - Fork 104
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
snapcraft is undocumented #156
Comments
@aspiers Correct - In both snapcraft and appimage the parameters are taken from the corresponding sections in the yaml files - and both of them can have configured multiple SCM sections with multiple urls configured - so I think adding parameters like we have in obs_scm and tar_scm wouldn`t make any sense. I think, the snapcraft.service and appimage.service should provide sufficient documentation and should be known as the first place when searching for documentation about the parameters of services |
@aspiers ping. Could we close the issue? The best documentation should be snapcraft.service anyway :-) |
Sorry, for some reason I am not getting a lot of the email notifications from GitHub for this project which I should - I didn't see this ping :-/
Sure, that's fine if the yaml files provide everything required.
I agree that they should, but TBH to me as someone who knows very little about snapcraft and appimage, I still get a slight "WTF" feeling when reading the README for these two
Also the README says "executes the corresponding services" for both services, but what does that actually mean? Is the source service actually building the snap or the appimage, or is that done as part of the Also, if it's only downloading sources, then why is that service mixed in with this repo which was primarily concerned with building tarballs? Do the YAML files refer to git repos or something like that? |
For the record:
It executes the services (or better equivalents), defined in the snapcraft.yml/appimage.yml
No, services never build packages
building should be not part of a service.
Correct. snapcraft.yml and appimage.yml could include the information, where to get the source code (maybe multiple repos) from. snapcraft/appimage services are only a pointer to the corresponding YAML file which include the information for the services. Unfortunatly I did not find any links to official documentation for both package formats in the short. Including examples of the current state of development in our doc`s might be a solution, but I fear that the will get outdated soon as both projects are under heavy development. |
This updates the repository's general description, and more thoroughly documents each source service. Fixes openSUSE#156.
This updates the repository's general description, and more thoroughly documents each source service. Fixes openSUSE#156.
This updates the repository's general description, and more thoroughly documents each source service. Fixes openSUSE#156.
This updates the repository's general description, and more thoroughly documents each source service. Fixes openSUSE#156.
This updates the repository's general description, and more thoroughly documents each source service. Fixes openSUSE#156.
This updates the repository's general description, and more thoroughly documents each source service. Fixes openSUSE#156.
This updates the repository's general description, and more thoroughly documents each source service. Fixes openSUSE#156.
This updates the repository's general description, and more thoroughly documents each source service. Fixes openSUSE#156.
This updates the repository's general description, and more thoroughly documents each source service. Fixes openSUSE#156.
This updates the repository's general description, and more thoroughly documents each source service. Fixes openSUSE#156.
This updates the repository's general description, and more thoroughly documents each source service. Fixes openSUSE#156.
Similarly to
obsinfo
in #155, the docs got a bit left behind when the snapcraft functionality was added. Adding one or two short paragraphs toREADME.md
would be great since it would go a long way towards clarifying what it's for and how it works. Presumably I'm only asking for ~10 minutes of your time ;-)The text was updated successfully, but these errors were encountered: