Bug 205401 – app-accessibility/festival-1.96_beta gcc 4.3 patch

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Ryan Hill (RETIRED) 2008-01-12 04:55:16 UTC]

attached is a patch from ubuntu to get festival to compile with GCC 4.3.

thanks. :)

[Comment 1 Ryan Hill (RETIRED) 2008-01-12 04:56:15 UTC]

Created attachment 140758 [details, diff]
festival-1.96_beta-gcc-4.3.patch

[Comment 2 William Hubbs 2008-01-12 17:49:58 UTC]

This is now applied in portage.  Thanks for the report. :-)

[Comment 3 Martin Jansa 2009-02-09 13:35:38 UTC]

I wasn't able to compile festival-1.96_beta-r1 with gcc-4.3.3 without adding <cstring> also to src/modules/MultiSyn/base_class/EST_TSimpleVector.cc (like in speech-tools-1.2.96_beta-gcc43-include.patch)

diff -uNr festival.orig/src/modules/MultiSyn/base_class/EST_TSimpleVector.cc festival/src/modules/MultiSyn/base_class/EST_TSimpleVector.cc
--- festival.orig/src/modules/MultiSyn/base_class/EST_TSimpleVector.cc  2009-02-09 13:27:28.000000000 +0100
+++ festival/src/modules/MultiSyn/base_class/EST_TSimpleVector.cc       2009-02-09 13:27:55.000000000 +0100
@@ -43,6 +43,7 @@
 #include "EST_TSimpleVector.h"
 #include "EST_matrix_support.h"
 #include <fstream>
+#include <cstring>
 #include "EST_cutils.h"

 template<class T> void EST_TSimpleVector<T>::copy(const EST_TSimpleVector<T> &a)

[Comment 4 William Hubbs 2009-02-09 13:55:15 UTC]

This should now be fixed.  Please wait for the change to propegate to
the mirrors, sync and try again.

Thanks,

William

[Comment 5 Martin Jansa 2009-02-09 14:00:14 UTC]

(In reply to comment #4)
> This should now be fixed.  Please wait for the change to propegate to
> the mirrors, sync and try again.
> 
> Thanks,
> 
> William

Yes, but I was trying to compile that ebuild -r1 from this morning already with speech-tools-1.2.96_beta-gcc43-include.patch, but it wasn't sufficient for me... so I added this include to festival source.

Thanks,

Martin

[Comment 6 William Hubbs 2009-02-09 14:18:47 UTC]

In reply to comment #5:

I fixed the issue in the ebuild at 13:50 gmt today (about 30-35 minutes ago),
so wait another 30 minutes from now, then sync and try to use the ebuild
in the tree again.

The patch you mention was being applied, just at the wrong point in the ebuild.

Sorry for not being clear about this in my original comment. :-)

William