Page MenuHomeFreeBSD

x11/libwacom: unbreak after rP244554 when libxml2 isn't bootlegged
ClosedPublic

Authored by jbeich on Mar 2 2020, 6:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Sep 25, 4:14 AM
Unknown Object (File)
Wed, Sep 24, 5:58 PM
Unknown Object (File)
Wed, Sep 24, 8:57 AM
Unknown Object (File)
Wed, Sep 17, 12:15 PM
Unknown Object (File)
Tue, Sep 16, 5:34 AM
Unknown Object (File)
Mon, Sep 15, 10:53 PM
Unknown Object (File)
Sep 15 2025, 9:22 AM
Unknown Object (File)
Sep 12 2025, 11:03 PM

Details

Summary
Test Plan

Builds fine on:

  • 11.3 i386 with MANPAGES=off in devel/glib20
  • 11.3 amd64

Runtime testing is not required as tests are not installed.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

manu added a subscriber: manu.

LGTM.
Thanks

This revision is now accepted and ready to land.Mar 4 2020, 8:24 AM
imp added a subscriber: imp.

seems sane and low risk of breaking other things.

rfyu28uyeg_snkmail.com added inline comments.
x11/libwacom/Makefile
14 ↗(On Diff #69086)

Making libxml2 (these patches) optional is okay. A separate commit would be worthwhile to get 'make test' working (which should then include TEST_DEPENDS for libxml2).

x11/libwacom/distinfo
9 ↗(On Diff #69086)

Can you please put anonymous patch files such as these in a "libwacom" DIST_SUBDIR?

x11/libwacom/Makefile
14 ↗(On Diff #69086)

make test not working is not a regression. USES=meson implicitly defines TEST_TARGET but tests could be run even before rP527162 e.g.,

$ make test TEST_TARGET=check
gmake[3]: Entering directory '/usr/ports/x11/libwacom/work/libwacom-libwacom-0.32/data/layouts'
/bin/sh: Syntax error: Bad fd number
x11/libwacom/distinfo
9 ↗(On Diff #69086)

These patches are ephemeral i.e., to be removed on the next update. However, DIST_SUBDIR only works for all distfiles.