Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 531698 - sys-auth/libfprint-0.6.0 - add support for vfs 0050 fingerprint scanner (and others from upstream)
Summary: sys-auth/libfprint-0.6.0 - add support for vfs 0050 fingerprint scanner (and ...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Michael Weber (RETIRED)
URL: https://github.com/payden/libfprint
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2014-12-04 18:19 UTC by Matthew Schultz
Modified: 2015-08-12 23:46 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
libfprint-0.5.1-add-new-drivers.patch (libfprint-0.5.1-add-new-drivers.patch,175.78 KB, patch)
2014-12-04 18:20 UTC, Matthew Schultz
Details | Diff
libfprint-0.5.1-r2.ebuild (libfprint-0.5.1-r2.ebuild,1.31 KB, text/plain)
2014-12-04 18:33 UTC, Matthew Schultz
Details
libfprint-0.6.0_vfs0050.patch (libfprint-0.6.0_vfs0050.patch,41.84 KB, patch)
2015-08-11 16:05 UTC, Matthew Schultz
Details | Diff
libfprint-0.6.0.ebuild (libfprint-0.6.0.ebuild,1.13 KB, text/plain)
2015-08-11 16:06 UTC, Matthew Schultz
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Schultz 2014-12-04 18:19:38 UTC
It doesn't appear there has been any upstream updates in a while for libfprint but someone has taken commits by upstream after 0.5.1 and added support for VFS 0050 fingerprint reader in their own github repo.  I will attach the patch for this.  I would suggest the other patch that was added to this ebuild for VFS 5011 be integrated with this patch since it does not coincide with the latest commits from upstream.

Reproducible: Always
Comment 1 Matthew Schultz 2014-12-04 18:20:29 UTC
Created attachment 390940 [details, diff]
libfprint-0.5.1-add-new-drivers.patch
Comment 2 Matthew Schultz 2014-12-04 18:33:48 UTC
Created attachment 390944 [details]
libfprint-0.5.1-r2.ebuild

I removed the vfs5011 patch since it doesn't reconcile with upstream.  I would suggest that you reconcile it and perhaps commit it to upstream and maybe also https://github.com/payden/libfprint.
Comment 3 Weedy 2015-03-01 10:12:41 UTC
0.5.1 and now 0.6.0 have been out for a while.

Can we get a version bump for both sys-auth/libfprint and sys-auth/fprintd?
Comment 4 Michael Weber (RETIRED) gentoo-dev 2015-08-09 22:11:04 UTC
I'd skip this in favor of bug 553242.
Comment 5 Matthew Schultz 2015-08-10 18:09:15 UTC
Alas, a version bump does not help.  Can't you just update this patch so it works with 0.6.0 too?
Comment 6 Matthew Schultz 2015-08-10 18:17:59 UTC
Better yet, why not use the fork that tracks upstream with this patch already built in?

https://github.com/zemen/libfprint/commits/master

Arch appears to do what I'm suggesting:

https://aur4.archlinux.org/packages/libfprint-vfs0050/
Comment 7 Matthew Schultz 2015-08-10 18:22:45 UTC
Here's the patch for 0.6.0:

https://github.com/zemen/libfprint/commit/23f1ef96dc612f80e5d7c9277b18d5973fc65cf6

Not sure why upstream still doesn't provide support for this fingerprint scanner.
Comment 8 Michael Weber (RETIRED) gentoo-dev 2015-08-10 20:12:38 UTC
(In reply to Matthew Schultz from comment #6)
> Better yet, why not use the fork that tracks upstream with this patch
> already built in?

I don't like switching upstream while keeping the package name.

I'm gonna the patch via USE=vanilla.

Regarding the patch, the last Hunk inside libfprint/fprint-list-udev-rules.c
 also need previous one [1]

[1] https://github.com/zemen/libfprint/commit/f1fdd71613a483e054cd11e19bdcfb49c95fa2cb
Comment 9 Michael Weber (RETIRED) gentoo-dev 2015-08-10 20:21:18 UTC
There's a nested comment in libfprint/drivers/vfs0050.h from line 341 to 381??
Comment 10 Matthew Schultz 2015-08-10 20:25:27 UTC
(In reply to Michael Weber from comment #9)
> There's a nested comment in libfprint/drivers/vfs0050.h from line 341 to
> 381??

Yeah I see that.  If you look at the latest version of that file, it looks like it was removed.
Comment 11 Michael Weber (RETIRED) gentoo-dev 2015-08-10 20:47:51 UTC
(In reply to Matthew Schultz from comment #10)
> (In reply to Michael Weber from comment #9)
> > There's a nested comment in libfprint/drivers/vfs0050.h from line 341 to
> > 381??
> 
> Yeah I see that.  If you look at the latest version of that file, it looks
> like it was removed.

commit bc485e300f416a29765fc1a5bba0434027a4e6f7
Author: Michael Weber <xmw@gentoo.org>
Date:   Mon Aug 10 22:41:48 2015 +0200

    sys-auth/libfprint: add support for vfs0050 scanner (bug 531698).


Can you contact the original upstream and ask for them to include the scanners?
It would be much simpler to have them maintain it instead of patching every next version bump.

Please report back, if the new version works for you.
Comment 12 Matthew Schultz 2015-08-11 16:03:32 UTC
I can't seem to get github to cooperate with its compare function but if you clone the repo and run this, you get a better patch that works (includes all the latest changes and bug fixes for that driver and not just that one commit that added the driver):

git diff bb66780cb58daf33d55524a9b434e218a991507c 62bb0228d655a80a3d39a2e1acf596ae166a5d7b > libfprint-0.6.0_vfs0050.patch
Comment 13 Matthew Schultz 2015-08-11 16:05:34 UTC
Created attachment 408804 [details, diff]
libfprint-0.6.0_vfs0050.patch
Comment 14 Matthew Schultz 2015-08-11 16:06:15 UTC
Created attachment 408806 [details]
libfprint-0.6.0.ebuild
Comment 15 Matthew Schultz 2015-08-11 16:08:46 UTC
Just to note, the original addition of the driver with that one patch did not work but I have tested the patch I just attached and verified that it works with my fingerprint reader.
Comment 16 Michael Weber (RETIRED) gentoo-dev 2015-08-11 20:20:29 UTC
I put the patch into my devspace due ~42kb size.
Thanks for testing

commit a70a2055a8dcfdfde4e69457f7b2192d50e64451
Author: Michael Weber <xmw@gentoo.org>
Date:   Tue Aug 11 22:17:48 2015 +0200

    sys-auth/libfprint: revbump for new vfs0050 patch (bug 531698).
    
    Package-Manager: portage-2.2.20

sys-auth/libfprint/Manifest
sys-auth/libfprint/libfprint-0.6.0-r1.ebuild
sys-auth/libfprint/libfprint-0.6.0.ebuild
Comment 17 Matthew Schultz 2015-08-12 13:14:32 UTC
(In reply to Michael Weber from comment #16)
> I put the patch into my devspace due ~42kb size.
> Thanks for testing
> 
> commit a70a2055a8dcfdfde4e69457f7b2192d50e64451
> Author: Michael Weber <xmw@gentoo.org>
> Date:   Tue Aug 11 22:17:48 2015 +0200
> 
>     sys-auth/libfprint: revbump for new vfs0050 patch (bug 531698).
>     
>     Package-Manager: portage-2.2.20
> 
> sys-auth/libfprint/Manifest
> sys-auth/libfprint/libfprint-0.6.0-r1.ebuild
> sys-auth/libfprint/libfprint-0.6.0.ebuild

I verified that -r1 works but why hasn't the ChangeLog been updated?
Comment 18 Matthew Schultz 2015-08-12 13:23:29 UTC
I have opened a bug to the main upstream to add support for this reader as well:

https://bugs.freedesktop.org/show_bug.cgi?id=91616
Comment 19 Michael Weber (RETIRED) gentoo-dev 2015-08-12 23:46:50 UTC
(In reply to Matthew Schultz from comment #17)
> I verified that -r1 works but why hasn't the ChangeLog been updated?

01:42:00 <@jmbsvicetto> chutzpah: we're currently just pushing the old 
                        changelogs into the rsync tree. AFAIK, we haven't 
                        created any script to push the git history into the 
                        changelogs

(In reply to Matthew Schultz from comment #18)
> I have opened a bug to the main upstream to add support for this reader as
> well:

Thanks