- Now the normal install instructions are in "Install-howto.md"
(and its corresponding .html file). Advanced materials are in INSTALL.
Previously each tutorial, and README, duplicated a lot of it.
- The shell variables use "_" (underscore) but the
user-visible options use "-" (dash).
Since the error messages are reports back to the user, the
error messages should use the user-visible dash instead of underscore.
- Package "cl-readable", the Common Lisp library package for readable,
had a subtle error in the pre-uninstall (preun) specification;
it used the wrong case for a variable (%cl_NAME should be %cl_name).
This meant that while it installed just fine, an attempt to
un-install it could produce this error:
"error: %preun(cl-readable-0.9.3-1.fc17.x86_64) scriptlet failed,
exit status 42"
This was solved by fixing the spec to use the correct variable name.
- Now that ignoring guile's reported Guile site directory
is exceedingly rare, there's really no point in doing
the expansion and checking it. It's just a lot of extra
code that's unlikely to help and is confusing. So, get rid of it.
- The GNU autoconf documents claim that "!" isn't portable across shells.
It's in POSIX, and widely available, but it's also easy to remove,
so this removes various uses of it.
- Another change to the GUILE_SITE configuration algorithm,
in an effort to "get it right as automatically as possible"
while allowing "make distcheck" to work.
A "make distcheck" sets prefix specially (to end in "/_inst"),
so we'll handle that specially. Otherwise, if we can get the
guile site directory value from guile, use it.
- GUILE_SITE is often "/usr/share/guile/site"; if prefix=/usr/local,
pass that through. Since these are the defaults, this common
case works "by default", yet "make distcheck" still works.
- Give up trying to patch GUILE_SITE for all cases.
Instead, get GUILE_SITE info from guile, compute
what GUILE_SITE would be using $prefix, and warn if there
appears to be an inconsistency. It's hard to figure out what to do
in all cases, but warning only when there's a problem (and giving
suggestions) seems reasonable.
With this change, "make distcheck" works just fine.
- Report the value of $GUILE_READLINE_AVAILABLE before error-checking it.
If there's a problem, the actual value is reported
(not just that there's a problem), and the error report looks nicer too.
- Only patch GUILE_SITE if $datarootdir has the expected value,
'${prefix}/share'. This is the default value, but users can change it.
This way, the "usual case" is patched, but we don't wrongly
patch GUILE_SITE if it produces unexpected values.
- If we're given an explicit GUILE_SITE, then do *NOT* patch it, just
use it directly. But if we are not given a specific GUILE_SITE
value, then query guile and try to patch it.
The goal is to try to make things work automatically, but give
the builder complete control.
- There are also several sanity checks with warnings for GUILE_SITE,
so that if things go wrong there's a hint about it.
- GUILE_SITE still requires patching so "make distcheck" will work.
Restore the code to patch it. However, do it as a separate step,
so that people invoking "configure" can see what's happening.
- This notionally reverts commit c63bc66add.
- This discusses more about ASDF and interaction with configure.
Basically, ASDF and configuration have to agree, so
1. set configure options to match what ASDF expects, or
2. change ASDF configuration to match how configure will install it.