Re: [PATCH] Add confirmation dialog when loading dlabel DDs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The condition is wrong and I will fix it..
There is a bug, but for rhel and I meant to add this to Fedora first.. I could clone it I suppose, but I'm not sure about confidentiality here..

--
Martin Sivák
msivak@xxxxxxxxxx
Red Hat Czech
Anaconda team / Brno, CZ

----- "Hans de Goede" <hdegoede@xxxxxxxxxx> wrote:

> Hi,
> 
> The stopNewt test is different from the startNewt test, which
> does not seem right. Also is there no bug number for this issue?
> 
> Not even a related one ?
> 
> Regards,
> 
> Hans
> 
> 
> On 06/28/2010 01:59 PM, Martin Sivak wrote:
> > ---
> >   loader/loader.c |   35 +++++++++++++++++++++++++++++++++++
> >   1 files changed, 35 insertions(+), 0 deletions(-)
> >
> > diff --git a/loader/loader.c b/loader/loader.c
> > index 55cb0f8..c13956c 100644
> > --- a/loader/loader.c
> > +++ b/loader/loader.c
> > @@ -2036,7 +2036,37 @@ int main(int argc, char ** argv) {
> >           logMessage(INFO, "Trying to detect vendor driver discs");
> >           dd = findDriverDiskByLabel();
> >           dditer = dd;
> > +
> > +        if (dd&&  !loaderData.ksfile) {
> > +            startNewt();
> > +        }
> > +
> > +
> >           while(dditer) {
> > +            /* If in interactive mode, ask for confirmation before
> loading the DD */
> > +            if (!loaderData.ksfile) {
> > +                char * buf;
> > +
> > +                checked_asprintf(&buf,
> > +                                 _("Driver disc was detected in %s.
> "
> > +                                   "Do you want to use it?."),
> dditer->data);
> > +
> > +                rc = newtWinChoice(_("Driver disc detected"),
> _("Use it"), _("Skip it"),
> > +                                   buf);
> > +                free(buf);
> > +                if (rc == 2) {
> > +                    logMessage(INFO, "Skipping driver disk %s.",
> (char*)(dditer->data));
> > +
> > +                    /* clean the device record */
> > +                    free((char*)(dditer->data));
> > +                    dditer->data = NULL;
> > +
> > +                    /* next DD */
> > +                    dditer = g_slist_next(dditer);
> > +                    continue;
> > +                }
> > +            }
> > +
> >               /* load the DD */
> >               if (loadDriverDiskFromPartition(&loaderData,
> (char*)(dditer->data))) {
> >                   logMessage(ERROR, "Automatic driver disk loader
> failed for %s.", (char*)(dditer->data));
> > @@ -2052,6 +2082,11 @@ int main(int argc, char ** argv) {
> >               /* next DD */
> >               dditer = g_slist_next(dditer);
> >           }
> > +
> > +        if (!dd&&  loaderData.ksfile) {
> > +            stopNewt();
> > +        }
> > +
> >           g_slist_free(dd);
> >       }
> >
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list



[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux