open-eid / gentoo

Gentoo ebuild (community provided)
10 stars 14 forks source link

Add app-misc/idemia-awp package #31

Closed ykshatroff closed 5 years ago

ykshatroff commented 5 years ago

... to enable working with new IDEMIA id cards

This idemia awp***.deb package contains some things I don't know the purpose of, I let them stay there fwiw. Also I'd be glad to correct anything I might have done wrong, as this is my first portage. As for the thing itself, it proved to work for me. (Got an ID card quite recently - didn't work without this package)

urmet commented 5 years ago

Ugh. I hoped that I could delay having to deal with this junk(open-eid/linux-installer#37) until I get a new card myself.

Just two comments on first glance:

  1. inheriting unpacker eclass would probably remove the need for a src_unpack function
  2. MY_PV is unused and can be removed
ykshatroff commented 5 years ago

Thanks for the feedback! Yep saw that issue but just wanted to have it working here :) Removed MY_PV, added unpacker - thanks, googling didn't help a lot about .debs handling.

jaakristioja commented 5 years ago

I suggest the ebuild be called app-misc/idemia-awp-bin instead.

ykshatroff commented 5 years ago

Thanks, indeed it's more appropriate, renamed.