Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 46657 - gtkspell-2.0.10 available
Summary: gtkspell-2.0.10 available
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: Lowest enhancement (vote)
Assignee: Gentoo Linux Gnome Desktop Team
Depends on:
Blocks: 83418
  Show dependency tree
Reported: 2004-04-02 19:48 UTC by Don Seiler (RETIRED)
Modified: 2005-08-04 15:12 UTC (History)
7 users (show)

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

gtkspell-2.0.6.ebuild (gtkspell-2.0.6.ebuild,984 bytes, text/plain)
2004-06-27 17:46 UTC, Don Seiler (RETIRED)
gtkspell-2.0.7-enchant.patch (gtkspell-2.0.7-enchant.patch,15.42 KB, patch)
2004-10-13 18:18 UTC, Ed Catmur
Details | Diff
gtkspell-2.0.7.ebuild (gtkspell-2.0.7.ebuild,1.06 KB, text/plain)
2004-10-13 18:19 UTC, Ed Catmur
gtkspell-2.0.9.ebuild (gtkspell-2.0.9.ebuild,868 bytes, text/plain)
2005-04-05 15:50 UTC, Yaakov S
2.0-gtk-doc.patch (2.0-gtk-doc.patch,2.22 KB, patch)
2005-04-05 15:50 UTC, Yaakov S
Details | Diff
gtkspell-3.0.0_pre20050303.ebuild (gtkspell-3.0.0_pre20050303.ebuild,967 bytes, text/plain)
2005-04-05 15:54 UTC, Yaakov S
3.0-gtk-doc.patch (3.0-gtk-doc.patch,2.39 KB, patch)
2005-04-05 15:56 UTC, Yaakov S
Details | Diff
3.0-gentoo-SLOT.patch (3.0-gentoo-SLOT.patch,3.31 KB, patch)
2005-04-05 16:02 UTC, Yaakov S
Details | Diff
Patches enchat 2.0.10 to use enchant (enchant-new-gtkspell.diff,8.68 KB, patch)
2005-05-23 10:11 UTC, Drew "DanteAliegri" Ogle
Details | Diff
GtkSpell 2.0.11 (gtkspell-2.0.11.ebuild,573 bytes, text/plain)
2005-07-22 18:50 UTC, Yaron Tausky
Fixed patch (gtkspell-2.0.10-enchant.patch,8.69 KB, patch)
2005-07-31 15:36 UTC, Yaron Tausky
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Don Seiler (RETIRED) gentoo-dev 2004-04-02 19:48:32 UTC
FYI.  It's got some nice changes that I became aware of when trying to actually work on an app using it.
Comment 1 Don Seiler (RETIRED) gentoo-dev 2004-05-03 07:39:12 UTC
I assume that you'll want this patched with the enchant patch so we aren't taking a step backwards from 2.0.4-r1.  I'll try and get the patch to work and test thing s out and post an ebuild.
Comment 2 foser (RETIRED) gentoo-dev 2004-05-03 07:57:33 UTC
iirc there's a cvs branch enchant enabled
Comment 3 Don Seiler (RETIRED) gentoo-dev 2004-05-03 08:03:21 UTC
Yes that is the gtkspell3 module.  It has some other feature changes as well.  They are sorting some things out before releasing it.
Comment 4 Lukasz Strzygowski (RETIRED) gentoo-dev 2004-06-06 14:28:01 UTC
Have you made any progress in this? I need this version as dependency.
Comment 5 Don Seiler (RETIRED) gentoo-dev 2004-06-06 18:27:12 UTC
No I haven't.  The 2.0.4 enchant patch does not fit into 2.0.5 and I don't have time to try and manually patch it and get a new diff.

Is it not a possiblity to just release a non-enchant-ized 2.0.5?
Comment 6 foser (RETIRED) gentoo-dev 2004-06-08 03:20:36 UTC
I think thats a regression, so not a good idea. I wonder why you need 2.0.5 lucas, it's API stable isn't it .. older versions likely would do ?

A possible solution would be to take a snapshot from the enchant enhanced tree.

Comment 7 Lukasz Strzygowski (RETIRED) gentoo-dev 2004-06-11 13:03:28 UTC
foser: a newer version of net-im/kf depends on it, but i see that after patching configure script, kf works also with the older version. 

rizzo: by the way, if you have no objection, i'll add myself to metadata.xml of net-im/kf as a maintainer.
Comment 8 Don Seiler (RETIRED) gentoo-dev 2004-06-11 13:38:43 UTC
Lukas feel free to take over net-im/kf.
Comment 9 Don Seiler (RETIRED) gentoo-dev 2004-06-27 17:30:10 UTC
Now gtkspell-2.0.6 is out.

foser I don't think of it as a regression, since the enchant stuff isn't even in their gtkspell2 tree.  It was added by us.  From an API point of view nothing is lost if gtkspell uses aspell vs enchant anyway, and no apps should expect it since gtkspell doesn't come with it.
Comment 10 Don Seiler (RETIRED) gentoo-dev 2004-06-27 17:46:51 UTC
Created attachment 34310 [details]

FWIW, here is an ebuild for 2.0.6, based off of 2.0.4 without the patches, and
including support for internationalization with gettext, which is new in 2.0.6.
Comment 11 foser (RETIRED) gentoo-dev 2004-08-06 11:33:14 UTC
that would be a regression afaic, since this version isn't really needed i'd rather wait for an enchant enabled release.
Comment 12 Ed Catmur 2004-10-13 18:18:24 UTC
Created attachment 41774 [details, diff]

Patch to enable enchant in gtkspell-2.0.7. Please test and report back.


$ cvs -z3 co -r
gtkspell_2_0_7 gtkspell2
$ cvs -z3 co
$ mv gtkspell3/ gtkspell3/
$ grep -rIl gtkspell-3.0 gtkspell3 | xargs sed -i -s -e
$ sed -i -e 's:3.0.0cvs:2.0.7:' gtkspell3/
$ diff -rupdN -x CVS -x COPYING -x .cvsignore -x ChangeLog gtkspell2 gtkspell3

The playing around with .pc files and versions is necessary to ensure that this
patched gtkspell still fills the 2.0 slot pkgconfig-wise.

This may become a problem if and when gtkspell3 diverges API-wise from
gtkspell2 - this has not happened yet; the only change to headers is licensing
(GPL to LGPL).

Changes to ebuild:
  * add enchant dep
  * license is now LGPL-2.1 (!!)
  * apply patch, autoconf is already in ebuild

I ran up against bug 58161 (.so not being appended to solibs): if this is a
general problem (and not a problem with my system) then run libtoolize -f ||
die before autoconf.
Comment 13 Ed Catmur 2004-10-13 18:19:28 UTC
Created attachment 41775 [details]
Comment 14 Peter Johanson (RETIRED) gentoo-dev 2005-01-26 09:02:15 UTC
Just a note that the newer tomboy releases prefer to have at least gtkspell-2.0.9, since there is some fix in 2.0.9 that they use. I can dig up details of what exactly was fixed if people would like.
Comment 15 Don Seiler (RETIRED) gentoo-dev 2005-01-26 10:33:08 UTC
You will be waiting for quite a while for an enchant-enabled release.  I've been speaking with noif (the gtkspell developer) probably every week about it since I'm waiting for it to work on gaim's spell checking methods.  There is much that needs discussion and decision before he'll even venture a guess at an ETA for gtkspell3.

Restating my desire to just abandon the enchant patch and add gtkspell-2.0.8 to portage.  The enchant patch is not supported in gtkspell2 upstream nor here.
Comment 16 Don Seiler (RETIRED) gentoo-dev 2005-02-24 11:48:49 UTC
Resistance is futile.
Comment 17 John Nilsson 2005-03-03 09:04:38 UTC
Just putting my vote for the enchantless release. In generall Gentoo should aim for vanilla releases as much as possible.
foser: Why did you add the patch? In what way would it be a regression to remove it. Is there any pacakge depending on the patch?
Comment 18 Yaakov S 2005-04-05 09:14:00 UTC
I'm not a dev, but what happened to this part of the Dev Handbook:

> Try to not make ebuilds preform unnecessary steps. Packaging unsupported patches
> as an "addition" is a bad idea unless they are thoroughly tested by you, widely 
> used, and audited for security vulnerabilities.

