Re: Help with an old regression in text rendering needed

On Fri, 2016-07-29 at 10:34 +0200, Milos Sramek wrote:
> Thank you, Caolán, for your suggestions.
>
>
> The first thing I've tried was to compile the current version with
> and without harfbuzz using
> ./autogen.sh --with-harfbuzz=no
> The result was the same (bad).
>
> I am, however, not sure if harfbuzz was really switched off

Yeah, you can't turn off harfbuzz under Linux, the option is only to
enable turning it *on* for other platforms since it became the only
implementation under Linux.

> I've tried to investigate it bit deeper:
>
> revision mentioned:
> bff8fa9 - (2013-05-06 11:08:29 +0200) Enable HarfBuzz by default -
> Khaled Hosny
>
> The last good revision according to my bibisection
> f0393d7 - (2013-05-06 16:54:53 +0200) [harfbuzz] Fix text width
> calculation, 3rd try - Khaled Hosny

Note that the very next commit after f0393d7 is actually bff8fa9, so
you can probably short circuit bisecting and jump straight
between bff8fa9 and f0393d7

> The first bad revision according to my bibisection
> 48ad2f6 - (2013-05-06 12:48:23 +0200) fdo#46808, Convert some code to
> getProcessComponentContext
>
> So, they really are from the same day.



> I unfortunately cannot compile these revisions, maybe because my
> compiter is too new (gcc version 5.4.0 20160609 (Ubuntu
> 5.4.0-6ubuntu1~16.04.1)). I happens because of std::auto_ptr:
> /workdir/unxlngx6.pro/UnpackedTarball/boost/boost/smart_ptr/scoped_pt
> r.hpp:66:31:
> warning: ‘template<class> class std::auto_ptr’ is deprecated
> [-Wdeprecated-declarations]

That's only a warning. So unless you have used --enable-werror to makes
warnings fatal then that warning shouldn't be a problem. But seeing as
f0393d7 built, presumably bff8fa9 (because its the next commit) builds
as well for comparison.

C.