Bug 426402 - remove extra copy of hunspell inside enchant
Summary: remove extra copy of hunspell inside enchant
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: enchant
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Marc Maurer
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-12-20 19:59 UTC by Caolan McNamara
Modified: 2008-05-10 15:46 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-01-26 20:19:21 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
patch (4.05 KB, patch)
2007-12-20 19:59 UTC, Caolan McNamara
no flags Details | Diff

Description Caolan McNamara 2007-12-20 19:59:43 UTC
I see that enchant has a full copy of hunspell in it, sounds like a good idea to
link against -lhunspell instead
(i.e. http://bugzilla.abisource.com/show_bug.cgi?id=11321)

As part of http://fedoraproject.org/wiki/Releases/FeatureDictionary I'd also
like to split enchant into bits that link against aspell and those that don't.

i.e. complete patch against our .spec attached.

I can take care of this if you want to just reassign it back to me.

Comment 1 Caolan McNamara 2007-12-20 19:59:43 UTC
Created attachment 290185 [details]
patch

Comment 2 Kevin Kofler 2008-01-08 04:33:12 UTC
Ping? Can this please be applied? Duplicate copies of system libraries suck.

Comment 3 Caolan McNamara 2008-01-08 09:25:20 UTC
Dom committed a probably better final fix than the one submitted to him to the
enchant cvs/svn in this area FWIW

Comment 4 Caolan McNamara 2008-01-26 20:19:21 UTC
Changes checked in and built

Comment 5 Fryderyk Dziarmagowski 2008-05-10 15:46:02 UTC
This patch is incomplete. myspell_checker.cpp shouldn't include local
hunspell.hxx header.


Note You need to log in before you can comment on or make changes to this bug.