Skip to content

base: lmp-passwd/group: add pulse user and group#463

Merged
ricardosalveti merged 1 commit intofoundriesio:masterfrom
munoz0raul:pulse
Nov 16, 2021
Merged

base: lmp-passwd/group: add pulse user and group#463
ricardosalveti merged 1 commit intofoundriesio:masterfrom
munoz0raul:pulse

Conversation

@munoz0raul
Copy link
Copy Markdown
Member

Required and used by the pulseaudio recipe.

Signed-off-by: Raul Muñoz [email protected]

Comment thread meta-lmp-base/files/lmp-group-table Outdated
systemd-resolve:x:993:
systemd-journal:x:994:
pulse-access:x:995:
pulse-rt:x:996:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use 973 and 974 for them, as I want to make sure that the last ones are used/reserved for systemd.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done!

Comment thread meta-lmp-base/files/lmp-group-table Outdated
pulse-rt:x:996:
fio:x:1000:
nogroup:x:65534:
nogroup:x:65534: No newline at end of file
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add newline back to the end of the line.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done!

Comment thread meta-lmp-base/files/lmp-passwd-table Outdated
systemd-resolve:x:993:993::/:/sbin/nologin
fio:x:1000:1000::/home/fio:/bin/sh
nobody:x:65534:65534:nobody:/nonexistent:/bin/sh
nobody:x:65534:65534:nobody:/nonexistent:/bin/sh No newline at end of file
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done!

Required and used by the pulseaudio recipe.

Signed-off-by: Raul Muñoz <[email protected]>
Copy link
Copy Markdown
Member

@ricardosalveti ricardosalveti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ricardosalveti ricardosalveti merged commit d8a769d into foundriesio:master Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants