From 59d070efee749dda3a605f33e1e0d9edf5485528 Mon Sep 17 00:00:00 2001
From: Gabriele Mohr <gs@suse.de>
Date: Wed, 10 Oct 2012 13:41:27 +0200
Subject: [PATCH] Don't ask for license agreement for openSUSE (bnc #780659)

use new method name needToAcceptLicense
---
 src/misc.cc | 90 ++++++++++++++++++++++++++++++++++---------------------------
 1 file changed, 50 insertions(+), 40 deletions(-)

diff --git a/src/misc.cc b/src/misc.cc
index 15fca12..ff1bf54 100644
--- a/src/misc.cc
+++ b/src/misc.cc
@@ -120,6 +120,8 @@ bool confirm_licenses(Zypper & zypper)
 
   for (ResPool::const_iterator it = God->pool().begin(); it != God->pool().end(); ++it)
   {
+    bool to_accept = true;
+    
     if (it->status().isToBeInstalled() &&
         !it->resolvable()->licenseToConfirm().empty())
     {
@@ -167,15 +169,20 @@ bool confirm_licenses(Zypper & zypper)
           " (" + kind_to_string_localized(it->resolvable()->kind(), 1) + ")" :
           string();
 
-      // introduction
-      s << str::form(
-          // translators: the first %s is the name of the package, the second
-          // is " (package-type)" if other than "package" (patch/product/pattern)
-          _("In order to install '%s'%s, you must agree"
-            " to terms of the following license agreement:"),
-            get_display_name(it->resolvable()).c_str(), kindstr.c_str());
-      s << endl << endl;
-
+      if ( !it->resolvable()->needToAcceptLicense() )
+        to_accept = false;
+      
+      if (to_accept)
+      {
+        // introduction
+        s << str::form(
+                       // translators: the first %s is the name of the package, the second
+                       // is " (package-type)" if other than "package" (patch/product/pattern)
+                       _("In order to install '%s'%s, you must agree"
+                         " to terms of the following license agreement:"),
+                       get_display_name(it->resolvable()).c_str(), kindstr.c_str());
+        s << endl << endl;
+      }
       // license text
       const string& licenseText = it->resolvable()->licenseToConfirm();
       if (licenseText.find("DT:Rich")==licenseText.npos)
@@ -187,41 +194,44 @@ bool confirm_licenses(Zypper & zypper)
       if (zypper.globalOpts().machine_readable || !show_text_in_pager(s.str()))
         zypper.out().info(s.str(), Out::QUIET);
 
-      // lincense prompt
-      string question = _("Do you agree with the terms of the license?");
-      //! \todo add 'v' option to view the license again, add prompt help
-      if (!read_bool_answer(PROMPT_YN_LICENSE_AGREE, question, license_auto_agree))
+      if (to_accept)
       {
-        confirmed = false;
-
-        if (zypper.globalOpts().non_interactive)
+        // lincense prompt
+        string question = _("Do you agree with the terms of the license?");
+        //! \todo add 'v' option to view the license again, add prompt help
+        if (!read_bool_answer(PROMPT_YN_LICENSE_AGREE, question, license_auto_agree))
         {
-          zypper.out().info(
-            _("Aborting installation due to the need for license confirmation."),
-            Out::QUIET);
-          zypper.out().info(boost::str(format(
-            // translators: %sanslate the '--auto-agree-with-licenses',
-            // it is a command line option
-            _("Please restart the operation in interactive"
-              " mode and confirm your agreement with required licenses,"
-              " or use the %s option.")) % "--auto-agree-with-licenses"),
-            Out::QUIET);
-
-          MIL << "License(s) NOT confirmed (non-interactive without auto confirmation)" << endl;
-        }
-        else
-        {
-          zypper.out().info(boost::str(format(
-              // translators: e.g. "... with flash package license."
-              //! \todo fix this to allow proper translation
-              _("Aborting installation due to user disagreement with %s %s license."))
-                % get_display_name(it->resolvable())
-                % kind_to_string_localized(it->resolvable()->kind(), 1)),
-              Out::QUIET);
+          confirmed = false;
+
+          if (zypper.globalOpts().non_interactive)
+          {
+            zypper.out().info(
+                              _("Aborting installation due to the need for license confirmation."),
+                              Out::QUIET);
+            zypper.out().info(boost::str(format(
+                                                // translators: %sanslate the '--auto-agree-with-licenses',
+                                                // it is a command line option
+                                                _("Please restart the operation in interactive"
+                                                  " mode and confirm your agreement with required licenses,"
+                                                  " or use the %s option.")) % "--auto-agree-with-licenses"),
+                              Out::QUIET);
+
+            MIL << "License(s) NOT confirmed (non-interactive without auto confirmation)" << endl;
+          }
+          else
+          {
+            zypper.out().info(boost::str(format(
+                                                // translators: e.g. "... with flash package license."
+                                                //! \todo fix this to allow proper translation
+                                                _("Aborting installation due to user disagreement with %s %s license."))
+                                         % get_display_name(it->resolvable())
+                                         % kind_to_string_localized(it->resolvable()->kind(), 1)),
+                              Out::QUIET);
             MIL << "License(s) NOT confirmed (interactive)" << endl;
-        }
+          }
 
-        break;
+          break;
+        }
       }
     }
   }
-- 
2.7.4