Skip to content

Conversation

@sarbesh
Copy link

@sarbesh sarbesh commented Dec 12, 2020

Archlinux package based on

  1. Release (Stable)
  2. Git commit (Development)

@rushabh-v
Copy link
Owner

Thanks for your contribution!

# 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 <[email protected]>
Copy link
Owner

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!

Copy link
Author

Choose a reason for hiding this comment

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

done


FILE=/usr/share/pam-configs/Facerec
SUDO=/etc/pam.d/sudo
TEXT='auth sufficient pam_python.so /lib/Auth/Facerec/pam.py'
Copy link
Owner

@rushabh-v rushabh-v Dec 12, 2020

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

Copy link
Author

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

Copy link
Owner

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

Copy link
Author

Choose a reason for hiding this comment

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

renamed

if grep -q $TEXT $SUDO;then
sed -i $TEXT $SUDO
else
echo "NotE"
Copy link
Owner

Choose a reason for hiding this comment

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

Looks like a typo!?

Copy link
Author

@sarbesh sarbesh Dec 19, 2020

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

Comment on lines +1 to +10
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")
Copy link
Owner

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?

Copy link
Author

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.

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