Procházet zdrojové kódy

* cmdline/acqprogress.cc:
- don't ask the user for media change if quiet >= 2, stdout is not
a tty and assume-yes, force-yes or trivial-only option is set to
avoid cpu eating endless loops in unattended runs like apt.cron
(Closes: #602354, LP: #665580)

David Kalnischkies před 15 roky
rodič
revize
996c6447ae
2 změnil soubory, kde provedl 18 přidání a 1 odebrání
  1. 12 0
      cmdline/acqprogress.cc
  2. 6 1
      debian/changelog

+ 12 - 0
cmdline/acqprogress.cc

@@ -11,6 +11,7 @@
 #include "acqprogress.h"
 #include "acqprogress.h"
 #include <apt-pkg/acquire-item.h>
 #include <apt-pkg/acquire-item.h>
 #include <apt-pkg/acquire-worker.h>
 #include <apt-pkg/acquire-worker.h>
+#include <apt-pkg/configuration.h>
 #include <apt-pkg/strutl.h>
 #include <apt-pkg/strutl.h>
 #include <apt-pkg/error.h>
 #include <apt-pkg/error.h>
 
 
@@ -19,6 +20,7 @@
 #include <stdio.h>
 #include <stdio.h>
 #include <signal.h>
 #include <signal.h>
 #include <iostream>
 #include <iostream>
+#include <unistd.h>
 									/*}}}*/
 									/*}}}*/
 
 
 using namespace std;
 using namespace std;
@@ -266,6 +268,16 @@ bool AcqTextStatus::Pulse(pkgAcquire *Owner)
 /* Prompt for a media swap */
 /* Prompt for a media swap */
 bool AcqTextStatus::MediaChange(string Media,string Drive)
 bool AcqTextStatus::MediaChange(string Media,string Drive)
 {
 {
+   // If we do not output on a terminal and one of the options to avoid user
+   // interaction is given, we assume that no user is present who could react
+   // on your media change request
+   if (isatty(STDOUT_FILENO) != 1 && Quiet >= 2 &&
+       (_config->FindB("APT::Get::Assume-Yes",false) == true ||
+	_config->FindB("APT::Get::Force-Yes",false) == true ||
+	_config->FindB("APT::Get::Trivial-Only",false) == true))
+
+      return false;
+
    if (Quiet <= 0)
    if (Quiet <= 0)
       cout << '\r' << BlankLine << '\r';
       cout << '\r' << BlankLine << '\r';
    ioprintf(cout,_("Media change: please insert the disc labeled\n"
    ioprintf(cout,_("Media change: please insert the disc labeled\n"

+ 6 - 1
debian/changelog

@@ -17,8 +17,13 @@ apt (0.8.9) UNRELEASED; urgency=low
   * apt-pkg/depcache.cc:
   * apt-pkg/depcache.cc:
     - don't install previously not installed providers in a try
     - don't install previously not installed providers in a try
       to statisfy a "Breaks: provides" dependency by upgrade
       to statisfy a "Breaks: provides" dependency by upgrade
+  * cmdline/acqprogress.cc:
+    - don't ask the user for media change if quiet >= 2, stdout is not
+      a tty and assume-yes, force-yes or trivial-only option is set to
+      avoid cpu eating endless loops in unattended runs like apt.cron
+      (Closes: #602354, LP: #665580)
 
 
- -- David Kalnischkies <kalnischkies@gmail.com>  Wed, 10 Nov 2010 13:22:39 +0100
+ -- David Kalnischkies <kalnischkies@gmail.com>  Thu, 11 Nov 2010 18:32:21 +0100
 
 
 apt (0.8.8) unstable; urgency=low
 apt (0.8.8) unstable; urgency=low