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

Add installation instructions #11

Merged
merged 1 commit into from
Nov 20, 2024
Merged

Add installation instructions #11

merged 1 commit into from
Nov 20, 2024

Conversation

ChocolateLoverRaj
Copy link
Owner

Fixes #10

@ChocolateLoverRaj ChocolateLoverRaj added the documentation Improvements or additions to documentation label Jul 10, 2024
@ChocolateLoverRaj ChocolateLoverRaj self-assigned this Jul 10, 2024
@alba4k
Copy link

alba4k commented Nov 20, 2024

Why isn't this merged?

@ChocolateLoverRaj ChocolateLoverRaj merged commit e776877 into main Nov 20, 2024
@ChocolateLoverRaj ChocolateLoverRaj deleted the usage-docs branch November 20, 2024 17:30
@alba4k
Copy link

alba4k commented Nov 20, 2024

That was fast, lol

by the way, @ChocolateLoverRaj , would you recommend using this? is it safe / done as it is? if so, I wouldn't mind creating an aur package for it if it's fine by you

@ChocolateLoverRaj
Copy link
Owner Author

@ChocolateLoverRaj , would you recommend using this?

I would recommend using this if people want to run PAM modules in parallel within a single PAM module (such as for sudo authentication). If they can directly run multiple PAM modules in their own code, such as what KDE and GDM do for lock screens, that would be better.

is it safe

It is hacky. See #3. Kind of like [pam-fprint-grosshack](https://gitlab.com/mishakmak/pam-fprint-grosshack)

done as it is?

Yes. It is ready to be used.

if so, I wouldn't mind creating an aur package for it if it's fine by you

That is fine with me. If you want you can add one of those tables of packaging status to the README too.

@alba4k
Copy link

alba4k commented Nov 20, 2024

I would be fine with it being like fprint-grosshack, as that is what I was using before. Will wait for some extra insight into #12 before adding this to the AUR

Wouldn't make a packaging tablet like that, just mention that it is avaiable as an aur package. also, is there any reason why the file name is libpam_any.so and not pam_any.so? is that how things are on fedora? I would likely rename it to pam_any.so on arch if there is no reason to keep it otherwise

immagine

@ChocolateLoverRaj
Copy link
Owner Author

is there any reason why the file name is libpam_any.so and not pam_any.so?

It's just the default Rust output file name. Probably caused by

crate-type = ["cdylib"]
. This can be changed.

@alba4k
Copy link

alba4k commented Nov 20, 2024

Check what fedora uses for that kind of stuff first I guess, but otherwise sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Instructions please?
2 participants