Bug 174942 - app-editors/gedit-2.18.0 doesn't build with USE="-spell"
Bug#: 174942 Product:  Gentoo Linux Version: unspecified Platform: x86
OS/Version: Linux Status: RESOLVED Severity: enhancement Priority: P2
Resolution: FIXED Assigned To: gnome@gentoo.org Reported By: flaschenkopp@gmx.de
Component: GNOME
URL: 
Summary: app-editors/gedit-2.18.0 doesn't build with USE="-spell"
Keywords:  
Status Whiteboard: 
Opened: 2007-04-17 17:27 0000
Description:   Opened: 2007-04-17 17:27 0000
When emerging app-editors/gedit-2.18.0 with USE="-spell", econf is failing if
there is no app-text/enchant present on the target system.
Otherwise it seems, if it were present the package would be built with support
for spelling, even with USE="-spell".

Reproducible: Always

Steps to Reproduce:
1. have no app-text/enchant on target system
2. emerge app-text/gedit-2.18.0 with USE="-spell"
3. 

Actual Results:  
checking for xgettext... /usr/bin/xgettext
checking for ENCHANT... no
configure: error: Enchant library not found or too old. Use --disable-spell to
build without spell plugin.

!!! Please attach the following file when filing a report to bugs.gentoo.org:
!!! /var/tmp/portage/app-editors/gedit-2.18.0/work/gedit-2.18.0/config.log

!!! ERROR: app-editors/gedit-2.18.0 failed.
Call stack:
  ebuild.sh, line 1614:   Called dyn_compile
  ebuild.sh, line 971:   Called qa_call 'src_compile'
  ebuild.sh, line 44:   Called src_compile
  ebuild.sh, line 1304:   Called gnome2_src_compile
  gnome2.eclass, line 70:   Called gnome2_src_configure
  gnome2.eclass, line 66:   Called econf '--disable-gtk-doc'
  ebuild.sh, line 577:   Called die

!!! econf failed
!!! If you need support, post the topmost build error, and the call stack if
relevant.
!!! A complete build log is located at
'/var/tmp/portage/app-editors/gedit-2.18.0/temp/build.log'.



We should follow the advice to use --disable-spell if support for spelling is
not wanted.

------- Comment #1 From labor_ratte 2007-04-17 17:30:42 0000 -------
Created an attachment (id=116532) [details]
patch to take care of the spell USE flag

The ebuild modified with this patch works as expected.

------- Comment #2 From Rémi Cardona 2007-04-17 18:32:29 0000 -------
Thanks for reporting this. Patched and committed in CVS. :)

FYI, here's what I actually committed (if you ever feel like patching other
ebuilds) :

+G2CONF="${G2CONF} \
+       $(use_enable spell)"
+