Paths

Table of Contentst

Differential D4913

Initial inclusion of fprintd
ClosedPublic
Actions

Authored by abdulocracy on Jan 1 2019, 4:35 PM.

Tags

None

Referenced Files

	F11007532: D4913.diff
	Tue, Aug 1, 8:32 PM

	F10977112: D4913.id12134.diff
	Sat, Jul 22, 1:54 PM

	F10898821: D4913.id12231.diff
	Jul 3 2023, 7:00 AM

	F10884821: D4913.id.diff
	Jun 25 2023, 3:43 AM

	F10869006: D4913.diff
	Jun 18 2023, 2:06 AM

	F10854759: D4913.id12231.diff
	Jun 13 2023, 9:50 PM

	F10854747: D4913.id12219.diff
	Jun 13 2023, 9:35 PM

	F10854721: D4913.id12232.diff
	Jun 13 2023, 9:15 PM

View All 23 Files

Subscribers

Tokens

"Manufacturing Defect?" token, awarded by Jacalz.

Details

Reviewers

Group Reviewers

Maniphest Tasks

T3574: Biometrics support for UPEK fingerprint scanner.

Commits

R4664:2224915bc5fe: Initial inclusion of fprintd

Summary

Inclusion of fprintd fixes T3574.

Test Plan

Install fprintd, enroll fingerprint, verify fingerprint.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

abdulocracy created this revision.Jan 1 2019, 4:35 PM

Herald added a reviewer: Triage Team. · View Herald TranscriptJan 1 2019, 4:35 PM

abdulocracy requested review of this revision.Jan 1 2019, 4:35 PM

Jacalz awarded a token.Jan 1 2019, 5:30 PM

DataDrake requested changes to this revision.Jan 7 2019, 1:09 PM

DataDrake added a subscriber: DataDrake.

DataDrake added inline comments.

package.yml
12	Should be sorted alphabetically.

This revision now requires changes to proceed.Jan 7 2019, 1:09 PM

Alphabetize builddeps.

abdulocracy marked an inline comment as done.Jan 7 2019, 1:57 PM

DataDrake requested changes to this revision.Jan 7 2019, 3:38 PM

DataDrake added inline comments.

pspec_x86_64.xml
23	Sorry, missed this before, should be in: /usr/share/dbus-1/system.d

This revision now requires changes to proceed.Jan 7 2019, 3:38 PM

kyrios123 added a subscriber: kyrios123.Jan 7 2019, 5:29 PM

kyrios123 added inline comments.

package.yml
21	How about using `tmpfiles` for this ?
pspec_x86_64.xml
31	also `--disable-static`

der_eismann added a subscriber: der_eismann.Jan 7 2019, 7:26 PM

der_eismann added inline comments.

package.yml
21	Also please make it shorter -> `install -dm00644` (or are we not doing this anymore @kyrios123?)

Fix dbus_confdir, add gtk-doc as builddep.

abdulocracy marked 2 inline comments as done.Jan 8 2019, 9:18 AM

Disable static libs.

abdulocracy marked an inline comment as done.Jan 8 2019, 9:25 AM

abdulocracy added inline comments.Jan 8 2019, 12:46 PM

package.yml
21	Haven't used it before, what would the ownership of the directory look like (UID, GID)?

DataDrake added inline comments.Jan 8 2019, 2:58 PM

package.yml
21	Should just me `install -d` for directories. Directories should are 00755 by default and that is fine for packaging.
21	root : root
pspec_x86_64.xml
31	Nice catch.

Remove -m00644.

abdulocracy marked an inline comment as done.Jan 8 2019, 6:01 PM

Use tmpfiles.

abdulocracy marked 3 inline comments as done.Jan 8 2019, 6:22 PM

LGTM. Thanks!

This revision is now accepted and ready to land.Jan 9 2019, 6:11 PM

Closed by commit R4664:2224915bc5fe: Initial inclusion of fprintd (authored by abdulocracy, committed by DataDrake). · Explain WhyFeb 14 2019, 12:55 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

1 line

11 lines

files/

0001-Fix-dbus_confdir.patch

39 lines

fprintd.tmpfiles

1 line

22 lines

pspec_x86_64.xml

119 lines

Diff 13026

Makefile

Loading...

abi_used_libs

Loading...

files/0001-Fix-dbus_confdir.patch

Loading...

files/fprintd.tmpfiles

Loading...

package.yml

Loading...

pspec_x86_64.xml

Loading...