Skip to content
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

Ship examples #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aerusso
Copy link

@aerusso aerusso commented Aug 12, 2023

Hello!

I'd like to use the "examples" scripts. Is it a mere oversight that they are not shipped in this packaging? If so, this trivial patch puts them in /usr/share/debuerreotype/examples, and silences the lintian warning about not shipping examples.

If this is intentional, I think it would be nice to explain how to properly access these useful scripts (or why that's a bad idea).

I did not opt to use examples, because they seem to me to be at least as useful as the scripts. Again, this might come from a misunderstanding on my part of the purpose of this packaging, so please correct me if I'm wrong. In that case, it may make more sense to rename this directory (in debuerreotype proper) from "examples" to something like "distributions" (?maybe?).

Signed-off-by: Antonio Russo <[email protected]>
@tianon
Copy link
Collaborator

tianon commented Mar 23, 2024

Sorry for the delay! I thought I'd responded to this one but apparently it got away from me 🤦

These are intentionally not shipped because I do not consider the version of the files included in any given release as being necessarily suitable for use as-is, and often make important changes to them between releases (and do not make a new release when all that has changed are the example scripts).

The intended usage is to get the example scripts from the head of the repository, but that also means ensuring compatibility between a given release and the commit of the scripts is unfortunately an exercise for the reader.

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.

2 participants