bugThe FreeType Project - Bugs: bug #45828, Required version of harfbuzz too...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #45828: Required version of harfbuzz too low in configure script

Submitter:  Francois Bissey <fbissey>
Submitted:  Wed 26 Aug 2015 03:22:06 AM UTC
   
 
Severity:  3 - Normal Item Group:  Incorrect behaviour
Status:  Fixed Privacy:  Public
Assigned to:  wl Open/Closed:  Closed
Planned Release:  2.6.1

Wed 26 Aug 2015 04:21:43 AM UTC, comment #1: 

Fixed in git, thanks.

Werner LEMBERG <wl>
Group administrator
Wed 26 Aug 2015 03:22:06 AM UTC, original submission:  

Hi,

I tried to compile freetype 2.6 on scientific linux 7.1 which comes with harfbuzz 0.9.20.
When harfbuzz is enabled (default behavior if harfbuzz is 0.9.19 or higher - still the default in git) the compilation fails for src/autofit/autofit.c with
In file included from /home/frb15/sagemath/sage-brial/local/var/tmp/sage/build/freetype-2.6/src/src/autofit/autofit.c:37:0:
/home/frb15/sagemath/sage-brial/local/var/tmp/sage/build/freetype-2.6/src/src/autofit/hbshim.c: In function 'af_get_coverage':
/home/frb15/sagemath/sage-brial/local/var/tmp/sage/build/freetype-2.6/src/src/autofit/hbshim.c:190:17: error: 'HB_SET_VALUE_INVALID' und$
     for ( idx = HB_SET_VALUE_INVALID; hb_set_next( gsub_lookups, &idx ); )
                 ^
/home/frb15/sagemath/sage-brial/local/var/tmp/sage/build/freetype-2.6/src/src/autofit/hbshim.c:190:17: note: each undeclared identifier $
make[3]: * [/home/frb15/sagemath/sage-brial/local/var/tmp/sage/build/freetype-2.6/src/objs/autofit.lo] Error 1

After checking that harfbuzz-devel was installed and comparing with another machine I found that `HB_SET_VALUE_INVALID` is defined in `harfbuzz/hb-set.h` in harfbuzz -0.9.38.

After some search on github I believe this definition has been added in harfbuzz 0.9.21 (or before the 0.9.22 release)
https://github.com/behdad/harfbuzz/commit/20cbc1f8eb0811e6cefa3be3550e80df9c372782

Therefore the check for harfbuzz should be for 0.9.21 or 0.9.22 and up.

Francois Bissey <fbissey>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by wl (Posted a comment)
  • -email is unavailable- added by fbissey (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-08-26 wl Item GroupNone Incorrect behaviour
        StatusNone Fixed
        Assigned toNone wl
        Open/ClosedOpen Closed
        Planned ReleaseNone 2.6.1

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code