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

The dependencies.sh script is only for CI. #2315

Merged
merged 1 commit into from
Jun 15, 2024
Merged

The dependencies.sh script is only for CI. #2315

merged 1 commit into from
Jun 15, 2024

Conversation

larsbrinkhoff
Copy link
Member

Removed the README.md text about dependencies.sh because it's only to aid continuous integration builds. We don't want the maintenance burden of keeping this up to date for everyone's operating system release.

Removed the text about dependencies.sh because it's only to aid
continuous integration builds.  We don't want the maintenance burden
of keeping this up to date for everyone's operating system release.
Copy link
Member

@eswenson1 eswenson1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I’ll trust your judgement, but I think it should have stayed, with perhaps a caveat added to allay your concerns. For the most widely used OSes, it is helpful for people to know what the dependencies are and how to get them installed.

@larsbrinkhoff
Copy link
Member Author

The suggested change in #2277 is adding more support for various targets. I for one am not ready to take on the maintenance of such things. If someone is, I suggest starting a separate repository with helpful scripts. Depending on how that turns out, it could be folded back into this repository.

@larsbrinkhoff larsbrinkhoff merged commit 4dbbdba into master Jun 15, 2024
14 checks passed
@larsbrinkhoff larsbrinkhoff deleted the lars/deps branch June 15, 2024 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants