Page MenuHomeFreeBSD

x11/bhotkeys-airplane: New port
ClosedPublic

Authored by dteske on Thu, Oct 8, 5:02 AM.
Tags
None
Referenced Files
F175981588: D60457.diff
Thu, Oct 15, 12:18 AM
F175970863: D60457.diff
Wed, Oct 14, 9:57 PM
Unknown Object (File)
Tue, Oct 13, 11:07 PM
Unknown Object (File)
Tue, Oct 13, 9:02 PM
Unknown Object (File)
Tue, Oct 13, 9:02 PM
Unknown Object (File)
Tue, Oct 13, 10:25 AM
Unknown Object (File)
Mon, Oct 12, 10:28 AM
Unknown Object (File)
Mon, Oct 12, 10:01 AM
Subscribers
None

Details

Summary

Fn airplane-mode and rfkill hotkeys for bhotkeys.
airplane-radiosd holds the radios for the session.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 77834
Build 74717: arc lint + arc unit

Event Timeline

dteske requested review of this revision.Thu, Oct 8, 5:02 AM
dteske created this revision.

LGTM and approved for commit.

This revision is now accepted and ready to land.Thu, Oct 8, 9:21 PM

I took a peek at the upstread code and noticed that the RC script does pw groupadd "$airplane_radiosd_group". It would be better if the port did this. See https://docs.freebsd.org/en/books/porters-handbook/book/#users-and-groups.

Same optional nit as in D60456 about the subject in the commit message.

dteske retitled this revision from x11/bhotkeys-airplane: new port to x11/bhotkeys-airplane: New port.Fri, Oct 9, 2:38 AM

Create group (airplane_radiosd:403) as part of package install

Update port to upstream 1.2 which drops group creation
and also fixes service typo in airplane-radiosd(8)

This revision now requires review to proceed.Sat, Oct 10, 3:30 PM

One thing I missed last time: You also want USE_RC_SUBR as you did in x11/framework-autorotate.

Ref: https://docs.freebsd.org/en/books/porters-handbook/book/#rc-scripts.

In D60457#1389268, @jrm wrote:

One thing I missed last time: You also want USE_RC_SUBR as you did in x11/framework-autorotate.

Ref: https://docs.freebsd.org/en/books/porters-handbook/book/#rc-scripts.

That's only for when the software doesn't ship its own rc.d script. bhotkeys-airplane has its own and also respects DESTDIR and PREFIX, so USE_RC_SUBR is not applicable. Porter's handbook on USE_RC_SUBR:

Scripts must be placed in the files subdirectory and a .in suffix must be added to their filename.

In the case of x11/framework-autorotate there is a files/framework_autorotate.in file to satisfy USE_RC_SUBR. However, in the case of this port, adding it causes a build error on missing file

This revision is now accepted and ready to land.Sat, Oct 10, 5:20 PM
This revision was automatically updated to reflect the committed changes.