Paths

Table of Contentst

Differential D56642

libpkgconf: Fix paths
ClosedPublic
Actions

Authored by des on Apr 26 2026, 9:27 AM.

Tags

None

Referenced Files

	F166654107: D56642.id176555.diff
	Sat, Aug 15, 6:15 AM

	Unknown Object (File)
	Thu, Aug 13, 1:07 AM

	Unknown Object (File)
	Tue, Aug 11, 8:21 PM

	Unknown Object (File)
	Tue, Aug 11, 7:52 PM

	Unknown Object (File)
	Mon, Aug 10, 10:44 AM

	Unknown Object (File)
	Mon, Aug 10, 7:10 AM

	Unknown Object (File)
	Sun, Aug 9, 5:28 PM

	Unknown Object (File)
	Sun, Aug 9, 4:34 AM

View All 59 Files

Subscribers

Details

Reviewers

antoine
khorben

Commits

rG4dd97955e68d: libpkgconf: Fix paths

Summary

${LOCALBASE:U} evaluates to exactly the same thing as ${LOCALBASE}.
Presumably what was meant was ${LOCALBASE:U/usr/local}.

Fixes: b8352da33f34 ("pkgconf: import into the base system")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.Apr 26 2026, 9:27 AM

Herald added subscribers: jrtc27, imp. · View Herald TranscriptApr 26 2026, 9:27 AM

des requested review of this revision.Apr 26 2026, 9:27 AM

Harbormaster completed remote builds in B72552: Diff 176555.Apr 26 2026, 9:27 AM

LGTM, thanks!

This revision is now accepted and ready to land.Apr 26 2026, 10:44 AM

Closed by commit rG4dd97955e68d: libpkgconf: Fix paths (authored by des). · Explain WhyApr 26 2026, 1:43 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rG4dd97955e68d: libpkgconf: Fix paths.

ngie added a subscriber: ngie.Apr 27 2026, 11:26 PM

ngie added inline comments.

lib/libpkgconf/Makefile
16	Why not just do this instead? `LOCALBASE?= /usr/local` instead of all of the `:U/usr/local` gymnastics? It avoids duplicate lines with `LOCALBASE` defaulting to `/usr/local` and reduces the potential for typos creeping in.

des added inline comments.Apr 28 2026, 10:31 AM

lib/libpkgconf/Makefile
16	This is the predominant idiom for existing uses of `${LOCALBASE}` in the tree.

Revision Contents
Changeset List

Path

Size

lib/

libpkgconf/

4 lines

tools/

build/

depend-cleanup.sh

4 lines

Diff 176576

lib/libpkgconf/Makefile

Loading...

tools/build/depend-cleanup.sh

Loading...