[Bug 469955] Review Request: fprintd - D-Bus service for Fingerprint reader access

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=469955


Till Maas <opensource at till.name> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |opensource at till.name




--- Comment #1 from Till Maas <opensource at till.name>  2008-11-06 08:38:02 EDT ---
Some issues I noticed at my first check:

- Why do you need a Conflict with pam_fprint, are Obsolutes/Provides not better
fitted here?
https://fedoraproject.org/wiki/Packaging/Conflicts

- Source0 is not a URL and it is not explained how to create the source
https://fedoraproject.org/wiki/Packaging/SourceURL

- Is it intentional that the config files in /etc are not marked as %config?

Btw. the release string of the linked libfprint seems to be wrong, it should be
0.1.pre.fc10 instead of 1.pre.fc10

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.