Script 'mail_helper' called by obssrc
Hello community,
here is the log from the commit of package PackageKit for openSUSE:Factory
checked in at 2021-03-24 16:09:40
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/PackageKit (Old)
and /work/SRC/openSUSE:Factory/.PackageKit.new.2401 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "PackageKit"
Wed Mar 24 16:09:40 2021 rev:217 rq:879958 version:1.2.2
Changes:
--------
--- /work/SRC/openSUSE:Factory/PackageKit/PackageKit.changes 2021-03-16
15:44:04.721005908 +0100
+++ /work/SRC/openSUSE:Factory/.PackageKit.new.2401/PackageKit.changes
2021-03-24 16:09:42.991735268 +0100
@@ -1,0 +2,6 @@
+Mon Mar 15 21:24:53 UTC 2021 - Andrei Dziahel <devel...@develop7.info>
+
+- Add PackageKit-remove-transaction-size-limit.patch: Remove large transaction
+ size sanity check (gh#hughsie/PackageKit/commit#ac5c8660)
+
+-------------------------------------------------------------------
New:
----
PackageKit-remove-transaction-size-limit.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ PackageKit.spec ++++++
--- /var/tmp/diff_new_pack.7nNYgz/_old 2021-03-24 16:09:43.739736053 +0100
+++ /var/tmp/diff_new_pack.7nNYgz/_new 2021-03-24 16:09:43.739736053 +0100
@@ -60,6 +60,8 @@
Patch7:
PackageKit-dnf-Add-support-for-coercing-upgrade-to-distupgrade.patch
# PATCH-FIX-UPSTREAM PackageKit-zypp-initialize-pool.patch
gh#hughsie/PackageKit/commit#3efa0c524, bsc#1180597 -- zypp: Make sure pool is
initialized at the beginning of some methods
Patch8: PackageKit-zypp-initialize-pool.patch
+# PATCH-FIX-UPSTREAM PackageKit-remove-transaction-size-limit.patch
gh#hughsie/PackageKit/commit#ff01813 gh#hughsie/PackageKit/commit#ff01813 --
Fix a "too many packages to process" error against full rebuilds
+Patch9: PackageKit-remove-transaction-size-limit.patch
BuildRequires: fdupes
BuildRequires: gcc-c++
@@ -172,7 +174,7 @@
License: GPL-2.0-or-later
Group: Productivity/Multimedia/Other
Requires(post): update-alternatives
-Requires(postun): update-alternatives
+Requires(postun):update-alternatives
Recommends: %{name} = %{version}
Supplements: (%{name} and gstreamer-plugins-base)
++++++ PackageKit-remove-transaction-size-limit.patch ++++++
>From ff0181389dc281effbd274f6e1ae01e8905ddf8a Mon Sep 17 00:00:00 2001
From: Richard Hughes <rich...@hughsie.com>
Date: Thu, 19 Nov 2020 19:15:08 +0000
Subject: [PATCH] Increase the number of packages that can be processed in one
transaction
Ohh TeX Live, you taunt me.
Fixes https://github.com/hughsie/PackageKit/issues/442
---
src/pk-transaction.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/pk-transaction.c b/src/pk-transaction.c
index 7cb1ced60..6167f0473 100644
--- a/src/pk-transaction.c
+++ b/src/pk-transaction.c
@@ -77,7 +77,7 @@ static gboolean pk_transaction_is_supported_content_type
(PkTransaction *transac
#define PK_TRANSACTION_MAX_ITEMS_TO_RESOLVE 10000
/* maximum number of packages that can be processed in one go */
-#define PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS 5200
+#define PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS 10000
struct PkTransactionPrivate
{
>From ac5c8660a82b6df96c9895eb97d242af3b6dac72 Mon Sep 17 00:00:00 2001
From: Nate Graham <n...@kde.org>
Date: Mon, 15 Mar 2021 11:54:14 -0600
Subject: [PATCH] Remove large transaction size sanity check
Some distros regularly push updates of tens of thousands of packages,
causing users to run into this "too many packages" error. This is not
really an error, it's just that the sanity check didn't anticipate that
the world might be insane. :) Let's remove it to avoid these issues.
---
src/pk-transaction.c | 131 +------------------------------------------
1 file changed, 2 insertions(+), 129 deletions(-)
diff --git a/src/pk-transaction.c b/src/pk-transaction.c
index 0724e5f25..258abb2cb 100644
--- a/src/pk-transaction.c
+++ b/src/pk-transaction.c
@@ -76,9 +76,6 @@ static gboolean pk_transaction_is_supported_content_type
(PkTransaction *transac
/* maximum number of items that can be resolved in one go */
#define PK_TRANSACTION_MAX_ITEMS_TO_RESOLVE 10000
-/* maximum number of packages that can be processed in one go */
-#define PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS 10000
-
struct PkTransactionPrivate
{
PkRoleEnum role;
@@ -2791,18 +2788,6 @@ pk_transaction_download_packages (PkTransaction
*transaction,
goto out;
}
- /* check for length sanity */
- length = g_strv_length (package_ids);
- if (length > PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS) {
- g_set_error (&error,
- PK_TRANSACTION_ERROR,
- PK_TRANSACTION_ERROR_NUMBER_OF_PACKAGES_INVALID,
- "Too many packages to process (%i/%i)",
- length, PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS);
- pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
- goto out;
- }
-
/* check package_ids */
ret = pk_package_ids_check (package_ids);
if (!ret) {
@@ -2901,18 +2886,6 @@ pk_transaction_depends_on (PkTransaction *transaction,
goto out;
}
- /* check for length sanity */
- length = g_strv_length (package_ids);
- if (length > PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS) {
- g_set_error (&error,
- PK_TRANSACTION_ERROR,
- PK_TRANSACTION_ERROR_NUMBER_OF_PACKAGES_INVALID,
- "Too many packages to process (%i/%i)",
- length, PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS);
- pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
- goto out;
- }
-
/* check package_ids */
ret = pk_package_ids_check (package_ids);
if (!ret) {
@@ -2965,18 +2938,6 @@ pk_transaction_get_details (PkTransaction *transaction,
goto out;
}
- /* check for length sanity */
- length = g_strv_length (package_ids);
- if (length > PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS) {
- g_set_error (&error,
- PK_TRANSACTION_ERROR,
- PK_TRANSACTION_ERROR_NUMBER_OF_PACKAGES_INVALID,
- "Too many packages to process (%i/%i)",
- length, PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS);
- pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
- goto out;
- }
-
/* check package_ids */
ret = pk_package_ids_check (package_ids);
if (!ret) {
@@ -3030,7 +2991,7 @@ pk_transaction_get_details_local (PkTransaction
*transaction,
goto out;
}
- /* check for length sanity */
+ /* check for empty package list */
length = g_strv_length (full_paths);
if (length == 0) {
g_set_error_literal (&error,
@@ -3040,14 +3001,6 @@ pk_transaction_get_details_local (PkTransaction
*transaction,
pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
goto out;
}
- if (length > PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS) {
- g_set_error (&error,
- PK_TRANSACTION_ERROR,
- PK_TRANSACTION_ERROR_NUMBER_OF_PACKAGES_INVALID,
- "Too many files to process (%i/%i)", length,
PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS);
- pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
- goto out;
- }
/* check all files exists and are valid */
length = g_strv_length (full_paths);
@@ -3129,7 +3082,7 @@ pk_transaction_get_files_local (PkTransaction
*transaction,
goto out;
}
- /* check for length sanity */
+ /* check for empty package list */
length = g_strv_length (full_paths);
if (length == 0) {
g_set_error_literal (&error,
@@ -3139,14 +3092,6 @@ pk_transaction_get_files_local (PkTransaction
*transaction,
pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
goto out;
}
- if (length > PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS) {
- g_set_error (&error,
- PK_TRANSACTION_ERROR,
- PK_TRANSACTION_ERROR_NUMBER_OF_PACKAGES_INVALID,
- "Too many files to process (%i/%i)", length,
PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS);
- pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
- goto out;
- }
/* check all files exists and are valid */
length = g_strv_length (full_paths);
@@ -3256,18 +3201,6 @@ pk_transaction_get_files (PkTransaction *transaction,
goto out;
}
- /* check for length sanity */
- length = g_strv_length (package_ids);
- if (length > PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS) {
- g_set_error (&error,
- PK_TRANSACTION_ERROR,
- PK_TRANSACTION_ERROR_NUMBER_OF_PACKAGES_INVALID,
- "Too many packages to process (%i/%i)",
- length, PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS);
- pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
- goto out;
- }
-
/* check package_ids */
ret = pk_package_ids_check (package_ids);
if (!ret) {
@@ -3466,18 +3399,6 @@ pk_transaction_required_by (PkTransaction *transaction,
goto out;
}
- /* check for length sanity */
- length = g_strv_length (package_ids);
- if (length > PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS) {
- g_set_error (&error,
- PK_TRANSACTION_ERROR,
- PK_TRANSACTION_ERROR_NUMBER_OF_PACKAGES_INVALID,
- "Too many packages to process (%i/%i)",
- length, PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS);
- pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
- goto out;
- }
-
/* check package_ids */
ret = pk_package_ids_check (package_ids);
if (!ret) {
@@ -3530,18 +3451,6 @@ pk_transaction_get_update_detail (PkTransaction
*transaction,
goto out;
}
- /* check for length sanity */
- length = g_strv_length (package_ids);
- if (length > PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS) {
- g_set_error (&error,
- PK_TRANSACTION_ERROR,
- PK_TRANSACTION_ERROR_NUMBER_OF_PACKAGES_INVALID,
- "Too many packages to process (%i/%i)",
- length, PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS);
- pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
- goto out;
- }
-
/* check package_ids */
ret = pk_package_ids_check (package_ids);
if (!ret) {
@@ -3770,18 +3679,6 @@ pk_transaction_install_packages (PkTransaction
*transaction,
goto out;
}
- /* check for length sanity */
- length = g_strv_length (package_ids);
- if (length > PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS) {
- g_set_error (&error,
- PK_TRANSACTION_ERROR,
- PK_TRANSACTION_ERROR_NUMBER_OF_PACKAGES_INVALID,
- "Too many packages to process (%i/%i)",
- length, PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS);
- pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
- goto out;
- }
-
/* check package_ids */
ret = pk_package_ids_check (package_ids);
if (!ret) {
@@ -3969,18 +3866,6 @@ pk_transaction_remove_packages (PkTransaction
*transaction,
goto out;
}
- /* check for length sanity */
- length = g_strv_length (package_ids);
- if (length > PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS) {
- g_set_error (&error,
- PK_TRANSACTION_ERROR,
- PK_TRANSACTION_ERROR_NUMBER_OF_PACKAGES_INVALID,
- "Too many packages to process (%i/%i)",
- length, PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS);
- pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
- goto out;
- }
-
/* check package_ids */
ret = pk_package_ids_check (package_ids);
if (!ret) {
@@ -4659,18 +4544,6 @@ pk_transaction_update_packages (PkTransaction
*transaction,
goto out;
}
- /* check for length sanity */
- length = g_strv_length (package_ids);
- if (length > PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS) {
- g_set_error (&error,
- PK_TRANSACTION_ERROR,
- PK_TRANSACTION_ERROR_NUMBER_OF_PACKAGES_INVALID,
- "Too many packages to process (%i/%i)",
- length, PK_TRANSACTION_MAX_PACKAGES_TO_PROCESS);
- pk_transaction_set_state (transaction,
PK_TRANSACTION_STATE_ERROR);
- goto out;
- }
-
/* check package_ids */
ret = pk_package_ids_check (package_ids);
if (!ret) {
