Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 6421 - aspell-0.50_pre20020730 not inheriting CXXFLAGS and is broken for gcc-3.2_pre
Summary: aspell-0.50_pre20020730 not inheriting CXXFLAGS and is broken for gcc-3.2_pre
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: Seemant Kulleen (RETIRED)
URL:
Whiteboard:
Keywords:
: 6265 (view as bug list)
Depends on:
Blocks:
 
Reported: 2002-08-13 15:39 UTC by Nicholas Wourms
Modified: 2003-02-04 19:42 UTC (History)
1 user (show)

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


Attachments
aspell.log (aspell.log,99.02 KB, text/plain)
2002-08-14 10:14 UTC, Nicholas Wourms
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nicholas Wourms 2002-08-13 15:39:55 UTC
I unmasked and was attempting to test aspell-0.50_pre20020730.  I am using
default-x86-2.0 with gcc-3.2_pre.  The first thing I noticed was the CXXFLAGS
from make.conf were not being used.  This was not a showstopper, but it isn't
optimized like it could be.  The next problem involved the new gcc-3.2 c++ ABI.
 Apparently this version of aspell doesn't like it.  Attempts to compile the
file modules/speller/phonet.cpp result in undeclared identifier errors:

line 177: `toupper' undeclared
line 270: `isdigit' undeclared

This reeks of a missing header file or deprecated namespace reference.  Not sure
which one it is yet.
Comment 1 Seemant Kulleen (RETIRED) gentoo-dev 2002-08-13 22:19:19 UTC
yes, it will NOT inherit your optimisations.  They break the ebuild. All you can
do is get over that bit.

As for the compile breaks, try the new snapshot and report.  And please don't
make things "blocker"s unless they really do break the installation of the
system.  That applies not to masked packages :P
Comment 2 Seemant Kulleen (RETIRED) gentoo-dev 2002-08-13 22:30:54 UTC
*** Bug 6265 has been marked as a duplicate of this bug. ***
Comment 3 Nicholas Wourms 2002-08-14 07:37:15 UTC
Well I am over it, I was never "on it" in the first place.  I didn't read the
ebuild, so I guess that's my fault.  However, I have run into quite a few
ebuilds which were not fixed to use optimizations.  So, I jumped the gun on that
one.  As for the status, I screwed up, because I was trying to set it to
"major", not blocker.  Guess I must of slipped up there as well.  Oh well, it's
not like I caused WWW III or anything...

As for the new snapshot, the good news is that it compiles fine.  However, parts
of the install phase seem to fail, but not fatally.  This causes portage to
install an unusable aspell.  Here are the problems [note: I unmerged the
previous aspell prior to building this one so that I could test a "fresh" install]:

1)When libtool attempts to relink libaspell against libaspell-common, ld
complains that it cannot find libaspell-common.  However, libaspell-common was
successfully installed.  It appears that libtool is not having the temporary
install dir path passed to -L when it attempts to relink.  Because this happens,
any libraries that attempt to relink in this package fail to do so.

2)After installing the manual, ebuild.sh complains that
/...//usr/share/doc/aspell-0.50_pre20020811 doesn't exist.  This would most
likely be due to the fact that the installer puts docs in
/...//usr/share/doc/aspell/.  

3)Compared to the previous edition, there were quite a few less headers
installed in /...//usr/include/...  It just seems suspect, but I'm not totally sure.

So that's what I observed.  YMMV, but I ran through the process 4 times to be
certain I wasn't missing anything.
Comment 4 Seemant Kulleen (RETIRED) gentoo-dev 2002-08-14 08:00:55 UTC
I've made some fixes to the ebuild, so please rsync (whenever your mirror
updates itself), and put it into a log so I can see these errors, please.

there is something interesting about the new aspell. it is supposed to encompass
both aspell and pspell, and you have to install a dictionary separately.  When I
did that, I got some weird behaviour.  Sylpheed-claws, for instance, doesn't
recognise the dictionary.
Comment 5 Nicholas Wourms 2002-08-14 09:41:08 UTC
What revision/timestamp will will the header on the new ebuild be?  Just so I
know the mirror got it.

As for the dictionaries, have you tried one of the dictionaries at:
http://savannah.gnu.org/download/aspell/dicts-cvs/?

Also, am I supposed to uninstall pspell?  What about the pspell-ispell package?
 I guess I can answer these questions through experimentation.

What bug number concerned the optimization errors?  I was thinking about trying
it out on the snapshot to see if it is still broken.
Comment 6 Seemant Kulleen (RETIRED) gentoo-dev 2002-08-14 09:58:04 UTC
# $Header:
/home/cvsroot/gentoo-x86/app-text/aspell/aspell-0.50_pre20020811.ebuild,v 1.2
2002/08/14 13:04:48 seemant Exp $


Also, I've put aspell-en in app-i18n, but it doesn't seem to work for me.  The
new aspell, btw, is a combination of pspell and aspell.  So, you can remove
pspell and pspell-ispell.  That's the dict from gnu.org, btw, which I was
referring to earlier.  See what you make of it, until I hear from Mr. Atkinson
Comment 7 Nicholas Wourms 2002-08-14 10:14:09 UTC
Created attachment 3072 [details]
aspell.log

This is the log transaction of the build and install.  It has installed a
little more this time around, but the relinking errors are still of some
concern.  Note also the inability to switch to the desired directory as
previously stated.
Comment 8 Seemant Kulleen (RETIRED) gentoo-dev 2002-08-14 11:26:18 UTC
nicholas,

bear with me, but what happens if you uncomment the lines in the ebuild:

inherit libtool


and then in src_compile, the line:
elibtoolize


do you still get those linking errors?
Comment 9 Nicholas Wourms 2002-08-14 11:42:11 UTC
Yes, that worked perfectly!  Still one minor issue with the ebuild.
I'm getting this upon installation:
make[2]: Nothing to be done for `install-exec-am'.
make[2]: Nothing to be done for `install-data-am'.
make[2]: Leaving directory `/var/tmp/portage/aspell-0.50_pre20020811/work/aspell-0.50-20020811'
make[1]: Leaving directory `/var/tmp/portage/aspell-0.50_pre20020811/work/aspell-0.50-20020811'
install: `man-text' is a directory
gzip: /var/tmp/portage/aspell-0.50_pre20020811/image/usr/share/doc/aspell-0.50_pre20020811/text/man-text: No such file or directory
strip:
/var/tmp/portage/aspell-0.50_pre20020811/image/usr/lib/libaspell-common-0.50.so

Just thought I'd point that out.  Thanks!
Comment 10 Nicholas Wourms 2002-08-14 11:49:52 UTC
I just tested aspell from the command line with the en dictionary, seems to work
fine there.  Perhaps there is a new way to interface it from external mail clients?