FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Ubuntu > Ubuntu User

 
 
LinkBack Thread Tools
 
Old 09-06-2010, 07:11 AM
Ales Kozumplik
 
Default Add noeject support to loader2 (#477887)

On 09/04/2010 12:02 AM, Brian C. Lane wrote:

When noeject is passed on the kernel command line disable ejecting the
cdrom.

Related: rhbz#477887
---
loader2/cdinstall.c | 30 +++++++++++++++++++++++-------
loader2/loader.c | 6 ++++--
loader2/loader.h | 2 ++
3 files changed, 29 insertions(+), 9 deletions(-)

diff --git a/loader2/cdinstall.c b/loader2/cdinstall.c
index 0b47b6a..ee837ad 100644
--- a/loader2/cdinstall.c
+++ b/loader2/cdinstall.c
@@ -108,8 +108,11 @@ static char * mediaCheckCdrom(char *cddriver) {
free(descr);
}

- ejectCdrom();
-
+ if (!FL_NOEJECT(flags))
+ ejectCdrom();
+ else
+ logMessage(INFO, "noeject in effect, not ejecting cdrom");
+


Ack in general.

You can optionally define a helper function somewhere, something like
eject_cdrom_if(flags) and that will check the flag itself. Given it's
rhel5 anyway and it's loader anyway I do not insist at all.


Ales

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 09-07-2010, 02:10 PM
Chris Lumens
 
Default Add noeject support to loader2 (#477887)

> Ack in general.
>
> You can optionally define a helper function somewhere, something
> like eject_cdrom_if(flags) and that will check the flag itself.
> Given it's rhel5 anyway and it's loader anyway I do not insist at
> all.

Or just have ejectCdrom do the flag check.

- Chris

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 09-07-2010, 03:16 PM
"Brian C. Lane"
 
Default Add noeject support to loader2 (#477887)

On Tue, Sep 07, 2010 at 10:10:31AM -0400, Chris Lumens wrote:
> > Ack in general.
> >
> > You can optionally define a helper function somewhere, something
> > like eject_cdrom_if(flags) and that will check the flag itself.
> > Given it's rhel5 anyway and it's loader anyway I do not insist at
> > all.
>
> Or just have ejectCdrom do the flag check.

Yeah, I didn't do that because it didn't cover all the cases. Seems more
clear to keep it out of the eject function.

--
Brian C. Lane / Anaconda Team
Port Orchard, WA (PST8PDT)
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 

Thread Tools




All times are GMT. The time now is 02:33 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org