Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 73297

Summary: beep-media-player playlist double click causes abort
Product: Gentoo Linux Reporter: postmodern <brodigan>
Component: Current packagesAssignee: Tony Vroon (RETIRED) <chainsaw>
Status: RESOLVED FIXED    
Severity: normal CC: sound
Priority: High    
Version: unspecified   
Hardware: AMD64   
OS: Linux   
URL: http://www.sosdg.org/~larne/bugs/show_bug.cgi?id=214
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: patch to fix double/triple click issue with beep/playlistwin.c
patch to beep-media-player-0.9.7.ebuild to apply the noclick-resize patch

Description postmodern 2004-12-03 21:41:22 UTC
When running beep-media-player with the playlist open, if the user double clicks on the lower right hand corner of the playlist beep-media-player exists with the error message on the console running X of

Gdk-ERROR **: file gdkwindow-x11.c: line 5129 (create_moveresize_window): assertion failed: (mv_resize->moveresize_emulation_window == NULL)
aborting...

Reproducible: Always
Steps to Reproduce:
1. double click on the lower right corner of the playlist
2.
3.

Actual Results:  
crash

Expected Results:  
not-crashing.
Comment 1 Tony Vroon (RETIRED) gentoo-dev 2004-12-04 04:21:00 UTC
I can't reproduce your problem here. I get a popup 0x0 and the application does not quit.

Please provide emerge info and the beep-media-player version you're running.
Comment 2 postmodern 2004-12-04 07:01:41 UTC
beep-media-player 0.9.7_rc2-r3

Gentoo Base System version 1.4.16
Portage 2.0.51-r3 (default-linux/amd64/2004.3, gcc-3.4.3, glibc-2.3.4.20040808-r1, 2.6.9-gentoo-r6 x86_64)
=================================================================
System uname: 2.6.9-gentoo-r6 x86_64 AMD Athlon(tm) 64 Processor 3400+
Autoconf: sys-devel/autoconf-2.59-r5
Automake: sys-devel/automake-1.8.5-r1
Binutils: sys-devel/binutils-2.15.90.0.1.1-r3
Headers:  sys-kernel/linux26-headers-2.6.8.1-r1
Libtools: sys-devel/libtool-1.5.2-r7
ACCEPT_KEYWORDS="amd64"
AUTOCLEAN="yes"
CFLAGS="-pipe -O3 -march=k8"
CHOST="x86_64-pc-linux-gnu"
COMPILER=""
CONFIG_PROTECT="/etc /usr/X11R6/lib/X11/xkb /usr/kde/2/share/config /usr/kde/3/share/config /usr/lib/mozilla/defaults/pref /usr/share/config /var/qmail/control"
CONFIG_PROTECT_MASK="/etc/gconf /etc/terminfo /etc/env.d"
CXXFLAGS="-pipe -O3 -march=k8"
DISTDIR="/usr/portage/distfiles"
FEATURES="autoaddcvs autoconfig ccache distlocks sandbox"
GENTOO_MIRRORS="http://gentoo.osuosl.org http://distro.ibiblio.org/pub/Linux/distributions/gentoo"
MAKEOPTS="-j2"
PKGDIR="/usr/portage/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY=""
SYNC="rsync://rsync.gentoo.org/gentoo-portage"
USE="amd64 X acpi alsa berkdb bitmap-fonts cdr crypt encode esd f77 fam flac fortran gif gpm gtk gtk2 imlib ipv6 java jp2 jpeg lzw lzw-tiff mozilla multilib ncurses network nls oggvorbis opengl oss pam perl png python readline samba speex sqlite ssl tcpd theora tiff truetype usb userlocales xml xml2 xmms xpm xrandr xv zlib"
Comment 3 Tony Vroon (RETIRED) gentoo-dev 2004-12-05 23:18:31 UTC
Okay, that must be an amd64-specific bug, your CFLAGS make sense.
The 0.9.7 release is expected in less then a week and should fix a few playlist-related bugs. I have reported your bug upstream and they have asked to test 0.9.7 when it comes out.
Comment 4 Tony Vroon (RETIRED) gentoo-dev 2004-12-16 08:13:12 UTC
Could you test on the new 0.9.7 release please?
If you wait two hours then sync your portage tree, you should be able to merge it. 
Comment 5 postmodern 2004-12-16 19:43:58 UTC
Installed beep-media-player-0.9.7 and the bug still persists with the exact same error message.
Comment 6 Tony Vroon (RETIRED) gentoo-dev 2004-12-17 08:59:37 UTC
I have submitted a bug upstream and added you to the CC list.
Comment 7 postmodern 2004-12-19 04:50:08 UTC
David Le Brun (david@dyn-ns.net) produced a nice patch that resolves the issue. Attached are the patch and a diff for the beep-media-player-0.9.7 ebuild.
Comment 8 postmodern 2004-12-19 04:50:51 UTC
Created attachment 46336 [details, diff]
patch to fix double/triple click issue with beep/playlistwin.c
Comment 9 postmodern 2004-12-19 04:54:11 UTC
Created attachment 46337 [details, diff]
patch to beep-media-player-0.9.7.ebuild to apply the noclick-resize patch
Comment 10 Tony Vroon (RETIRED) gentoo-dev 2004-12-19 06:59:31 UTC
Reopening and adding upstream patch. Compiles fine here on x86.
Comment 11 Tony Vroon (RETIRED) gentoo-dev 2004-12-19 07:03:46 UTC
In CVS as beep-media-player-0.9.7-r1.