! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Performing an eject depends on system eject to use pumount.
Status:
CLOSED: FIXED
Product:
Thunar-vfs
Component:
General

Comments

Description Rodrigo Coacci 2006-08-29 13:47:51 CEST
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.6) Gecko/20050225 Firefox/1.0.1
Build Identifier: 

I just can't eject my cdrom from thunar, but doing unmount and eject from xfdesktop it works normally. I guess Thunar isn't unmounting before ejecting the cdrom, I don't know why.
Here is what it complains about:

umount: /media/NFSMW_DISC2 isn't in fstab (and you are not root)
eject: unmount of `/media/NFSMW_DISC2' failed.

Seems that eject is using umount instead ofp umount, and also that correct eject behavior is distro dependent. At least in Arch linux the eject isn't aware of pmount/pumount

Reproducible: Always

Steps to Reproduce:
1.Mount any cdrom
2.Try to eject it from Thunar


Actual Results:  
An error shows up

Expected Results:  
Should have ejected the media

System config:

HAL 0.5.7
pmount 0.9.11
FAM 2.7.0
D-BUS 0.62
Arch Linux distro
Comment 1 Benedikt Meurer editbugs 2006-08-29 16:01:43 CEST
Created attachment 756 
Possible fix

Try this patch please (make sure you quit Thunar using Thunar -q first).
Comment 2 Rodrigo Coacci 2006-08-30 23:31:42 CEST
Worked flawlessly, big thanks! Please commit.
Comment 3 Benedikt Meurer editbugs 2006-08-31 08:19:40 CEST
Err, not committed yet.
Comment 4 Benedikt Meurer editbugs 2006-08-31 08:22:18 CEST
Committed with revision 22959.

2006-08-31	Benedikt Meurer <benny@xfce.org>

	* thunar-vfs/thunar-vfs-volume-hal.c(thunar_vfs_volume_hal_eject):
	  Unmount volumes prior to ejecting them if they are currently
	  mounted. Bug #2236.

Bug #2236

Reported by:
Rodrigo Coacci
Reported on: 2006-08-29
Last modified on: 2010-11-07

People

Assignee:
Jannis Pohlmann
CC List:
1 user

Version

Version:
unspecified

Attachments

Possible fix (2.69 KB, patch)
2006-08-29 16:01 CEST , Benedikt Meurer
no flags

Additional information