In the end, the gtkspell enchant patch is not supported upstream, not supported here, apparently not supported by enchant[1] itself (the apparent source of our original patch, but it hasn't been updated since Dec. 2003), and I wonder if other distros support it either (debian[2] doesn't, and googling didn't turn up much else).

Bottom line: let's admit that we can't support this patch -- which we haven't for over a year -- and go back to the vanilla gtkspell.  If someone is really interested/bored/whatever, then make a separate ebuild for gtkspell-3 (preferably SLOTted), and maybe we can help the upstream devs with gtkspell3 more than we will by getting stuck on a year-and-a-half old patch.

Comment 19 Yaakov S 2005-04-05 15:50:36 UTC
Created attachment 55405 [details]

OK, so I went ahead and did it myself, and while doing so found a packaging bug
which affects gtkspell2 as well..  Here's an ebuild for a vanilla gtkspell2,
with a patch for gtk-doc detection and installation of the .devhelp file.
Comment 20 Yaakov S 2005-04-05 15:50:54 UTC
Created attachment 55406 [details, diff]
Comment 21 Yaakov S 2005-04-05 15:54:35 UTC
Created attachment 55407 [details]

And here's an ebuild for gtkspell3 from cvs.  I just checked out the code from
cvs and tarballed the module directory as-is; the date is based on the most
recently changed files.  This needs two patches, see below.
Comment 22 Yaakov S 2005-04-05 15:56:51 UTC
Created attachment 55408 [details, diff]

This does the same thing as the 2.0-gtk-doc.patch, but for the cvs sources.
Comment 23 Yaakov S 2005-04-05 16:02:32 UTC
Created attachment 55409 [details, diff]

This patch makes sure that everything is parallel-installable, to allow for the
separate ebuild SLOT.

In the end, I don't know whether this should be imported to the tree (p.mask'ed
of course) or not, but maybe someone will find it helpful.
Comment 24 Yaakov S 2005-04-05 16:06:05 UTC
Re comment 12: the COPYING file in the CVS sources is still GPL-2.
Comment 25 Yaakov S 2005-04-05 16:09:09 UTC
Forgot to mention that enchant needs to be bumped (manually atm) to 1.1.6 for the 3.0 ebuild; it requires an API introduced in that version.  A simple rename and digest wfm.
Comment 26 Yaakov S 2005-04-06 06:38:42 UTC
Actually, an enchant bump was already requested in bug 87370.
Comment 27 Don Seiler (RETIRED) gentoo-dev 2005-04-27 07:27:53 UTC
Comment #18 said it perfectly for me.
Comment 28 Drew "DanteAliegri" Ogle 2005-05-23 10:10:10 UTC
Here is a rework of Dom L.'s enchanting of gtkspell.
(I've emailed this to dom as well.)

Hopefully I can get 2.0.10 in my desktop now ;)
Comment 29 Drew "DanteAliegri" Ogle 2005-05-23 10:11:50 UTC
Created attachment 59633 [details, diff]
Patches enchat 2.0.10 to use enchant

Said patch.
Comment 30 Don Seiler (RETIRED) gentoo-dev 2005-05-24 07:44:29 UTC
(In reply to comment #29)
> Created an attachment (id=59633) [edit]
> Patches enchat 2.0.10 to use enchant

I'd rather see our gtkspell2 package purged of the enchant taint.
Comment 31 Drew "DanteAliegri" Ogle 2005-05-24 11:04:03 UTC
well, for what it's worth the patch is pretty simple.
The only two files of note that get patched are and gtkspell.c

All I really want is gtkspell 2.0.10 in my portage.

If needed, I'll make sure the patch applies to all the gtkspell2s.
Comment 32 Drew "DanteAliegri" Ogle 2005-05-26 09:25:20 UTC
Don, if I were to make an ebuild option for using enchant or not, would that
satisfy everyone and allow gtkspell 2.0.10 to go into portage?
Comment 33 Yaron Tausky 2005-07-22 16:45:00 UTC
gtkspell 2.0.11 is now available.
Comment 34 Yaron Tausky 2005-07-22 18:50:30 UTC
Created attachment 64099 [details]
GtkSpell 2.0.11

Crafted a GtkSpell 2.0.11 ebuild, no support for Enchant.
Please, whatever you just DECIDE already, GtkSpell 2.0.4 was released in the
beginning of 2003!
Comment 35 foser (RETIRED) gentoo-dev 2005-07-31 10:49:17 UTC
gtkspell-2.0.11 has been added with the patch from drew (and a minor bugfix to it)

please test
Comment 36 Yaron Tausky 2005-07-31 15:22:14 UTC
Fails to compile for me with this error:
../gtkspell/.libs/ undefined reference to `enchant_dict_add_to_pwl'

However, I believe that to fix that we should just replace this function (in the
patch) with enchant_dict_add_to_personal (PWL stands for personal word list).
Comment 37 Yaron Tausky 2005-07-31 15:36:37 UTC
Created attachment 64834 [details, diff]
Fixed patch
Comment 38 Yaron Tausky 2005-07-31 17:01:22 UTC
Okay, sorry, I was too quick to jump to conclusions. Here's the deal:
enchant_dict_add_to_personal was deprecated in enchant-1.1.6 in favour of
enchant_dict_add_to_pwl (which the patch uses). I think that gtkspell-2.0.11
should just depend on >=enchant-1.1.6, and that enchant-1.1.6 should be marked
stable (current stable version for x86 is 1.1.5).
Comment 39 foser (RETIRED) gentoo-dev 2005-08-01 04:29:14 UTC
I updated the enchant dep in gtkspell, thanks for catching that.
Comment 40 Yaron Tausky 2005-08-04 15:12:11 UTC
Works perfectly for me, on x86.