Page MenuHomeFreeBSD
Differential D35494

framework: add new variable PKGCONFIG_BASE
ClosedPublic
Actions

Authored by tcberner on Jun 15 2022, 4:00 PM.
Tags
None
Referenced Files
F151807527: D35494.id107625.diff
Fri, Apr 10, 7:45 PM
F151664687: D35494.id106974.diff
Thu, Apr 9, 9:07 PM
Unknown Object (File)
Thu, Apr 9, 6:43 PM
Unknown Object (File)
Thu, Apr 9, 6:43 PM
Unknown Object (File)
Thu, Apr 9, 6:43 PM
Unknown Object (File)
Thu, Apr 9, 3:30 PM
Unknown Object (File)
Thu, Apr 9, 2:15 PM
Unknown Object (File)
Thu, Apr 9, 2:15 PM
View All Files
Subscribers
mat

Details

Reviewers
se
bapt
Group Reviewers
portmgr
Commits
R11:882f16714d90: framework: add new variable PKGCONFIG_BASE
Summary

The base system started to provide certain pkgconfig files for
components it provides. This new variable allows ports to prefer base
copmonents explicitely using for example

PKGCONFIG_BASE=foo

This will create a symlingk of /usr/libdata/pkgconfig/foo.pc to
${WKRDIR}/.pkgconfig and adding said path to pkgconfigs search path.

This allows us to switch easily between base and port implementation
of different components (say openssl) by simply defining the appropriate
variable in (say ssl.mk).

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 45990
Build 42878: arc lint + arc unit

Event Timeline

tcberner created this revision.Jun 15 2022, 4:00 PM
Herald added 1 blocking reviewer(s): portmgr. · View Herald TranscriptJun 15 2022, 4:00 PM
Herald added a subscriber: mat. · View Herald Transcript
tcberner requested review of this revision.Jun 15 2022, 4:00 PM
Harbormaster completed remote builds in B45990: Diff 106952.Jun 15 2022, 4:00 PM
bapt requested changes to this revision.Jun 15 2022, 4:03 PM
bapt added inline comments.
Mk/bsd.port.mk
1688

Here that should be PKG_CONFIGi_DIR=${PKG_CONFIG_LINKDIR}:${LOCALBASE}/libdata/pkgconfig

5162

I don't think the realpath call is necessary here.

This revision now requires changes to proceed.Jun 15 2022, 4:03 PM
tcberner added inline comments.Jun 15 2022, 4:06 PM
Mk/bsd.port.mk
5162

this was added, as some pc files are symlinks

tcberner updated this revision to Diff 106953.Jun 15 2022, 4:08 PM
  • document realpath usage
  • append default path to environment
Harbormaster completed remote builds in B45991: Diff 106953.Jun 15 2022, 4:08 PM
bapt added inline comments.Jun 15 2022, 6:19 PM
Mk/bsd.port.mk
1688

the real variable is PKG_CONFIG_LIBDIR

5162

RLN will expand to install -l rs, which in case of a symlink or symlink will to do the path expansion, so I stand by my the realpath call is not useful:

bapt@lappy:~ $ readlink /usr/libdata/pkgconfig/menu.pc
menuw.pc
bapt@lappy:~ $ install -l rs /usr/libdata/pkgconfig/menu.pc ./bla
bapt@lappy:~ $ readlink bla
../../libdata/pkgconfig/menuw.pc
tcberner updated this revision to Diff 106974.Jun 15 2022, 9:27 PM
  • Use proper pkg-config environment variable
  • Remove unnecessary realpath useage, which install (RLN) already handles
  • Also append /usr/libdata/pkgconfig to the environment
Harbormaster completed remote builds in B46001: Diff 106974.Jun 15 2022, 9:27 PM
tcberner marked 4 inline comments as done.Jun 15 2022, 9:29 PM
tcberner updated this revision to Diff 106975.Jun 15 2022, 9:34 PM
  • Use proper pkg-config environment variable
  • Remove unnecessary realpath useage, which install (RLN) already handles
  • Also append /usr/libdata/pkgconfig to the environment
Harbormaster completed remote builds in B46002: Diff 106975.Jun 15 2022, 9:34 PM
bapt accepted this revision.Jun 16 2022, 2:09 PM
This revision is now accepted and ready to land.Jun 16 2022, 2:09 PM
Closed by commit R11:882f16714d90: framework: add new variable PKGCONFIG_BASE (authored by tcberner). · Explain WhyJun 30 2022, 7:14 PM
This revision was automatically updated to reflect the committed changes.
tcberner added a commit: R11:882f16714d90: framework: add new variable PKGCONFIG_BASE.

Revision Contents
Changeset List

PathSize
Mk/
48 lines

Diff 106952

View Options

Mk/bsd.port.mk

Loading...