pkgconf: determine the default paths dynamically

[khorben]

Summary

This computes the correct LOCALBASE from the environment or from the "user.localbase" sysctl, in this order.

Sponsored by: The FreeBSD Foundation

Test Plan

Tested as follows:

$ sysctl user.localbase
user.localbase: /usr/local
$ pkgconf --dump-personality
Triplet: default
DefaultSearchPaths: /usr/local/libdata/pkgconfig:/usr/libdata/pkgconfig:/usr/local/share/pkgconfig
SystemIncludePaths: /usr/include
SystemLibraryPaths: /usr/lib
$ LOCALBASE=/opt/vendor pkgconf --dump-personality
Triplet: default
DefaultSearchPaths: /opt/vendor/libdata/pkgconfig:/usr/libdata/pkgconfig:/opt/vendor/share/pkgconfig
SystemIncludePaths: /usr/include
SystemLibraryPaths: /usr/lib
$ sudo sysctl -w user.localbase=/opt/vendor2
user.localbase: /usr/local -> /opt/vendor2
$ pkgconf --dump-personality
Triplet: default
DefaultSearchPaths: /opt/vendor2/libdata/pkgconfig:/usr/libdata/pkgconfig:/opt/vendor2/share/pkgconfig
SystemIncludePaths: /usr/include
SystemLibraryPaths: /usr/lib

[khorben]

contrib/pkgconf/libpkgconf/personality.c

121

Reading this again, I think this code will never be reached, so we wouldn't even need -DLOCALBASE=... in CFLAGS.

[khorben]

This update takes care of PERSONALITY_PATH in addition to PKG_DEFAULT_PATH.

[des]

contrib/pkgconf/libpkgconf/personality.c

108

Just use a stack buffer. The result will never be longer than MAXPATHLEN including the terminating null character.

129

It seems odd to use err() here, how does the rest of pkgconf deal with string handling failures?

[khorben]

Use a stack buffer of MAXPATHLEN for localbase, as suggested by @des.

[khorben]

contrib/pkgconf/libpkgconf/personality.c

129

Interestingly, I just identified two similar functions that use malloc(), check for errors, and report them: bool pkgconf_buffer_append_fmt() and bool pkgconf_buffer_append_vfmt() (true is success, false is error). But then when called from build_default_search_path() (the same place we are patching) possible errors are ignored, and the result inconsistent. I don't think there's much risk there so we could probably do the same... 🤔

[des]

contrib/pkgconf/libpkgconf/personality.c

108

this snprintf() is pointless, I would do the reverse: add str = localbase; to the other branch and pass str to pkgconf_buffer_append() below. You may want to rename localbase to buf and str to localbase.

[khorben]

Simplified the code obtaining LOCALBASE as per @des' suggestion. (Thanks!)

[des]

contrib/pkgconf/libpkgconf/personality.c

108

111

redundant braces

409

412

redundant braces

[khorben]

Removed redundant braces.

[des]

please mark obsolete comments as done and fix the remaining four issues

contrib/pkgconf/libpkgconf/personality.c

111

redundant

412

redundant

[khorben]

Remove redundant code, as suggested by des@. (Thanks!)