Created attachment 76917 [details]
created by `git format-patch V_0_5_0`
(1) add -f command line option to fprintd-enroll, let it can enroll all the fingers. obselete the hard-coded solution.
(2) update document occordingly
(3) remove data/fprintd.1, because that file is created by build process. It should not be maintained by git.
Created attachment 76918 [details]
created by `git format-patch V0_5_0`, tar.bz2 format
(In reply to comment #0)
> Created attachment 76917 [details]
> created by `git format-patch V_0_5_0`
> (1) add -f command line option to fprintd-enroll, let it can enroll all the
> fingers. obselete the hard-coded solution.
> (2) update document occordingly
> (3) remove data/fprintd.1, because that file is created by build process. It
> should not be maintained by git.
Could you please try to use git-bz to upload your patches?
I can't review your patch with the builtin tools if they're not separate and plain text.
It's a single script, and there's the documentation for it:
git bz attach https://bugs.freedesktop.org/show_bug.cgi?id=62644 V_0_5_0
should do the right thing.
Sorry, git bz did not work fine on my machine, I need some time to learn howto use it.....
Created attachment 78991 [details]
Hm, just created similar patch :) I should search before code.
Bastien, actually man page alredy documents that fprintd-enroll support -f option so I think this patch worth to apply this or similar patch in any case. Man page states:
For fprintd-enroll, the finger to enroll. Possible values are:
Brock.Zheng as for your patches, you do not need to learn how to use git-bz. You can just checkout master and manually copy _final_ changes in files and provide one for one change. Proplem with current patchset is that it provides some changes, then fixes bugs in this changes then updates changes... developers need only final result. Thanks anyway for your work - I've noticed that I need to update data/fprintd.pod from there :)
Please, review updated patch.
Created attachment 79011 [details] [review]
I just realized that it's possible to make patch two strings shorter :)
*** Bug 66199 has been marked as a duplicate of this bug. ***
Pushed after fixing the indentation and simplify the username selection a bit more.