|Summary:||kde-base/kdelibs-3.5.5-r7 re-keywording request|
|Product:||Gentoo Linux||Reporter:||Diego Elio Pettenò (RETIRED) <flameeyes>|
|Component:||New packages||Assignee:||MIPS Porters <mips>|
|Package list:||Runtime testing required:||---|
|Bug Depends on:||152274|
Just a simple unicode text in hebrew with few errors for testing hspell
a newer simple file
Description Diego Elio Pettenò (RETIRED) 2006-10-25 13:37:09 UTC
Comment 1 Diego Elio Pettenò (RETIRED) 2006-10-25 13:41:24 UTC
Alon, we might need a version of hspell-1.0 that builds a .so, as it needs to use PIC mode for AMD64 most likely.
Comment 2 Diego Elio Pettenò (RETIRED) 2006-10-25 14:05:10 UTC
Created attachment 100484 [details, diff] hspell-1.0-shared.patch This patch adds support for libhspell.so, that will in turn allow kdelibs to build correctly against hspell on arches where PIC is needed for shared libraries (AMD64 and IA64 comes to mind). AMD64 and the rest of PIC-based arches, please wait till this is merged in -r1 for keywording, or kdelibs will be broken again.
Comment 3 Carsten Lohrke (RETIRED) 2006-10-25 14:42:14 UTC
Diego: The hspell2 patch doesn't solve the problem that aspell and ispell are automagically detected, when building the kspell2 plugins...
Comment 4 Diego Elio Pettenò (RETIRED) 2006-10-25 14:54:43 UTC
Uh, aspell is not automagically detected (--without-aspell works for me). ispell maybe, let me check twice, but it should be detected at _runtime_ which is a good kind of automagic dependency.
Comment 5 Diego Elio Pettenò (RETIRED) 2006-10-25 14:57:35 UTC
Checked twice: ispell support is *always* built, and ispell is detected at runtime if present, so there's no need to support it as useflag in kdelibs (really, *who* would be using ispell?). Aspell in KSpell2 will be enabled/disabled by the spell useflag, that by coincidence will also bring it in for the KSpell1 support (that is discovered automatically at runtime). HSpell in KSpell2 will be enabled/disabled by linguas_he plus spell useflag, that by coincidence will also bring it in for the KSpell1 support (that is discovered automatically at runtime). Which means that as it is, it works, no automagic linking dependencies.
Comment 6 Carsten Lohrke (RETIRED) 2006-10-25 16:52:38 UTC
Forget about the automagic dependencies popping into my mind seeing you readding kspell2 support. What I really had vaguely remembered was the former existing spell? || dependency issue, but since you force aspell on hebrew speaking users, this isn't an issue.
Comment 7 Diego Elio Pettenò (RETIRED) 2006-10-25 18:01:38 UTC
aspell is used for everything else but Hebrew, so it's just simpler to force it on for everybody, this way we solve the problem for good. Maybe users who only speak Hebrew and don't care about other languages' spellcheckers might want to get rid of aspell, but as that requires an higher complexity solution, I'd say we're good as we are :)
Comment 8 Markus Rothe (RETIRED) 2006-10-25 23:48:47 UTC
(PPC64) hspell does not like parallel building (MAKEOPTS="-j4"): [...] rm -f hebrew.wgz.lingsizes.tmp rm -f hebrew.wgz.prefixes gzip -9 < hebrew.wgz.prefixes.tmp >hebrew.wgz.prefixes /bin/sh: hebrew.wgz.prefixes.tmp: No such file or directory make: *** [hebrew.wgz.stems] Error 1
Comment 9 Diego Elio Pettenò (RETIRED) 2006-10-26 01:25:31 UTC
Created attachment 100512 [details, diff] hspell-1.0-shared.patch This patch works with --as-needed and fixes libhspell.a building too. Now building kdelibs to see if it works as expected.
Comment 10 Alon Bar-Lev (RETIRED) 2006-10-26 01:57:07 UTC
Thanks for all the help! I will look at it all today. Regarding the aspell issue, there is no problem in forcing aspell for all users.
Comment 11 Alex Bogak 2006-10-26 02:20:36 UTC
Created attachment 100517 [details] Just a simple unicode text in hebrew with few errors for testing hspell Diego, First, I got similar failure while installing on x86 when MAKEOPTS more than 1, just got it on different file. Second, I attached some simple paragraph for testing the hspell in the future upon your request. It has few errors in it. If there's any questions about it, just let me know.
Comment 12 Alex Bogak 2006-10-26 02:31:39 UTC
Created attachment 100518 [details] a newer simple file Previous file got corrupted
Comment 13 Diego Elio Pettenò (RETIRED) 2006-10-26 04:22:48 UTC
Thanks Alex :) For the ones testing, if your locale is UTF-8, just use "iconv -f UTF-16 file" to get the transcoded version. To run the hspell utility by hand, I've used the command "iconv -f UTF-16 -t ISO-8859-8 hebrew | hspell -c | iconv -f ISO-8859-8".. I suppose it works as it outputs something, but I cannot read it so I just hope it's correct ;)
Comment 14 Diego Elio Pettenò (RETIRED) 2006-10-26 05:24:11 UTC
I've just confirmed that hspell kspell2 plugin builds (and seems to work) fine with the patch.
Comment 15 Alex Bogak 2006-10-26 14:06:46 UTC
(In reply to comment #8) > (PPC64) > > hspell does not like parallel building (MAKEOPTS="-j4"): > > [...] > rm -f hebrew.wgz.lingsizes.tmp > rm -f hebrew.wgz.prefixes > gzip -9 < hebrew.wgz.prefixes.tmp >hebrew.wgz.prefixes > /bin/sh: hebrew.wgz.prefixes.tmp: No such file or directory > make: *** [hebrew.wgz.stems] Error 1 > Diego, 1. The same error with -j4 I have on AMD64. Sometime it installs, sometimes it doesn't. 2. With MAKEOPTS="-j1" install is successful in 100% Checked with the following: set temp PORT_LOGDIR location in /etc/make.conf run the following: for i in 1 2 3 4 5 6 7 8 9 10; do emerge hspell; done checked a log - all success. removed temp PORT_LOGDIR from /etc/make.conf 3. Given example file is successfully tested. I see some misses, but its hspell issue, not ebuild. Anything else I can help with?
Comment 16 Alex Bogak 2006-10-26 14:09:16 UTC
forgot to mention, all above is regarding hspell-1.0
Comment 17 Alon Bar-Lev (RETIRED) 2006-10-26 14:11:38 UTC
Use -j1... I will fix this in my next commit.
Comment 18 Alon Bar-Lev (RETIRED) 2006-10-26 18:51:49 UTC
I've committed somewhat larger (a lot) patch, to work with automake, continue of work started at upstream bugzilla. I hope upstream will merge this one as it work correctly. It also solves the parallel make issue, pkg-config issue and more. Please check it out.
Comment 19 Alex Bogak 2006-10-27 02:22:59 UTC
(In reply to comment #18) AMD64, hspell-1.0-r1 Compiles fine with -j4. Works fine.
Comment 20 Alon Bar-Lev (RETIRED) 2006-10-27 02:53:57 UTC
I have updated bug#152274, in order to add all kdelibs lost keywords into hspell.
Comment 21 Diego Elio Pettenò (RETIRED) 2006-10-27 05:37:44 UTC
Comment 22 Diego Elio Pettenò (RETIRED) 2006-10-27 05:50:58 UTC
Comment 23 Markus Rothe (RETIRED) 2006-10-27 05:57:17 UTC
Comment 24 Gustavo Zacarias (RETIRED) 2006-10-27 06:56:05 UTC
~sparc is back!
Comment 25 Guy Martin (RETIRED) 2006-11-24 06:09:59 UTC
This one is stable on hppa. Now dealing with -r6.
Comment 26 Diego Elio Pettenò (RETIRED) 2006-11-30 19:14:58 UTC
Going to unify with bug #155505.
Comment 27 Diego Elio Pettenò (RETIRED) 2006-11-30 19:15:20 UTC
*** Bug 155505 has been marked as a duplicate of this bug. ***
Comment 28 Jason Wever (RETIRED) 2006-12-17 19:06:01 UTC
Comment 29 Jeroen Roovers (RETIRED) 2007-01-21 22:43:52 UTC
Comment 30 Joshua Kinard 2007-06-03 03:32:46 UTC
-r10 stable on mips.