unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* What version of harfbuzz is required?
@ 2020-11-11 15:51 Colin Baxter
  2020-11-11 16:04 ` Robert Pluim
  0 siblings, 1 reply; 7+ messages in thread
From: Colin Baxter @ 2020-11-11 15:51 UTC (permalink / raw)
  To: emacs-devel

Hello,

When building emacs from master I get the warning:

WARNING: This configuration uses the Cairo graphics library,
but not the HarfBuzz font shaping library.  We recommend the use
of HarfBuzz when using Cairo, please install HarfBuzz development
packages.

I have harfbuzz bin and dev libraries (0.9.35-2+deb8u1) installed, but
presumably they are too old. I am using i686 GNU/Linux 3.16.0-11-686-pae
#1 SMP Debian 3.16.84-1 (2020-06-09).

What minimal harfbuzz version do I need?

Thanks

Colin Baxter.



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: What version of harfbuzz is required?
  2020-11-11 15:51 What version of harfbuzz is required? Colin Baxter
@ 2020-11-11 16:04 ` Robert Pluim
  2020-11-11 16:38   ` Colin Baxter
  0 siblings, 1 reply; 7+ messages in thread
From: Robert Pluim @ 2020-11-11 16:04 UTC (permalink / raw)
  To: Colin Baxter; +Cc: emacs-devel

Colin Baxter <m43cap@yandex.com> writes:

> Hello,
>
> When building emacs from master I get the warning:
>
> WARNING: This configuration uses the Cairo graphics library,
> but not the HarfBuzz font shaping library.  We recommend the use
> of HarfBuzz when using Cairo, please install HarfBuzz development
> packages.
>
> I have harfbuzz bin and dev libraries (0.9.35-2+deb8u1) installed, but
> presumably they are too old. I am using i686 GNU/Linux 3.16.0-11-686-pae
> #1 SMP Debian 3.16.84-1 (2020-06-09).
>
> What minimal harfbuzz version do I need?

From configure.ac:

    if test "${HAVE_W32}" = "yes"; then
      EMACS_CHECK_MODULES([HARFBUZZ], [harfbuzz >= 1.2.3])
    else
      EMACS_CHECK_MODULES([HARFBUZZ], [harfbuzz >= 0.9.42])
    fi

and the configure output says:
    checking for harfbuzz >= 0.9.42... yes

I guess we could put the version in the warning as well, but it seems
a little excessive.

Robert



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: What version of harfbuzz is required?
  2020-11-11 16:04 ` Robert Pluim
@ 2020-11-11 16:38   ` Colin Baxter
  2020-11-12 11:06     ` Robert Pluim
  0 siblings, 1 reply; 7+ messages in thread
From: Colin Baxter @ 2020-11-11 16:38 UTC (permalink / raw)
  To: emacs-devel

Hello Robert,
>>>>> Robert Pluim <rpluim@gmail.com> writes:

    > Colin Baxter <m43cap@yandex.com> writes:
    >> Hello,
    >> 
    >> When building emacs from master I get the warning:
    >> 
    >> WARNING: This configuration uses the Cairo graphics library, but
    >> not the HarfBuzz font shaping library.  We recommend the use of
    >> HarfBuzz when using Cairo, please install HarfBuzz development
    >> packages.
    >> 
    >> I have harfbuzz bin and dev libraries (0.9.35-2+deb8u1)
    >> installed, but presumably they are too old. I am using i686
    >> GNU/Linux 3.16.0-11-686-pae #1 SMP Debian 3.16.84-1 (2020-06-09).
    >> 
    >> What minimal harfbuzz version do I need?

    > From configure.ac:

    >     if test "${HAVE_W32}" = "yes"; then
    > EMACS_CHECK_MODULES([HARFBUZZ], [harfbuzz >= 1.2.3]) else
    > EMACS_CHECK_MODULES([HARFBUZZ], [harfbuzz >= 0.9.42]) fi

    > and the configure output says: checking for harfbuzz >=
    > 0.9.42... yes

Ok, 0.9.42. I suppose it's time to bite the bullet and upgrade my Debian.

    > I guess we could put the version in the warning as well, but it
    > seems a little excessive.

It could be useful in avoiding eye strain in searching the configure
output for a mention of 'harfbuzz' :-)

Thanks.



Colin.



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: What version of harfbuzz is required?
  2020-11-11 16:38   ` Colin Baxter
@ 2020-11-12 11:06     ` Robert Pluim
  2020-11-12 12:00       ` Colin Baxter
  2020-11-12 14:44       ` Eli Zaretskii
  0 siblings, 2 replies; 7+ messages in thread
From: Robert Pluim @ 2020-11-12 11:06 UTC (permalink / raw)
  To: Colin Baxter; +Cc: emacs-devel

Colin Baxter <m43cap@yandex.com> writes:

>     > and the configure output says: checking for harfbuzz >=
>     > 0.9.42... yes
>
> Ok, 0.9.42. I suppose it's time to bite the bullet and upgrade my
> Debian.

The non-HarfBuzz code is still functional, and won't go away for a
while yet.

>
>     > I guess we could put the version in the warning as well, but it
>     > seems a little excessive.
>
> It could be useful in avoiding eye strain in searching the configure
> output for a mention of 'harfbuzz' :-)

You donʼt run ./configure in a shell inside emacs? ;-)

Eli, something like this OK for master?

