First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 106487
Alias:
Product:
Component:
Status: RESOLVED
Resolution: DUPLICATE of bug 122256
Assigned To: Gentoo's Team for Core System packages <base-system@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Stuart W. Finlayson <stu@santa-li.com>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
eject-2.0.13-toggle-fix.patch eject-2.0.13-toggle-fix.patch patch Stuart W. Finlayson 2005-09-18 23:30 0000 1.36 KB Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 106487 depends on: Show dependency tree
Bug 106487 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug

Description:   Opened: 2005-09-18 23:29 0000 
This relates to the patch that was added for bug #62612, which adds MacOSX   
style of ejecting/loading the cd-rom tray through the use of an option -T.     
   
I think I found three problems with this patch.  First, the "exit(0);" in the   
section that handles the -T option is misplaced.  Allowing options that appear 
after the -T option to be processed.  Second, the EjectCdrom() is not checked 
for errors.  I found this by mounting the device and running `eject -T -v`, 
which lead to me seeing the third problem.  Third, the section that handles 
the -T option is misplaced.  It should occur after the device(s) is/are 
unmounted, otherwise it will not eject.

------- Comment #1 From Stuart W. Finlayson 2005-09-18 23:30:43 0000 ------- 
Created an attachment (id=68792) [details]
eject-2.0.13-toggle-fix.patch

Fixes problems with earlier patch.

------- Comment #2 From SpanKY 2006-02-09 20:55:07 0000 ------- 

*** This bug has been marked as a duplicate of 122256 ***
First Last Prev Next    No search results available      Search page      Enter new bug