Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 84323 - [PATCH] inkscape wrong behavior with gtkspell.
Summary: [PATCH] inkscape wrong behavior with gtkspell.
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Marc Hildebrand (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-03-06 11:22 UTC by Simone Gotti (RETIRED)
Modified: 2005-05-24 14:27 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
files/inkscape-0.40-configure.ac.patch (inkscape-0.40-configure.ac.patch,514 bytes, patch)
2005-03-06 11:23 UTC, Simone Gotti (RETIRED)
Details | Diff
inkscape-0.40-r1-to-r2.ebuild.patch (inkscape-0.40-r1-to-r2.ebuild.patch,1005 bytes, patch)
2005-03-06 11:26 UTC, Simone Gotti (RETIRED)
Details | Diff
inkscape-0.40-r1-to-r2.ebuild.patch (inkscape-0.40-r1-to-r2.ebuild.patch,1.02 KB, patch)
2005-03-06 14:02 UTC, Simone Gotti (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simone Gotti (RETIRED) gentoo-dev 2005-03-06 11:22:24 UTC
inkscape has a check for gtkspell, and it's not selectable with a configure options. This is wrong as it will break portage compiling support for gtkspell if it's installed. I found this problem with another bug report made to the kde herd.

The right think to do is to use a use flag "spell" and fix configure to make it selectable.

I made a patch for "configure.ac" and a patch for the ebuild. I can't really test it as I have to compile a lot of deps but I don't have time. 
Please review.
Comment 1 Simone Gotti (RETIRED) gentoo-dev 2005-03-06 11:23:28 UTC
Created attachment 52798 [details, diff]
files/inkscape-0.40-configure.ac.patch

Add the --with-gtkspell configure flag and use it to enable/disable gtkspell
support.
Comment 2 Simone Gotti (RETIRED) gentoo-dev 2005-03-06 11:26:27 UTC
Created attachment 52799 [details, diff]
inkscape-0.40-r1-to-r2.ebuild.patch

The ebuild will need a revision bump to r2.

It adds a "spell" use flag, applies the configure.ac patch and runs "autoconf"
to create the new configure script.
Comment 3 Marco Squarcina 2005-03-06 12:59:40 UTC
Doesn't work here:

athlon64 -O2 -fweb -frename-registers -ftracer -pipe LDFLAGS=-Wl,-O1 host_alias=x86_64-pc-linux-gnu  --no-create --no-recursion
configure: WARNING: If you wanted to set the --build type, don't use --host.
    If a cross compiler is detected then cross compile mode will be used.
checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-pc-linux-gnu
checking for a BSD-compatible install... /bin/install -c
checking whether build environment is sane... yes
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking for x86_64-pc-linux-gnu-strip... x86_64-pc-linux-gnu-strip
checking for intltool >= 0.22... 0.31.1 found
checking for perl... /usr/bin/perl
checking for XML::Parser... ok
checking for iconv... /usr/bin/iconv
checking for msgfmt... /usr/bin/msgfmt
checking for msgmerge... /usr/bin/msgmerge
checking for xgettext... /usr/bin/xgettext
checking for style of include used by make... GNU
checking for x86_64-pc-linux-gnu-g++... x86_64-pc-linux-gnu-g++
checking for C++ compiler default output file name... a.out
checking whether the C++ compiler works... yes
checking whether we are cross compiling... no
checking for suffix of executables... 
checking for suffix of object files... o
checking whether we are using the GNU C++ compiler... yes
checking whether x86_64-pc-linux-gnu-g++ accepts -g... yes
checking dependency style of x86_64-pc-linux-gnu-g++... gcc3
checking for library containing strerror... none required
checking for x86_64-pc-linux-gnu-g++... (cached) x86_64-pc-linux-gnu-g++
checking whether we are using the GNU C++ compiler... (cached) yes
checking whether x86_64-pc-linux-gnu-g++ accepts -g... (cached) yes
checking dependency style of x86_64-pc-linux-gnu-g++... (cached) gcc3
checking for x86_64-pc-linux-gnu-gcc... x86_64-pc-linux-gnu-gcc
checking whether we are using the GNU C compiler... yes
checking whether x86_64-pc-linux-gnu-gcc accepts -g... yes
checking for x86_64-pc-linux-gnu-gcc option to accept ANSI C... none needed
checking dependency style of x86_64-pc-linux-gnu-gcc... gcc3
checking how to run the C++ preprocessor... x86_64-pc-linux-gnu-g++ -E
checking for egrep... grep -E
checking for ANSI C header files... yes
checking for a sed that does not truncate output... /bin/sed
checking for ld used by x86_64-pc-linux-gnu-gcc... /usr/x86_64-pc-linux-gnu/bin/ld
checking if the linker (/usr/x86_64-pc-linux-gnu/bin/ld) is GNU ld... yes
checking for /usr/x86_64-pc-linux-gnu/bin/ld option to reload object files... -r
checking for BSD-compatible nm... /usr/bin/x86_64-pc-linux-gnu-nm -B
checking whether ln -s works... yes
checking how to recognise dependent libraries... pass_all
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking how to run the C preprocessor... x86_64-pc-linux-gnu-gcc -E
checking dlfcn.h usability... yes
checking dlfcn.h presence... yes
checking for dlfcn.h... yes
checking how to run the C++ preprocessor... x86_64-pc-linux-gnu-g++ -E
checking for x86_64-pc-linux-gnu-g77... no
checking for x86_64-pc-linux-gnu-f77... no
checking for x86_64-pc-linux-gnu-xlf... no
checking for x86_64-pc-linux-gnu-frt... no
checking for x86_64-pc-linux-gnu-pgf77... no
checking for x86_64-pc-linux-gnu-fort77... no
checking for x86_64-pc-linux-gnu-fl32... no
checking for x86_64-pc-linux-gnu-af77... no
checking for x86_64-pc-linux-gnu-f90... no
checking for x86_64-pc-linux-gnu-xlf90... no
checking for x86_64-pc-linux-gnu-pgf90... no
checking for x86_64-pc-linux-gnu-epcf90... no
checking for x86_64-pc-linux-gnu-f95... no
checking for x86_64-pc-linux-gnu-fort... no
checking for x86_64-pc-linux-gnu-xlf95... no
checking for x86_64-pc-linux-gnu-ifc... no
checking for x86_64-pc-linux-gnu-efc... no
checking for x86_64-pc-linux-gnu-pgf95... no
checking for x86_64-pc-linux-gnu-lf95... no
checking for x86_64-pc-linux-gnu-gfortran... no
checking for g77... no
checking for f77... no
checking for xlf... no
checking for frt... no
checking for pgf77... no
checking for fort77... no
checking for fl32... no
checking for af77... no
checking for f90... no
checking for xlf90... no
checking for pgf90... no
checking for epcf90... no
checking for f95... no
checking for fort... no
checking for xlf95... no
checking for ifc... no
checking for efc... no
checking for pgf95... no
checking for lf95... no
checking for gfortran... no
checking whether we are using the GNU Fortran 77 compiler... no
checking whether  accepts -g... no
checking the maximum length of command line arguments... 32768
checking command to parse /usr/bin/x86_64-pc-linux-gnu-nm -B output from x86_64-pc-linux-gnu-gcc object... ok
checking for objdir... .libs
checking for x86_64-pc-linux-gnu-ar... x86_64-pc-linux-gnu-ar
checking for x86_64-pc-linux-gnu-ranlib... x86_64-pc-linux-gnu-ranlib
checking for x86_64-pc-linux-gnu-strip... (cached) x86_64-pc-linux-gnu-strip
checking for correct ltmain.sh version... no

*** Gentoo sanity check failed! ***
*** libtool.m4 and ltmain.sh have a version mismatch! ***
*** (libtool.m4 = 1.5.10, ltmain.sh = 1.5.6) ***

Please run:

  libtoolize --copy --force

if appropriate, please contact the maintainer of this
package (or your distribution) for help.

make: *** [config.status] Error 1

!!! ERROR: media-gfx/inkscape-0.40-r2 failed.
!!! Function gnome2_src_compile, Line 40, Exitcode 2
!!! compile failure
!!! If you need support, post the topmost build error, NOT this status message.
Comment 4 Simone Gotti (RETIRED) gentoo-dev 2005-03-06 14:02:54 UTC
Created attachment 52815 [details, diff]
inkscape-0.40-r1-to-r2.ebuild.patch

Better ebuild. Forgot to update the libtool half part after the autoconf.
Comment 5 Marco Squarcina 2005-03-07 02:15:54 UTC
OK! This one works fine. Without "spell" use flag and without gtkspell it runs. Adding spell support:

darkstar inkscape # USE="spell" emerge -vp inkscape

These are the packages that I would merge, in order:

Calculating dependencies ...done!
[ebuild  N    ] app-text/gtkspell-2.0.4-r1  -doc 0 kB 
[ebuild   R   ] media-gfx/inkscape-0.40-r2  -debug -doc -gnome (-mmx) +spell* 0 kB [1]

And compiles fine (and works) also in this case

Cheers
Comment 6 Marc Hildebrand (RETIRED) gentoo-dev 2005-05-24 14:27:40 UTC
Hi!
I fixed this for inkscape-0.41.
Please test and open a new bug if there are any issues.
Thanks for the patches.
Cheers,

Marc.