[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1)
From: |
Jefferson Felix |
Subject: |
Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1) |
Date: |
Tue, 5 Oct 2021 16:42:31 -0300 |
I just did this change locally, only the message is different (running
Guile 2), because someone also can build from source using guile 2.
I agree with this solution.
Em ter., 5 de out. de 2021 às 15:25, Jean Abou Samra <jean@abou-samra.fr>
escreveu:
> [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
>
--
---
Jefferson dos Santos Felix
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Lukas-Fabian Moser, 2021/10/05
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Jefferson Felix, 2021/10/05
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Jean Abou Samra, 2021/10/05
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1),
Jefferson Felix <=
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Jefferson Felix, 2021/10/05
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Jonas Hahnfeld, 2021/10/05
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Jefferson Felix, 2021/10/05
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Jean Abou Samra, 2021/10/05
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Jefferson Felix, 2021/10/05
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Lukas-Fabian Moser, 2021/10/05
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Jean Abou Samra, 2021/10/05
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Jefferson Felix, 2021/10/05
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Jonas Hahnfeld, 2021/10/06
- Re: Lilypond is now on Homebrew for macOS Mojave or higher (Intel or M1), Aaron Hill, 2021/10/05