Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 451730 - =app-text/stardict-3.0.4 : Version bump
Summary: =app-text/stardict-3.0.4 : Version bump
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-13 14:11 UTC by Agostino Sarubbo
Modified: 2014-05-09 14:17 UTC (History)
4 users (show)

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


Attachments
ebuild for 3.0.4 (stardict-3.0.4.ebuild,4.49 KB, text/plain)
2013-06-12 06:58 UTC, Yixun Lan
Details
patch to fix g_thread_xx err, due to >glib-2.31 (stardict-3.0.4-fix-glib-thread.patch,1.41 KB, patch)
2013-06-12 06:59 UTC, Yixun Lan
Details | Diff
ebuild diff, when compare with 3.0.3-r2 (stardict-3.0.4-diff.patch,3.19 KB, patch)
2013-06-12 07:00 UTC, Yixun Lan
Details | Diff
metadata.xml file, add IUSE and maintainer info (metadata.xml,1.67 KB, text/xml)
2013-06-12 07:02 UTC, Yixun Lan
Details
build.log (build.log,149.94 KB, text/x-log)
2013-08-24 19:38 UTC, Andrew Savchenko
Details
stardict-3.0.4-gmodule-underlinking.patch (stardict-3.0.4-gmodule-underlinking.patch,341 bytes, patch)
2013-08-24 19:39 UTC, Andrew Savchenko
Details | Diff
stardict-3.0.4.ebuild.patch (stardict-3.0.4.ebuild.patch,3.24 KB, patch)
2013-08-24 19:39 UTC, Andrew Savchenko
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2013-01-13 14:11:39 UTC
$summary
Comment 1 Yixun Lan archtester gentoo-dev 2013-06-12 06:58:46 UTC
Created attachment 350804 [details]
ebuild for 3.0.4

hey, here is my first attempt to come up with a new 3.0.4 version, for review..

1. add user patch support
2. add more dict IUSEs
3. fix g_thread_{init, creat} err due to >glib-2.31
4. USE=gucharmap requires --enable-deprecations 

problem:
fail to compile festival and gpe, but I think these should report to upstream.
I'm not sure whether to keep those IUSE and mask them in package.use.mask, or simply remove it. the former may allow us much more easily to accept patches from users..

Lan Yixun (dlan) (13):
      app-text/stardict: init, import original ebuild from official portage tree
      app-text/stardict: metadata, add myself as maintainer
      app-text/stardict: bumped version, only copy from 3.0.3, no changes
      app-text/stardict: modify according to new version (3.0.4)
      app-text/stardict: convert to src_prepare
      app-text/stardict: user patch support
      app-text/stardict: update configure options
      app-text/stardict: enable a few default dictionaries
      app-text/stardict: disable festival and gpe, it's broken, fail to compile
      app-text/stardict: add more dict related IUSEs..
      app-text/stardict: glib patches, fix thread error
      app-text/stardict: fix deperacted macro problem
      app-text/stardict: manifest.xml generated

I've put the full commit history in my overlay, branch "stardict"
(use "layman -a dlan && git checkout origin/stardict")
Comment 2 Yixun Lan archtester gentoo-dev 2013-06-12 06:59:27 UTC
Created attachment 350806 [details, diff]
patch to fix g_thread_xx err, due to >glib-2.31
Comment 3 Yixun Lan archtester gentoo-dev 2013-06-12 07:00:44 UTC
Created attachment 350808 [details, diff]
ebuild diff, when compare with 3.0.3-r2
Comment 4 Yixun Lan archtester gentoo-dev 2013-06-12 07:02:39 UTC
Created attachment 350810 [details]
metadata.xml file, add IUSE and maintainer info
Comment 5 Andrew Savchenko gentoo-dev 2013-08-24 19:38:51 UTC
Created attachment 356884 [details]
build.log

Hello,

this packages fails to build with dev-libs/glib-2.36.4 due to underlinking issue:

libtool: link: i686-pc-linux-gnu-g++ -march=native -m32 -O2 -funswitch-loops -fpredictive-commoning -fgcse-after-reload -ftree-loop-im -fweb -frename-registers -fomit-frame-pointer -mfpmath=sse -pipe -frecord-gcc-switches -fvisibility-inlines-hidden -march=native -m32 -O2 -funswitch-loops -fpredictive-commoning -Wl,-O1 -Wl,--as-needed -pipe -o stardict stardict.o mainwin.o conf.o skin.o prefsdlg.o dictmanagedlg.o pluginmanagedlg.o floatwin.o readword.o selection.o splash.o gtktextviewpango.o pangoview.o articleview.o class_factory.o inifile.o x11_iskeyspressed.o docklet.o gtk_iskeyspressed.o progresswin.o tray.o dictmanage.o eggaccelerators.o globalhotkeys.o hotkeyeditor.o tomboykeybinder.o desktop.o log.o cmdlineopts.o -pthread  lib/.libs/libstardict.a -lSM -lICE -lsigc-2.0 -lgtk-x11-2.0 -lgdk-x11-2.0 -lpangocairo-1.0 -latk-1.0 -lcairo -lgdk_pixbuf-2.0 -lgio-2.0 -lpangoft2-1.0 -lpango-1.0 -lgobject-2.0 -lfreetype -lfontconfig -lgthread-2.0 -lglib-2.0 -lz -lX11 -pthread
/usr/lib/gcc/i686-pc-linux-gnu/4.7.3/../../../../i686-pc-linux-gnu/bin/ld: lib/.libs/libstardict.a(pluginmanager.o): undefined reference to symbol 'g_module_symbol'

This can be fixed by linking with libgmodule-2.0.so. It is used in dict/src/lib/pluginmanager.cpp but not linked to.

Patch proposed below fixes this.
Comment 6 Andrew Savchenko gentoo-dev 2013-08-24 19:39:29 UTC
Created attachment 356886 [details, diff]
stardict-3.0.4-gmodule-underlinking.patch

Fix underlinking with gmodule-2.0.
Comment 7 Andrew Savchenko gentoo-dev 2013-08-24 19:39:59 UTC
Created attachment 356888 [details, diff]
stardict-3.0.4.ebuild.patch

Updated ebuild (just underlinking patch added).
Comment 8 Samuli Suominen (RETIRED) gentoo-dev 2014-02-08 07:27:42 UTC
Comment on attachment 350808 [details, diff]
ebuild diff, when compare with 3.0.3-r2

Some major issues with this patch, can't go in as-is

The whole purpose is to avoid deprecated dependencies, such as app-text/scrollkeeper which is a relic from old GNOME2 releases
which got replaced long ago by app-text/rarian.

No point in adding USE flags for options that don't bring in new dependencies, lets just let them be auto enabled on their own.

No way libsigc++ is only a build-time DEPEND. How could it be when it's a shared library, and likely if it's used, stardict links against the library and then, the dependency would belong to RDEPEND and DEPEND, in this case I see COMMON_DEPEND is used in the ebuild.
Also libsigc++ dependency needs a SLOT, either :1.2 or :2
Comment 9 Samuli Suominen (RETIRED) gentoo-dev 2014-02-08 07:28:52 UTC
Comment on attachment 356888 [details, diff]
stardict-3.0.4.ebuild.patch

seems like the problems from my last comment got forward ported here, obsoleting this attachement too
Comment 10 Samuli Suominen (RETIRED) gentoo-dev 2014-05-09 14:17:36 UTC
3.0.4 in portage w/ the underlinking issue fixed