-
Notifications
You must be signed in to change notification settings - Fork 14
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
Added ArchLinux package support #26
base: master
Are you sure you want to change the base?
Conversation
Thanks for your contribution! |
archlinux/facerec-git/PKGBUILD
Outdated
# The following guidelines are specific to BZR, GIT, HG and SVN packages. | ||
# Other VCS sources are not natively supported by makepkg yet. | ||
|
||
# Maintainer: Your Name <youremail@domain.com> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, put your name and email here!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
src/cli/arch/cli.sh
Outdated
|
||
FILE=/usr/share/pam-configs/Facerec | ||
SUDO=/etc/pam.d/sudo | ||
TEXT='auth sufficient pam_python.so /lib/Auth/Facerec/pam.py' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this won't work. The file will be /usr/lib/Auth/Facerec/pam_ptn.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/usr/lib is symlink to /lib
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but the file name is pam_ptn.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
renamed
src/cli/arch/cli.sh
Outdated
if grep -q $TEXT $SUDO;then | ||
sed -i $TEXT $SUDO | ||
else | ||
echo "NotE" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a typo!?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have removed the block, as it was not functing as expected during installation, instead user has to manually ass the following line auth sufficient pam_python.so /lib/Auth/Facerec/pam_ptn.py
to /etc/pam.d/sudo
to enable and disable the authentication package
import sys | ||
import numpy as np | ||
|
||
from os import system | ||
|
||
if __name__ == '__main__': | ||
|
||
sp = np.array(sys.path) | ||
np.save("/usr/lib/Auth/Facerec/deps_path.npy", sp) | ||
system("sudo python3 /usr/lib/Auth/Facerec/cli_info.py") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't we use the build.py that is already there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do not need common-auth for ArchLinux instead, it should be done manually by adding or removing auth sufficient pam_python.so /lib/Auth/Facerec/pam_ptn.py
from /etc/pam.d/sudo
to enable and disable respectively.
Archlinux package based on