guile-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH] Add to the 2.1.x branch GUILE_SITE_CCACHE_DIR and GUILE_EXTENSIO


From: Freja Nordsiek
Subject: [PATCH] Add to the 2.1.x branch GUILE_SITE_CCACHE_DIR and GUILE_EXTENSION_DIR Autoconf macros along with needed siteccachdir entry in pkgconfig file
Date: Sun, 12 Mar 2017 10:38:29 +0100

Guile provides the GUILE_SITE_DIR Autoconf macro for finding where a
user should install scheme files, but there are no equivalent macros
for the directories to put compiled go files and C extensions into.
The patch adds the equivalent macros to do this for the 2.1.x branch
(it might work as is for the 2.0.x branch but I haven't tested it),
which are more or less copy-pastes of the GUILE_SITE_DIR macro.

GUILE_SITE_CCACHE_DIR does require an additional entry to be put into
Guile's pkgconfig file, which I named siteccachedir to fit in with the
rest. One major issue is that the GUILE_SITE_CCACHE_DIR macro will
fail for any version of Guile that does not have the new entry in the
pkgconfig file, which would be all 2.0.x and 2.1.x releases thus far.
One way around it would be to instead of using pkgconfig for it would
to instead use the following macro that uses the Guile interpreter
itself to find the directory (this is more or less the version I use
in one of my own projects).

    AC_DEFUN([GUILE_SITE_CCACHE_DIR],
     [AC_REQUIRE([GUILE_PROGS])
      AC_MSG_CHECKING([for Guile site-ccache directory])
      GUILE_SITE_CCACHE=`$GUILE -c "(display (%site-ccache-dir))"`
      if test $? != "0" -o "$GUILE_SITE_CCACHE" = ""; then
        AC_MSG_FAILURE([siteccachedir not found])
      fi
      AC_MSG_RESULT([$GUILE_SITE_CCACHE])
      AC_SUBST([GUILE_SITE_CCACHE])
     ])

I honestly think this is the better version since it will work even
without the entry in the pkgconfig file but using pkgconfig fits in
better with the existing macros and may be preferred by others. Also,
this version requires GUILE_PROGS to have succeeded. Even if this
version is accepted, I still think the added entry to the pkgconfig
file is a good idea.


Freja Nordsiek

Attachment: 0001-Added-GUILE_SITE_CCACHE_DIR-and-GUILE_EXTENSION_DIR-.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]