Page MenuHomeFreeBSD

misc/bosd: new port
Needs ReviewPublic

Authored by dteske on Wed, Sep 9, 5:50 AM.
Tags
None
Referenced Files
F171230332: D59517.id186240.diff
Wed, Sep 9, 4:44 PM
F171229796: D59517.diff
Wed, Sep 9, 4:37 PM
F171186537: D59517.diff
Wed, Sep 9, 8:48 AM
Subscribers
None

Details

Reviewers
jrm
fuz
Summary

X11 on-screen display engine: PNG glyphs, countdown digits,
outlined text, and a gauge bar. A warm daemon per channel
repaints in place.

Diff Detail

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

Event Timeline

dteske requested review of this revision.Wed, Sep 9, 5:50 AM
dteske created this revision.

This one may need a Makefile patch to remove -I/usr/local/include and -L/usr/local/lib. Check if it then compiles without USES=localbase; after all, it seems to be using pkg-config to find its dependencies. I recommend USES=localbase:ldflags instead of USES=localbase.

Would it be a good idea to ship the glyph builder python script?

Category misc seems weird. Maybe x11 (remember, wayland stuff too goes into x11), deskutils, or graphics?

Did you test your ports with Poudriere? If yes, what architectures and operating system versions did you test?

misc/bosd/pkg-descr
6

Please don't put WWW tags into pkg-descr anymore. These have been replaced by WWW macros in Makefile.
That said, the same as in D59509 applies to this DR: if the website is just the github repository, you can omit it; it'll be auto-generated.

In D59517#1365748, @fuz wrote:

This one may need a Makefile patch to remove -I/usr/local/include and -L/usr/local/lib. Check if it then compiles without USES=localbase; after all, it seems to be using pkg-config to find its dependencies. I recommend USES=localbase:ldflags instead of USES=localbase.

I control the upstream, so I can adjust things to better suit ports (read: avoiding a local patch). FreeBSD ports is the first class target of this software, and currently I'm not worried about supporting any other Operating System or build environment (this is squarely for my work with the Framework Laptop 12 system integration -- largely for supporting the media function keys on the Framework keyboards and providing immediate responsive feedback to the user when a key is pressed).

Would it be a good idea to ship the glyph builder python script?

Absolutely, I just had not decided on where to put it yet, and I wanted to bundle an example with it too. I'll make that a priority for 4.0 before I update the review again.

Category misc seems weird. Maybe x11 (remember, wayland stuff too goes into x11), deskutils, or graphics?

You're right, my list of choices to conform to other OSD libraries/utilities was:

a. deskutils
b. misc
c. sysutils
d. x11

Based on what we've already ported:

$ ls -1d /usr/ports/*/{xob,dunst,*osd}
/usr/ports/deskutils/notify-osd/
/usr/ports/misc/xosd/
/usr/ports/sysutils/dunst/
/usr/ports/sysutils/nbosd/
/usr/ports/x11/xob/

Thanks for clarifying that we can stuff Wayland compatible solutions into x11 as well (I intend to support Wayland)

I'll move it to x11

Did you test your ports with Poudriere? If yes, what architectures and operating system versions did you test?

I tested them without poudriere. My test process currently is:

  1. portclippy Makefile
  2. portfmt -D Makefile
  3. make package
  4. pkg install -f work/pkg/*.pkg
  5. pkg list <name>
  6. Exercise the software some

But you are absolutely right, and I was thinking about a lot about how Florence went. That the missing dependencies that I had to add to Florence would have been immediately caught by poudriere because it would have built in a jail without visibility of what's installed on my system.

I'm new to poudriere and getting more comfortable with it. I'll work in a testport run with this before next update.

ASIDE: My first exposure to poudriere was in D58855 wherein I was asked to make sure that the test process for security/linux-rl9-ca-certificates did not break against the kernel VFS patchset

misc/bosd/pkg-descr
6

Derp! Thanks! Will omit and update D59509 as well, in-kind.