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

Switch back to upstream pam-bindings #10

Merged
merged 1 commit into from
Mar 18, 2023
Merged

Conversation

smortex
Copy link
Contributor

@smortex smortex commented Mar 6, 2023

The project was updated upsteam. Switching to it, we can get rid of the libc = ~0.1.5 dependency that cause trouble on FreeBSD:

CARGO_CRATES=libc-0.1.5 may be unstable on FreeBSD 12.0. Consider updating to the latest
version (higher than 0.2.37).

CARGO_CRATES=libc-0.1.5 may be unstable on aarch64 or not build on armv6, armv7, powerpc64.
Consider updating to the latest version (higher than 0.2.49).

It looks to my that your fork has one commit that was never proposed for integration in the upsteam crate. There seems to be conflicts with upsteam main branch, but if your change still make sense you should submit it as a PR 😉

The project was updated upsteam.  Swithing to it, we can get rid of the
`libc = ~0.1.5` dependency that cause trouble on FreeBSD:

CARGO_CRATES=libc-0.1.5 may be unstable on FreeBSD 12.0. Consider
updating to the latest version (higher than 0.2.37).

CARGO_CRATES=libc-0.1.5 may be unstable on aarch64 or not build on
armv6, armv7, powerpc64. Consider updating to the latest version (higher
than
0.2.49).
@z4yx
Copy link
Owner

z4yx commented Mar 18, 2023

Great! It's better to switch to an upstream project.

@z4yx z4yx merged commit 4712e3e into z4yx:main Mar 18, 2023
@z4yx z4yx mentioned this pull request Mar 18, 2023
@smortex smortex deleted the pam-upstream branch March 18, 2023 19:03
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