Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 220403 - media-video/gxine-0.5.11: probably a threading problem
Summary: media-video/gxine-0.5.11: probably a threading problem
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Media-video project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-05-05 16:34 UTC by Rafał Mużyło
Modified: 2008-08-06 14:38 UTC (History)
1 user (show)

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


Attachments
patch to the current ebuild (build-fix.patch,1.15 KB, patch)
2008-08-04 15:14 UTC, Rafał Mużyło
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rafał Mużyło 2008-05-05 16:34:10 UTC
If I'm right, this is an upstream bug.
For quite a while now (actually so long, that I can't really tell when it began), I've been having a problem with gxine refusing to start video playback.
This can be caused by many reasons (well, I'm using ati-drivers, so video issues are plenty), it can be a problem with audio/video sync, cause often even if video playback doesn't go past first few frames, audio still plays for a few seconds.
However, I think it's actually may be caused by the fact, that I built spidermonkey with threadsafe flag.
This suspicion is due to the fact, that cxfe doesn't have that problem and also that when I go into preferences dialog of gxine, sometimes switching from one tab to another takes several seconds.
Comment 1 Patrick McLean gentoo-dev 2008-08-03 03:02:51 UTC
please try with media-video/gxine-0.5.503 and if it still happens, reopen this bug, or mark it fixed if it works.
Comment 2 Rafał Mużyło 2008-08-03 11:02:36 UTC
I'm reopening this bug to say, that while this problem seems to be fixed
(though only little testing done, so not fully confirmed yet),
the ebuild for gxine-0.5.903 is broken and unmergeable
(at least without turning off the sandbox).
First of all, the detection of spidermonkey has changed
and now while lib is still found, headers are not,
which causes compile to stop with an error.
Adding '--with-spidermonkey=/usr/include/js' to econf
fixes that, but while compile ends now successfully,
emerge stops here, because of sandbox violation:
open_wr:   /root/.xine/catalog.cache
Comment 3 Rafał Mużyło 2008-08-03 11:05:05 UTC
Forgot to add, the violation seems to happen during:
exec > gxine.desktop; cat gxine.desktop.in; /usr/bin/xine-list-1.1; echo
Comment 4 Rafał Mużyło 2008-08-03 12:42:37 UTC
I fixed the problem using following sed line:
-e '/^XINE_LIST=/s:XINE_LIST=.*$:XINE_LIST=:'

Now it merged correctly and it looks like the original
problem is really fixed (still not 100% sure about it).
Comment 5 Rafał Mużyło 2008-08-03 12:43:25 UTC
That sed line is for configue.ac, of course.
Comment 6 Bob Raitz 2008-08-04 07:50:28 UTC
(In reply to comment #5)
> That sed line is for configue.ac, of course.
> 
Forgive my ignorance, but exactly here am I to put that line? In the source or the ebuild?

Blessed be!
Pappy
Comment 7 Rafał Mużyło 2008-08-04 08:29:40 UTC
In the ebuild, in src_unpack.
But I decided to simply add eautoreconf there, too.
Comment 8 Rafał Mużyło 2008-08-04 08:34:52 UTC
@video team: shouldn't it be considered an upstream bug,
that during compile, program changes files that don't belong to it ?
I mean, it's not that obvious, why /root/.xine/catalog.cache
gets changed. It happens simply because xine-list-1.1 does
xine_init, which first reads user's .xine/catalog.cache,
then saves the new values.
Comment 9 Todd M. Hébert 2008-08-04 14:35:38 UTC
(In reply to comment #7)
> In the ebuild, in src_unpack.
> But I decided to simply add eautoreconf there, too.
> 

Any chance of showing what a patched ebuild would look like?
 I don't see mention of whether to add lines, or replace existing lines, or amend the existing lines.
Also, is:
Adding '--with-spidermonkey=/usr/include/js' to econf

also necessary.. and where does that go within the econf section under src_compile?

Thanks,

-Todd
Comment 10 Rafał Mużyło 2008-08-04 15:14:41 UTC
Created attachment 162208 [details, diff]
patch to the current ebuild

This is was I used.
Yeah, I know - rocket launcher on a fly,
but it worked for me so I don't care.
The patch mentioned in this patch is simply
something I picked up from gxine's mercurial,
that looked like it should be applied
(unrelated to these problems).
Comment 11 Patrick McLean gentoo-dev 2008-08-06 14:38:11 UTC
The sandbox violations and spidermonkey location should now be fixed in CVS. As to the threading issue, it looks like it's an upstream bug. Could you report it upstream at http://bugs.xine-project.org/