[gentoo-commits] proj/portage:master commit in: lib/portage/dbapi/

["Sam James" <sam@gentoo.org>]

commit:     ebd2bf2b6d8ecd1e68f16e9f2672f09c2647c997
Author:     Florian Schmaus <flow <AT> gentoo <DOT> org>
AuthorDate: Sat Sep 13 18:36:08 2025 +0000
Commit:     Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Sun Sep 14 00:15:05 2025 +0000
URL:        https://gitweb.gentoo.org/proj/portage.git/commit/?id=ebd2bf2b

bintree: Add missing 'break' in UseCachedCopyOfRemoteIndex

When we added support to fetch Packages.gz first, a new lopo was added
to the _populate_remote() logic, iterating over ("Packages.gz",
"Packages"). There are two situtations where we want to break out of
the loop: when the remote package index was fetched or when the cached
version of the package index was used. However,
fd55382a4935 ("bintree: Accelerate index fetch by requesting
Packages.gz first"), only added a break statement for the first
case (successful fetch). This adds the missing break statement for the
second case (use cached version).

Fixes: fd55382a4935 ("bintree: Accelerate index fetch by requesting Packages.gz first")
Signed-off-by: Florian Schmaus <flow <AT> gentoo.org>
Part-of: https://github.com/gentoo/portage/pull/1465
Signed-off-by: Sam James <sam <AT> gentoo.org>

 lib/portage/dbapi/bintree.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/portage/dbapi/bintree.py b/lib/portage/dbapi/bintree.py
index 46f5e7f8b9..9d695059d6 100644
--- a/lib/portage/dbapi/bintree.py
+++ b/lib/portage/dbapi/bintree.py
@@ -1670,6 +1670,7 @@ class binarytree:
                             )
                             + "\n"
                         )
+                    break
                 except OSError as e:
                     if (
                         remote_pkgindex_file == "Packages.gz"