#11687: Sanitize `sage-env`
------------------------+---------------------------------------------------
Reporter: leif | Owner: leif
Type: defect | Status: new
Priority: critical | Milestone: sage-4.7.2
Component: scripts | Keywords: BUILD sage-spkg pkg-config .pc
pkgconfig PKG_CONFIG_PATH SAGE_PATH Cygwin environment variables
Work_issues: | Upstream: N/A
Reviewer: | Author:
Merged: | Dependencies:
------------------------+---------------------------------------------------
Comment(by leif):
We could also define (and export) `PKG_CONFIG_TOP_BUILD_DIR`, namely to
`${SAGE_ROOT}/local`, and use
{{{
prefix=${pc_top_builddir}
}}}
or, a bit more flexible, define it to just `${SAGE_ROOT}` and use
{{{
prefix=${pc_top_builddir}/local
}}}
in all `pkg-config` (`*.pc`) files, which avoids wrapping `pkg-config`
just to add `--define-variable=SAGE_ROOT="${SAGE_ROOT}"` to each
invocation.
This is even safer than using
{{{
prefix=$${SAGE_ROOT}/local
}}}
though I haven't [yet] encountered problems with the latter. (The `$$` has
the same meaning as in Makefiles, i.e. escapes the `$` such that the
resulting environment variable will be interpreted -- or substituted --
later, by the shell.)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11687#comment:1>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
