[Lukas]
Would it be considered reasonable to hard-wire a special
startup message in LilyPond proper if it is being built
with Guile2 (or rather: detects guile-2 on startup)?


I like the principle. However, Homebrew seems to want versioned
tarballs, and I am not sure Phil and Jonas should have to go
through the multi-hour release process for this.


[Felix]
Yes, exactly. We don't use patches and code replacements are avoided as
much as possible.


How about making an exception? After all, what is the purpose
of avoiding code replacements in the first place? Presumably
that's in order not to introduce own bugs and not to have to
maintain the code in parallel. Applying something like the
following should be rather safe in this regard.

diff --git a/lily/lily-version.cc b/lily/lily-version.cc
index 3d2b268b0a..f946750821 100644
--- a/lily/lily-version.cc
+++ b/lily/lily-version.cc
@@ -21,6 +21,7 @@

 #include "config.hh"
 #include "version.hh"
+#include "lily-guile.hh"

 using std::string;

@@ -45,6 +46,9 @@ string
 gnu_lilypond_version_string ()
 {
   string str = gnu_lilypond_string () + " " + version_string ();
+  #if GUILEV2
+  str = str + " with Guile 2 (from Homebrew)";
+  #endif
   return str;
 }

Thanks,
Jean

Reply via email to