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

sslh: add patch to work with libconfig 1.7.3 #32233

Merged
merged 1 commit into from Aug 3, 2021
Merged

Conversation

pfpulux
Copy link
Contributor

@pfpulux pfpulux commented Jul 28, 2021

General

Have the results of the proposed changes been tested?

  • I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
  • I generally don't use the affected packages but briefly tested this PR

Copy link
Member

@ericonr ericonr left a comment

Choose a reason for hiding this comment

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

If possible, can you link the commits you used on top of the patch?

@@ -10,6 +10,7 @@ license="GPL-2.0-or-later"
homepage="https://www.rutschle.net/tech/sslh/README.html"
distfiles="https://github.com/yrutschle/sslh/archive/v${version}.tar.gz"
checksum=2e457e59592f8e523cade8d9302b0fdc87f8ea0322beb674dd7f067547a93ea9
patch_args=-Np1
Copy link
Member

Choose a reason for hiding this comment

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

This is not necessary anymore.

Copy link
Member

Choose a reason for hiding this comment

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

(we default to Np1)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This patch is a merge from the last release until the patch from upstream, but only the files handle the config.

@ericonr ericonr merged commit 5609d4b into void-linux:master Aug 3, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants