Opened 17 months ago

Last modified 3 months ago

#29779 new enhancement

pkg-config installed from SPKG pkgconf should not depend on environment variable SAGE_LOCAL

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-9.5
Component: packages: standard Keywords: sd111
Cc: mjo Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges


(from #29411)

Change History (9)

comment:1 Changed 15 months ago by mkoeppe

  • Cc mjo added

comment:2 Changed 15 months ago by mjo

It looks like we can specify the *.pc file location at build time using --with-pkg-config-dir:

Afterwards, we can probably just symlink pkgconf to "pkg-config", and then rely on PATH to choose the right one.

comment:3 Changed 15 months ago by mjo

src/bin/sage-env already prepends the sage-specific *.pc file path to PKG_CONFIG_PATH in case the system pkg-config is used. From pkgconf's spkg-configure.m4:

AS_IF([test -z "$ac_cv_path_PKGCONF"], [
   AC_MSG_RESULT([installing pkgconf spkg])], [
dnl the following as needed as long as Sage creates .pc files during build and/or configure
   AC_MSG_RESULT([using pkg-config from the system])

Then in sage-env,

if [ -n "$SAGE_PKG_CONFIG_PATH" ]; then
    # set up external pkg-config to look into SAGE_LOCAL/lib/pkgconfig/
    # (Sage's pkgconf spkg takes care of this, if installed)

So we "shouldn't" be hurt by ditching the pkgconf wrapper script and hard-coding the full list of paths into pkgconf at build time.

comment:4 Changed 15 months ago by mkoeppe

Sounds right to me

comment:5 Changed 12 months ago by mkoeppe

  • Milestone changed from sage-9.2 to sage-9.3

comment:6 Changed 11 months ago by mkoeppe

  • Keywords sd111 added

Hoping we can make progress on this ticket this week -

comment:7 Changed 7 months ago by mkoeppe

  • Milestone changed from sage-9.3 to sage-9.4

Sage development has entered the release candidate phase for 9.3. Setting a new milestone for this ticket based on a cursory review of ticket status, priority, and last modification date.

comment:8 Changed 5 months ago by mkoeppe

Any progress here?

comment:9 Changed 3 months ago by mkoeppe

  • Milestone changed from sage-9.4 to sage-9.5
Note: See TracTickets for help on using tickets.