diff --git i/configure.ac w/configure.ac
index f0c8e5210f..888b415148 100644
--- i/configure.ac
+++ w/configure.ac
@@ -3444,16 +3444,17 @@ AC_DEFUN
 fi   # "${HAVE_X11}" != "yes"
 
 HAVE_HARFBUZZ=no
+### On MS-Windows we use hb_font_get_nominal_glyph, which appeared
+### in HarfBuzz version 1.2.3
+if test "${HAVE_W32}" = "yes"; then
+  harfbuzz_required_ver=1.2.3
+else
+  harfbuzz_required_ver=0.9.42
+fi
 if test "${HAVE_X11}" = "yes" && test "${HAVE_FREETYPE}" = "yes" \
         || test "${HAVE_W32}" = "yes"; then
   if test "${with_harfbuzz}" != "no"; then
-    ### On MS-Windows we use hb_font_get_nominal_glyph, which appeared
-    ### in HarfBuzz version 1.2.3
-    if test "${HAVE_W32}" = "yes"; then
-      EMACS_CHECK_MODULES([HARFBUZZ], [harfbuzz >= 1.2.3])
-    else
-      EMACS_CHECK_MODULES([HARFBUZZ], [harfbuzz >= 0.9.42])
-    fi
+    EMACS_CHECK_MODULES([HARFBUZZ], [harfbuzz >= $harfbuzz_required_ver])
     if test "$HAVE_HARFBUZZ" = "yes"; then
       AC_DEFINE(HAVE_HARFBUZZ, 1, [Define to 1 if using HarfBuzz.])
       ### mingw32 and Cygwin-w32 don't use -lharfbuzz, since they load
@@ -5918,9 +5919,9 @@ m4_define
 
 if test "${HAVE_CAIRO}" = "yes" && test "${HAVE_HARFBUZZ}" = no; then
   AC_MSG_WARN([This configuration uses the Cairo graphics library,
-    but not the HarfBuzz font shaping library.  We recommend the use
-    of HarfBuzz when using Cairo, please install HarfBuzz development
-    packages.])
+    but not the HarfBuzz font shaping library (minimum version $harfbuzz_required_ver).
+    We recommend the use of HarfBuzz when using Cairo, please install
+    appropriate HarfBuzz development packages.])
 fi
 
 # Let plain 'make' work.



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: What version of harfbuzz is required?
  2020-11-12 11:06     ` Robert Pluim
@ 2020-11-12 12:00       ` Colin Baxter
  2020-11-12 14:44       ` Eli Zaretskii
  1 sibling, 0 replies; 7+ messages in thread
From: Colin Baxter @ 2020-11-12 12:00 UTC (permalink / raw)
  To: emacs-devel

>>>>> Robert Pluim <rpluim@gmail.com> writes:

---- cut ----

    > You donʼt run ./configure in a shell inside emacs? ;-)

Usually, but sometimes in vc-dir with M-x compile <RET> then subsequent
commands.

Best wishes,

Colin Baxter.


-- 
Colin Baxter
URL: http://www.Colin-Baxter.com



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: What version of harfbuzz is required?
  2020-11-12 11:06     ` Robert Pluim
  2020-11-12 12:00       ` Colin Baxter
@ 2020-11-12 14:44       ` Eli Zaretskii
  2020-11-12 15:56         ` Robert Pluim
  1 sibling, 1 reply; 7+ messages in thread
From: Eli Zaretskii @ 2020-11-12 14:44 UTC (permalink / raw)
  To: Robert Pluim; +Cc: emacs-devel

> From: Robert Pluim <rpluim@gmail.com>
> Date: Thu, 12 Nov 2020 12:06:49 +0100
> Cc: emacs-devel@gnu.org
> 
> Colin Baxter <m43cap@yandex.com> writes:
> 
> >     > I guess we could put the version in the warning as well, but it
> >     > seems a little excessive.
> >
> > It could be useful in avoiding eye strain in searching the configure
> > output for a mention of 'harfbuzz' :-)
> 
> You donʼt run ./configure in a shell inside emacs? ;-)
> 
> Eli, something like this OK for master?

Yes, thanks.



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: What version of harfbuzz is required?
  2020-11-12 14:44       ` Eli Zaretskii
@ 2020-11-12 15:56         ` Robert Pluim
  0 siblings, 0 replies; 7+ messages in thread
From: Robert Pluim @ 2020-11-12 15:56 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: emacs-devel

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Robert Pluim <rpluim@gmail.com>
>> Date: Thu, 12 Nov 2020 12:06:49 +0100
>> Cc: emacs-devel@gnu.org
>> 
>> Colin Baxter <m43cap@yandex.com> writes:
>> 
>> >     > I guess we could put the version in the warning as well, but it
>> >     > seems a little excessive.
>> >
>> > It could be useful in avoiding eye strain in searching the configure
>> > output for a mention of 'harfbuzz' :-)
>> 
>> You donʼt run ./configure in a shell inside emacs? ;-)
>> 
>> Eli, something like this OK for master?
>
> Yes, thanks.

Done as 56a7c73f9c



^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2020-11-12 15:56 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-11-11 15:51 What version of harfbuzz is required? Colin Baxter
2020-11-11 16:04 ` Robert Pluim
2020-11-11 16:38   ` Colin Baxter
2020-11-12 11:06     ` Robert Pluim
2020-11-12 12:00       ` Colin Baxter
2020-11-12 14:44       ` Eli Zaretskii
2020-11-12 15:56         ` Robert Pluim

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).