Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

bump libyubikey, ykpers, yubico-piv-tool, and add ykneomgr #34875

Closed
wants to merge 4 commits into from

Conversation

jesboat
Copy link
Contributor

@jesboat jesboat commented Dec 11, 2014

No description provided.


option :universal

depends_on "help2man" => :build # <- not supposed to, but it does
Copy link
Member

Choose a reason for hiding this comment

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

Can you clarify on what this comment means?

@jacknagel
Copy link
Contributor

ping

@jacknagel
Copy link
Contributor

ping again

@jacknagel
Copy link
Contributor

Updates applied in a3aaf1f, 467b67d, and f53e3f4. Feel free to resubmit ykneomgr in a new PR.

@jacknagel jacknagel closed this Jan 18, 2015
@realdimas
Copy link
Contributor

@MikeMcQuaid, @jacknagel would you consider inclusion of ykneomgr.rb in its current form?
It is more than 6 months since original commit and it is unlikely that author would continue working on tests for ykneomgr.rb.
I feel like community would be better off having a formula with simplest tests (--version) than no formula at all.

@MikeMcQuaid
Copy link
Member

@forkbomber I'd like a test but a simple test is better than nothing, yup.

@lukehoersten
Copy link

Any progress on this?

@MikeMcQuaid
Copy link
Member

@lukehoersten No. You could submit a PR yourself. Thanks!

@realdimas realdimas mentioned this pull request Nov 1, 2015
@realdimas
Copy link
Contributor

@MikeMcQuaid - see #45572 with "--version" test as agreed.

@Homebrew Homebrew locked and limited conversation to collaborators Feb 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants