devel/dbus: add pkg-message
Needs ReviewPublic
Actions

Authored by vishwin on Feb 4 2023, 9:16 PM.

Details

Reviewers

tcberner
fluffy
arrowd

Summary

This message is mostly to remind desktop/workstation users to enable and start the daemon, as this step is sometimes overlooked.

Born out of discussion from PR 269310.

Test Plan

Any better way to word this?

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 49528
Build 46418: arc lint + arc unit

Event Timeline

vishwin created this revision.Feb 4 2023, 9:16 PM

Herald added a subscriber: mat. · View Herald TranscriptFeb 4 2023, 9:16 PM

vishwin requested review of this revision.Feb 4 2023, 9:16 PM

Harbormaster completed remote builds in B49528: Diff 116501.Feb 4 2023, 9:17 PM

vishwin edited the summary of this revision. (Show Details)Feb 4 2023, 9:23 PM

arrowd added inline comments.Feb 5 2023, 7:06 AM

devel/dbus/pkg-message.in
13	Maybe tell user to run `sysrc dbus_enable="YES"`?

grahamperrin added a subscriber: grahamperrin.Feb 5 2023, 3:08 PM

grahamperrin added inline comments.

devel/dbus/pkg-message.in
4
6–8	… ensure … make sure … – ensure alone is enough. For what it's worth, I'd make the entire message less verbose …

grahamperrin added inline comments.Feb 5 2023, 3:09 PM

devel/dbus/pkg-message.in
10–19

vishwin added inline comments.Feb 5 2023, 6:33 PM

devel/dbus/pkg-message.in
6–8	New users, particularly those who are used to having this done for them on other systems, are not aware of the rationale to enable and start a service/daemon that is not immediately apparent to proper function of their applications.

Revision Contents
Changeset List

Path

Size

devel/

dbus/

pkg-message.in

26 lines

Diff 116501

View Options

devel/dbus: add pkg-messageNeeds ReviewPublicActions