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

libyubikey: Initial checkin of makefile #7754

Merged
merged 4 commits into from Mar 22, 2019

Conversation

Projects
None yet
3 participants
@stuwilkins
Copy link
Contributor

stuwilkins commented Dec 22, 2018

Signed-off-by: Stuart B. Wilkins stuwilkins@mac.com

Maintainer: Stuart Wilkins
Compile Tested on Marvell Armada 37x/38x/XP (Linksys WRT3200ACM (Rango))
Run tested: Marvell Armada 37x/38x/XP (Linksys WRT3200ACM (Rango))

Description:

This adds the yubikey libraries and utilities. Included is the yubico_pam module which can be used to add yubikey authentication to pam for use with ssh (for example).

Tested and verified with openssh.

This is the first attempt at a package pull request, so any comments welcome! Appologies for the mess on #7664

Thanks!

libyubikey: Initial checkin of makefile
Signed-off-by: Stuart B. Wilkins <stuwilkins@mac.com>

@hnyman hnyman added the New package label Dec 26, 2018

@aparcar

This comment has been minimized.

Copy link
Member

aparcar commented Mar 18, 2019

I guess it's okay to merge, as these are 6 commits, should I squash them? @hnyman ?

@hnyman

This comment has been minimized.

Copy link
Contributor

hnyman commented Mar 19, 2019

Ideal would be one commit per package.
(squash the three yubico-pam commits)

stuwilkins added some commits Dec 22, 2018

ykclient: Initial checkin of makefile
Signed-off-by: Stuart B. Wilkins <stuwilkins@mac.com>
ykpers: Initial checkin of makefile
Signed-off-by: Stuart B. Wilkins <stuwilkins@mac.com>
yubico-pam: Initial checkin of makefile
Signed-off-by: Stuart B. Wilkins <stuwilkins@mac.com>

@stuwilkins stuwilkins force-pushed the stuwilkins:yubico_pam_libs branch from 53af60a to f23a96d Mar 22, 2019

@stuwilkins

This comment has been minimized.

Copy link
Contributor Author

stuwilkins commented Mar 22, 2019

@hnyman I swashed the 3 commits to one....

@aparcar aparcar merged commit b7f04aa into openwrt:master Mar 22, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.