pkgcore 0.12.44__tar.gz → 0.12.45__tar.gz

{pkgcore-0.12.44 → pkgcore-0.12.45}/NEWS.rst

@@ -2,6 +2,260 @@
 Release Notes
 =============
 
+----------------------------
+pkgcore 0.12.45 (2026-10-03)
+----------------------------
+
+Fixes
+~~~~~
+
+- resolver: resolve a package's IDEPEND instead of resolving its PDEPEND
+  twice (Arthur Zamarin)
+
+- resolver: when a candidate fails to insert, check the next one against its
+  own dependencies rather than the previous candidate's (Arthur Zamarin)
+
+- binpkg: installing or replacing from a binpkg repo no longer fails with a
+  TypeError (Arthur Zamarin)
+
+- unmerging a package with files matching ``UNINSTALL_IGNORE`` no longer
+  crashes (Arthur Zamarin)
+
+- fetch: a download that succeeds on the last allowed attempt is no longer
+  reported as failed, so ``attempts=1`` works (Arthur Zamarin)
+
+- GPG signed Manifests can be read again (Arthur Zamarin)
+
+- ebd: a sandbox violation shows its access summary again instead of hanging
+  the build (Arthur Zamarin)
+
+- ebd: ``nonfatal`` works again for ``newins`` and the other ``new*`` helpers
+  (Arthur Zamarin)
+
+- ebd: a failed ``nonfatal doins`` (and other install helpers) no longer breaks
+  every later call in the same phase (Arthur Zamarin)
+
+- ebd: ``eapply_user`` no longer reuses the options of the last ``eapply``
+  call, such as ``-p0`` (Arthur Zamarin)
+
+- EAPI 9: ``ver_replacing`` can be called (Arthur Zamarin)
+
+- profiles can be looked up by absolute path, which also fixes
+  ``pkgcheck --profiles /path/to/profile`` (Arthur Zamarin)
+
+- ``ACCEPT_KEYWORDS`` wildcards (``*``, ``~*`` and ``**``) work without any
+  package.accept_keywords entries (Arthur Zamarin)
+
+- USE is now stacked in portage's order: make.conf USE, including ``-*`` and
+  USE_EXPAND variables such as ``VIDEO_CARDS``, overrides the profile's
+  package.use, and ``USE`` from the environment overrides package.use
+  (Arthur Zamarin)
+
+- ebd: environments saved by pkgcore or portage are filtered again before
+  the ``pkg_*`` phases load them, so stale internal variables no longer leak
+  in (Arthur Zamarin)
+
+- an unquoted ``EAPI=8 # comment`` line is accepted again (Arthur Zamarin)
+
+- binpkgs containing device nodes can be read (Arthur Zamarin)
+
+- vdb: installing a package that has no ebuild no longer crashes and leaves a
+  half-written entry behind (Arthur Zamarin)
+
+- ebd: ``eapply`` on a directory applies only the patches directly inside it,
+  as PMS says, instead of descending into subdirectories (Arthur Zamarin)
+
+- with ``XDG_CONFIG_HOME``, ``XDG_CACHE_HOME`` or ``XDG_DATA_HOME`` set, pkgcore
+  now uses a ``pkgcore`` subdirectory of it, as it does for the defaults. A
+  config at ``$XDG_CONFIG_HOME/pkgcore.conf`` has to move to
+  ``$XDG_CONFIG_HOME/pkgcore/pkgcore.conf`` (Arthur Zamarin)
+
+- rsync sync: the timestamp check uses the repo's ssh, proxy and rsync
+  settings, and a failed check falls back to a full sync instead of aborting
+  (Arthur Zamarin)
+
+- a malformed line in ``use.desc``, ``use.local.desc`` or ``desc/*.desc`` is
+  logged and skipped instead of making every USE description unavailable
+  (Arthur Zamarin)
+
+- ``FEATURES=unmerge-backup`` saves a binpkg of the package being removed again
+  (Arthur Zamarin)
+
+- ``pkgcore.bugzilla``: searching an empty list of bug ids or packages
+  returns nothing instead of every bug on the server (Arthur Zamarin)
+
+- ``pkgcore.bugzilla``: ``limit`` and ``offset`` apply to a whole search even
+  when a long one is split into several requests, rather than to each request
+  (Arthur Zamarin)
+
+- ``pkgcore.bugzilla``: a 429 response is retried whatever its body
+  (Arthur Zamarin)
+
+- ``pkgcore.bugzilla``: a connection reset while reading the response is
+  retried and raised as ``BugzillaConnectionError``, instead of escaping as a
+  bare ``OSError`` (Arthur Zamarin)
+
+- ``pkgcore.bugzilla``: redirects are refused with a caller supplied opener
+  too, so the api key isn't sent on to another host (Arthur Zamarin)
+
+- ``pkgcore.bugzilla.ListChange``: combining a ``setting()`` with adds or
+  removes keeps it a set of the resulting list, instead of dropping the set
+  (Arthur Zamarin)
+
+- ``pkgcore.bugzilla.BugUpdate``: an update that clears a field, such as
+  ``whiteboard=""``, is no longer falsy (Arthur Zamarin)
+
+- ``--api-key`` is stripped, and a blank one falls back to the environment and
+  dotfiles like an unset one, instead of being sent as the key (Arthur Zamarin)
+
+- a configured binpkg repo passes install, uninstall and replace through to the
+  underlying repo again (Arthur Zamarin)
+
+- looking a repo up by path no longer matches a sibling whose name starts the
+  same, such as ``gentoo-extra`` for ``gentoo`` (Arthur Zamarin)
+
+- ``bugzilla_cassette`` fixture: a test that fails no longer gets an extra
+  "unused recordings" error on top (Arthur Zamarin)
+
+- ``pinspect profile status`` and ``pinspect query get_profiles`` work again
+  (Arthur Zamarin)
+
+- cvs sync: the first checkout names the module to check out (Arthur Zamarin)
+
+- rsync sync: ``metadata/timestamp.chk`` is read as UTC with its offset, rather
+  than as local time, so the "is the tree new enough" check isn't skewed by the
+  local timezone (Arthur Zamarin)
+
+- libtool archive fixing no longer rewrites every ``.la`` file that has an
+  ``inherited_linker_flags`` entry, and moves thread flags into that entry
+  when present, as portage does (Arthur Zamarin)
+
+- an unknown ``profile-formats`` entry no longer adds ``pms`` next to the known
+  ones, so ``portage-2`` repos with an extra format keep directory support
+  (Arthur Zamarin)
+
+- the bash version an EAPI needs is compared numerically, so bash 5.10 won't
+  disable EAPI 9, and a bash that can't be queried no longer fails the import
+  (Arthur Zamarin)
+
+- http, tar and sqfs sync: a DNS or connection failure is reported as a sync
+  error instead of crashing with ``AttributeError`` (Arthur Zamarin)
+
+- a failed fetch through package operations created without an observer
+  reports the fetch failure instead of crashing with ``AttributeError``
+  (Arthur Zamarin)
+
+- the unsafe world writable SetUID/SetGID warning names the right bit, and a
+  world writable setgid directory is left alone, as portage does
+  (Arthur Zamarin)
+
+- ebd: shutting down a daemon that stopped answering kills it instead of
+  hanging forever, as on Ctrl-C during a phase (Arthur Zamarin)
+
+- ebd: a multi-line error from a ``nonfatal`` helper, such as ``install``'s
+  stderr, no longer leaves stray lines that confuse the next helper call
+  (Arthur Zamarin)
+
+- config: a native int given to an int typed setting, as from Python built
+  config sections, is accepted instead of failing to convert (Arthur Zamarin)
+
+- ``pconfig dump`` and ``pconfig dump-uncollapsed`` show int settings
+  (Arthur Zamarin)
+
+- ``pmaint eclass <dir>`` reads the eclasses from that directory rather than
+  the current one (Arthur Zamarin)
+
+- incremental settings from ``/etc/profile.env``, such as ``CONFIG_PROTECT_MASK``,
+  are kept even when the profile doesn't set them (Arthur Zamarin)
+
+- a bad ``parent`` entry in a profile outside a repo, such as a stale
+  ``/etc/portage/make.profile`` directory, is logged instead of crashing with
+  ``AttributeError`` (Arthur Zamarin)
+
+- ebd: ``doman -i18n=<lang>`` installs into that language's man directory
+  instead of failing (Arthur Zamarin)
+
+- merging a symlink over an existing directory resolves the link's target from
+  the link's own directory, and an absolute target under ``ROOT``, rather than
+  checking the wrong path or the host system (Arthur Zamarin)
+
+- ``pmerge -x`` and ``--pdb-intercept`` given more than once keep every value,
+  not only the last one (Arthur Zamarin)
+
+- ``pclean`` removes files even when its output is redirected, as from cron;
+  only ``--pretend`` lists them, one bare path per line when redirected
+  (Arthur Zamarin)
+
+- ``restrictions``: the DNF solutions of a negated ``OrRestriction`` are only
+  the negated terms, without the original ones after them (Arthur Zamarin)
+
+- ``pkgcore.pkgsets.glsa``: an unaffected glob range such as ``1.5*`` no longer
+  marks the vulnerable versions safe and the fixed ones vulnerable, and glob and
+  ``-r0`` revision ranges respect the GLSA's slot (Arthur Zamarin)
+
+- ``pkgcore.pkgsets.glsa``: a GLSA range with a subslot, such as
+  ``slot="0/esr78"``, matches that slot and subslot instead of nothing, and
+  ``slot="*"`` matches any slot (Arthur Zamarin)
+
+- ``pkgcore.ebuild.atom``: an ``=*`` glob only matches up to a version
+  component boundary, so ``=cat/pkg-1*`` no longer matches ``cat/pkg-10``, as
+  in Portage; GLSA glob ranges follow the same rule (Arthur Zamarin)
+
+- ``pkgcore.ebuild.ebd_ipc``: ``dosym`` checks for a directory target in the
+  image instead of on the host, and ``dohard`` links a file from the image
+  instead of the host's copy (Arthur Zamarin)
+
+- ``pkgcore.ebuild.processor``: environment values containing both a quote and
+  a backslash, such as ``it's C:\new``, reach the ebuild unchanged instead of
+  with the backslash sequences expanded (Arthur Zamarin)
+
+- ``pkgcore.ebuild.ebd_ipc``: ``doins`` given only directories without ``-r``
+  fails instead of installing nothing and succeeding, as in Portage
+  (Arthur Zamarin)
+
+- ``pkgcore.ebuild.conditionals``: merging a package with ``:=`` deps that
+  aren't all installed, such as ``|| ( a:= b:= )``, no longer crashes; the
+  unmatched atoms are recorded as written, as in Portage. Recording the slots
+  also no longer changes the same atoms elsewhere in the process
+  (Arthur Zamarin)
+
+- ``pkgcore.ebuild.portageq``: ``pinspect query mass_best_version`` and its
+  ``pinspect portageq`` form print each best version instead of failing with a
+  TypeError (Arthur Zamarin)
+
+- ``pkgcore.merge.triggers.BinaryDebug``: configured extra strip flags are
+  passed once per binary instead of piling up with each one stripped
+  (Arthur Zamarin)
+
+- ``pkgcore.config.basics``: an inline config section of the wrong type is
+  reported as such instead of failing with an ``AttributeError``
+  (Arthur Zamarin)
+
+- ``pquery -`` and ``patom -F FORMAT -`` skip blank lines read from stdin
+  instead of rejecting them as invalid atoms (Arthur Zamarin)
+
+- ``pquery -1`` stops at the first match instead of printing one per repo
+  (Arthur Zamarin)
+
+- ``pkgcore.fs.fs.get_major_minor``: device files with a major or minor number
+  above 255, such as NVMe disks, get their real numbers instead of truncated
+  ones (Arthur Zamarin)
+
+- ``pkgcore.util.commandline.StoreRepoObject``: ``-r``/``--repo`` looks up
+  repos in the domain chosen with ``--domain`` instead of always in the default
+  domain (Arthur Zamarin)
+
+- ``FEATURES=multilib-strict`` without ``MULTILIB_STRICT_DIRS`` or
+  ``MULTILIB_STRICT_DENY`` set skips the check, as in Portage, instead of
+  crashing every merge (Arthur Zamarin)
+
+- profiles: a repo inside a directory named ``profiles`` gets the right
+  profile names and file paths (Arthur Zamarin)
+
+- ``pmaint regen``: ``_eclasses_`` in the metadata cache follows each
+  package's inherit order, so regenerating no longer reorders unchanged
+  entries (Arthur Zamarin)
+
 ----------------------------
 pkgcore 0.12.44 (2029-09-26)
 ----------------------------
@@ -62,6 +316,10 @@ Fixes
 - eclass: separate an ``@SUBSECTION`` heading from the prose above it, so it
   renders as a heading (Arthur Zamarin)
 
+- ``ContainmentMatch``: render its values in sorted order, so the string form of
+  a restriction is stable across runs rather than following set iteration order
+  (Arthur Zamarin)
+
 ----------------------------
 pkgcore 0.12.42 (2026-09-05)
 ----------------------------

{pkgcore-0.12.44 → pkgcore-0.12.45}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pkgcore
-Version: 0.12.44
+Version: 0.12.45
 Summary: package managing framework
 Author-email: Michał Górny <mgorny@gentoo.org>, Tim Harder <radhermit@gmail.com>, Brian Harring <ferringb@gmail.com>, Arthur Zamarin <arthurzam@gentoo.org>, Marien Zwart <marienz@gentoo.org>
 Maintainer-email: Arthur Zamarin <arthurzam@gentoo.org>

{pkgcore-0.12.44 → pkgcore-0.12.45}/build/sphinx/man/patom.1

@@ -28,7 +28,7 @@ level margin: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .\" new: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .in \\n[rst2man-indent\\n[rst2man-indent-level]]u
 ..
-.TH "PATOM" "1" "Sep 26, 2026" "0.12.44" "pkgcore"
+.TH "PATOM" "1" "Oct 03, 2026" "0.12.45" "pkgcore"
 .SH NAME
 patom \- Wrapper for running commandline scripts.
 .SH SYNOPSIS

{pkgcore-0.12.44 → pkgcore-0.12.45}/build/sphinx/man/pclean.1

@@ -28,7 +28,7 @@ level margin: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .\" new: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .in \\n[rst2man-indent\\n[rst2man-indent-level]]u
 ..
-.TH "PCLEAN" "1" "Sep 26, 2026" "0.12.44" "pkgcore"
+.TH "PCLEAN" "1" "Oct 03, 2026" "0.12.45" "pkgcore"
 .SH NAME
 pclean \- Wrapper for running commandline scripts.
 .SH SYNOPSIS

{pkgcore-0.12.44 → pkgcore-0.12.45}/build/sphinx/man/pclonecache.1

@@ -28,7 +28,7 @@ level margin: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .\" new: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .in \\n[rst2man-indent\\n[rst2man-indent-level]]u
 ..
-.TH "PCLONECACHE" "1" "Sep 26, 2026" "0.12.44" "pkgcore"
+.TH "PCLONECACHE" "1" "Oct 03, 2026" "0.12.45" "pkgcore"
 .SH NAME
 pclonecache \- Wrapper for running commandline scripts.
 .SH SYNOPSIS

{pkgcore-0.12.44 → pkgcore-0.12.45}/build/sphinx/man/pconfig.1

@@ -28,7 +28,7 @@ level margin: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .\" new: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .in \\n[rst2man-indent\\n[rst2man-indent-level]]u
 ..
-.TH "PCONFIG" "1" "Sep 26, 2026" "0.12.44" "pkgcore"
+.TH "PCONFIG" "1" "Oct 03, 2026" "0.12.45" "pkgcore"
 .SH NAME
 pconfig \- Wrapper for running commandline scripts.
 .SH SYNOPSIS

{pkgcore-0.12.44 → pkgcore-0.12.45}/build/sphinx/man/pebuild.1

@@ -28,7 +28,7 @@ level margin: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .\" new: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .in \\n[rst2man-indent\\n[rst2man-indent-level]]u
 ..
-.TH "PEBUILD" "1" "Sep 26, 2026" "0.12.44" "pkgcore"
+.TH "PEBUILD" "1" "Oct 03, 2026" "0.12.45" "pkgcore"
 .SH NAME
 pebuild \- Wrapper for running commandline scripts.
 .SH SYNOPSIS

{pkgcore-0.12.44 → pkgcore-0.12.45}/build/sphinx/man/pinspect.1

@@ -28,7 +28,7 @@ level margin: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .\" new: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .in \\n[rst2man-indent\\n[rst2man-indent-level]]u
 ..
-.TH "PINSPECT" "1" "Sep 26, 2026" "0.12.44" "pkgcore"
+.TH "PINSPECT" "1" "Oct 03, 2026" "0.12.45" "pkgcore"
 .SH NAME
 pinspect \- Wrapper for running commandline scripts.
 .SH SYNOPSIS

{pkgcore-0.12.44 → pkgcore-0.12.45}/build/sphinx/man/pmaint.1

@@ -28,7 +28,7 @@ level margin: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .\" new: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .in \\n[rst2man-indent\\n[rst2man-indent-level]]u
 ..
-.TH "PMAINT" "1" "Sep 26, 2026" "0.12.44" "pkgcore"
+.TH "PMAINT" "1" "Oct 03, 2026" "0.12.45" "pkgcore"
 .SH NAME
 pmaint \- Wrapper for running commandline scripts.
 .SH SYNOPSIS

{pkgcore-0.12.44 → pkgcore-0.12.45}/build/sphinx/man/pmerge.1

@@ -28,7 +28,7 @@ level margin: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .\" new: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .in \\n[rst2man-indent\\n[rst2man-indent-level]]u
 ..
-.TH "PMERGE" "1" "Sep 26, 2026" "0.12.44" "pkgcore"
+.TH "PMERGE" "1" "Oct 03, 2026" "0.12.45" "pkgcore"
 .SH NAME
 pmerge \- Wrapper for running commandline scripts.
 .SH SYNOPSIS

{pkgcore-0.12.44 → pkgcore-0.12.45}/build/sphinx/man/pquery.1

@@ -28,7 +28,7 @@ level margin: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .\" new: \\n[rst2man-indent\\n[rst2man-indent-level]]
 .in \\n[rst2man-indent\\n[rst2man-indent-level]]u
 ..
-.TH "PQUERY" "1" "Sep 26, 2026" "0.12.44" "pkgcore"
+.TH "PQUERY" "1" "Oct 03, 2026" "0.12.45" "pkgcore"
 .SH NAME
 pquery \- Wrapper for running commandline scripts.
 .SH SYNOPSIS

{pkgcore-0.12.44 → pkgcore-0.12.45}/data/lib/pkgcore/ebd/ebuild-daemon-lib.bash

@@ -94,7 +94,7 @@ __ebd_ipc_cmd() {
 # ask the python side to display sandbox complaints
 __request_sandbox_summary() {
 	local line
-	__ebd_write_line "__request_sandbox_summary ${SANDBOX_LOG}"
+	__ebd_write_line "request_sandbox_summary ${SANDBOX_LOG}"
 	__ebd_read_line line
 	while [[ ${line} != "end_sandbox_summary" ]]; do
 		echo "${line}"

{pkgcore-0.12.44 → pkgcore-0.12.45}/data/lib/pkgcore/ebd/ebuild-daemon.bash

@@ -285,10 +285,8 @@ __ebd_process_ebuild_phases() {
 }
 
 __ebd_process_metadata() {
-	# protect the env.
-	# note the local usage is redundant in light of it, but prefer to write it this
-	# way so that if someone ever drops the (), it'll still not bleed out.
-	(
+	# only run via $(), which protects the env
+	{
 		# Heavy QA checks (IFS, shopt, etc) are suppressed for speed
 		declare -r PKGCORE_QA_SUPPRESSED=false
 		# Wipe __mode; it bleeds from our parent.
@@ -317,7 +315,7 @@ __ebd_process_metadata() {
 		__execute_phases "${2:-depend}" && exit 0
 		__ebd_process_sandbox_results
 		exit 1
-	)
+	}
 }
 
 __make_preloaded_eclass_func() {

{pkgcore-0.12.44 → pkgcore-0.12.45}/data/lib/pkgcore/ebd/ebuild.bash

@@ -86,13 +86,13 @@ alias __load_environ='{
 	PKGCORE_EXISTING_PATH=${PATH};
 	__timed_call __environ_sanitize_saved_env "${PKGCORE_TARGET_ENV}"
 	if [[ -n ${PKGCORE_PERF_DEBUG} ]]; then
-		echo "timing source ${PKGCORE_TARGET_ENV}" >&2
-		time source "${PKGCORE_TARGET_ENV}" >&2
-		echo "timed source ${PKGCORE_TARGET_ENV}" >&2
+		echo "timing source ${T}/.scrubbed-env" >&2
+		time source "${T}"/.scrubbed-env >&2
+		echo "timed source ${T}/.scrubbed-env" >&2
 	else
-		source "${PKGCORE_TARGET_ENV}" >&2
+		source "${T}"/.scrubbed-env >&2
 	fi
-	[[ $? == 0 ]] || die "sourcing saved env failed";
+	[[ $? == 0 ]] || die "sourcing scrubbed env failed";
 	__ensure_PATH "${PKGCORE_EXISTING_PATH}";
 	__timed_call __load_eapi_libs
 	__timed_call __source_bashrcs
@@ -502,26 +502,23 @@ __dump_metadata_keys() {
 	# be invoked after ebuild code has done it's thing, as such we no longer care,
 	# and directly screw w/ it for speed reasons- about 5% speedup in metadata regen.
 	set -f
-	local key phases phase
+	local key phases phase val out
+	local -a words
 	for key in "${PKGCORE_METADATA_KEYS[@]}"; do
 		if [[ ${key} == DEFINED_PHASES ]]; then
 			for phase in "${PKGCORE_EBUILD_PHASES[@]}"; do
 				__is_function "${phase}" && phases+=( ${phase} )
 			done
-			__ebd_write_line "key DEFINED_PHASES=${phases[@]:--}"
-		else
-			# deref the val, if it's not empty/unset, then spit a key command to EBD
-			# after using echo to normalize whitespace (specifically removal of newlines)
-			if [[ ${!key:-unset} != "unset" ]]; then
-				# note that we explicitly bypass the normal functions, and directly
-				# write to the FD. This is done since it's about 25% faster for our usage;
-				# if we used the functions, we'd have to subshell the 'echo ${!key}', which
-				# because of bash behaviour, means the content would be read byte by byte.
-				echo -n "key ${key}=" >&${PKGCORE_EBD_WRITE_FD}
-				echo ${!key} >&${PKGCORE_EBD_WRITE_FD}
-			fi
+			out+="key DEFINED_PHASES=${phases[@]:--}"$'\n'
+		elif [[ ${!key:-unset} != "unset" ]]; then
+			# normalize whitespace (specifically removal of newlines)
+			words=( ${!key} )
+			printf -v val '%s ' "${words[@]}"
+			out+="key ${key}=${val% }"$'\n'
 		fi
 	done
+	# a single write is notably faster than writing each key separately
+	printf '%s' "${out}" >&${PKGCORE_EBD_WRITE_FD}
 	set +f
 }
 

pkgcore-0.12.45/data/lib/pkgcore/ebd/helpers/9/pkg_postinst/ver_replacing

@@ -0,0 +1,10 @@
+#!/usr/bin/env pkgcore-ebuild-helper
+
+source "${PKGCORE_EBD_PATH}"/eapi/7/global.bash || die "failed sourcing eapi/7/global.bash"
+
+[[ $# -eq 2 ]] || die "Usage: ver_replacing <op> <ver>"
+
+for v in ${REPLACING_VERSIONS}; do
+	ver_test "${v}" "$@" && exit 0
+done
+exit 1

{pkgcore-0.12.44 → pkgcore-0.12.45}/data/lib/pkgcore/ebd/helpers/internals/helper-lib.bash

@@ -75,6 +75,5 @@ __helper_failed() {
 
 __helper_check_exit() {
 	[[ $1 == 0 ]] && return
-	shift
 	__helper_exit "$@"
 }

{pkgcore-0.12.44 → pkgcore-0.12.45}/data/lib/pkgcore/ebd/isolated-functions.bash

@@ -217,7 +217,7 @@ __var_push() {
 
 		# If the specified variable currently has a value we save it;
 		# otherwise, just push the variable name onto the stack.
-		if orig_val=$(declare -p ${var} 2>/dev/null); then
+		if [[ -v ${var} ]] && orig_val=$(declare -p ${var} 2>/dev/null); then
 			orig_val=${orig_val/declare *${var}=[\'\"]/${var}=}
 			orig_val=${orig_val%[\'\"]}
 		else

{pkgcore-0.12.44 → pkgcore-0.12.45}/src/pkgcore/__init__.py

@@ -1,4 +1,4 @@
 __title__ = "pkgcore"
-__version__ = "0.12.44"
-__version_info__ = (0, 12, 44)
+__version__ = "0.12.45"
+__version_info__ = (0, 12, 45)
 __python_mininum_version__ = (3, 12, 0)

pkgcore-0.12.45/src/pkgcore/_verinfo.py

@@ -0,0 +1 @@
+version_info={'rev': 'fad4a6e62ddc65690410f29602f81ec55777e998', 'date': 'Sat Oct 03 14:13:04 2026', 'timestamp': 1791036784, 'tag': '0.12.45', 'commits': 0}

{pkgcore-0.12.44 → pkgcore-0.12.45}/src/pkgcore/binpkg/repository.py

@@ -34,7 +34,7 @@ class force_unpacking(triggers.base):
     priority = 5
     _hooks = ("sanity_check",)
     _label = "forced decompression"
-    _engine_type = triggers.INSTALLING_MODES
+    _engine_types = triggers.INSTALLING_MODES
 
     def __init__(self, format_op):
         self.format_op = format_op
@@ -84,7 +84,7 @@ class BinPkg(ebuild_built.generate_new_factory):
             t = force_unpacking(op_inst.format_op)
             t.register(engine_inst)
 
-        super()._add_format_triggers(self, pkg, op_inst, format_op_inst, engine_inst)
+        super()._add_format_triggers(pkg, op_inst, format_op_inst, engine_inst)
 
     def scan_contents(self, location):
         return scan(location, offset=location)

{pkgcore-0.12.44 → pkgcore-0.12.45}/src/pkgcore/bugzilla/apikey.py

@@ -33,6 +33,7 @@ __all__ = (
     "find_api_key",
 )
 
+import argparse
 import os
 import stat
 import typing
@@ -126,13 +127,30 @@ def find_api_key(
     return None
 
 
+class _StoreApiKey(argparse.Action):
+    """Store ``--api-key`` through :func:`find_api_key`, so a blank one is absent"""
+
+    def __call__(
+        self,
+        parser: argparse.ArgumentParser,
+        namespace: argparse.Namespace,
+        values: typing.Any,
+        option_string: str | None = None,
+    ) -> None:
+        setattr(namespace, self.dest, find_api_key(values))
+
+
 class BugzillaApiKey:
     """Adds ``--api-key``, defaulting through :func:`find_api_key`"""
 
     @classmethod
     def mangle_argparser(cls, parser: arghparse.ArgumentParser) -> None:
         parser.add_argument(
-            "--api-key", metavar="TOKEN", help="Bugzilla API key", docs=API_KEY_DOCS
+            "--api-key",
+            metavar="TOKEN",
+            action=_StoreApiKey,
+            help="Bugzilla API key",
+            docs=API_KEY_DOCS,
         )
         parser.bind_delayed_default(1000, "api_key")(cls._default_api_key)
 

{pkgcore-0.12.44 → pkgcore-0.12.45}/src/pkgcore/bugzilla/changes.py

@@ -90,6 +90,12 @@ class ListChange[T]:
     def __or__(self, other: "ListChange[T]") -> "ListChange[T]":
         if other.replace is not None:
             return other
+        if self.replace is not None:
+            # still a wholesale set, now of the list other's changes produce
+            kept = tuple(x for x in self.replace if x not in other.remove)
+            return ListChange(
+                replace=kept + tuple(x for x in other.add if x not in kept)
+            )
         return ListChange(
             add=self.add + tuple(x for x in other.add if x not in self.add),
             remove=self.remove + tuple(x for x in other.remove if x not in self.remove),
@@ -320,9 +326,19 @@ class BugUpdate:
             raise BugzillaUsageError("DUPLICATE and dupe_of must be used together")
 
     def __bool__(self) -> bool:
-        return any(
-            bool(getattr(self, field.name)) for field in dataclasses.fields(self)
-        )
+        """Whether :meth:`to_wire` sends any change.
+
+        A scalar counts once it isn't None, so ``whiteboard=""`` clearing the
+        whiteboard is a change; list fields count when they change anything.
+        """
+        for field in dataclasses.fields(self):
+            value = getattr(self, field.name)
+            if isinstance(value, ListChange | tuple):
+                if value:
+                    return True
+            elif value is not None:
+                return True
+        return False
 
     @classmethod
     def sanity_check(

{pkgcore-0.12.44 → pkgcore-0.12.45}/src/pkgcore/bugzilla/client.py

@@ -2,9 +2,11 @@
 
 __all__ = ("DEFAULT_URL", "EVERYTHING", "PAGE_SIZE", "Bugzilla")
 
+import dataclasses
 import typing
 import urllib.parse
 import urllib.request
+from itertools import chain
 
 from ..log import logger
 from .bug import (
@@ -119,13 +121,33 @@ class Bugzilla:
 
         Use this for the narrow projections a full :class:`Bug` doesn't need;
         anything left out of ``fields`` is simply absent from the results.
+
+        A query too long for one request is split into batches. ``limit`` and
+        ``offset`` still apply to the whole search, over the batches' results
+        in order; with ``order`` set as well that can't be honoured, so it's
+        refused.
         """
         base = [("include_fields", ",".join(fields))]
         overhead = len(f"{self.base_url}/rest/bug?") + len(urllib.parse.urlencode(base))
+        batches = list(query.batches(base_length=overhead))
+        offset = query.offset or 0
+        if len(batches) <= 1 or (query.limit is None and not offset):
+            return tuple(
+                chain.from_iterable(self._paged_search(b, base) for b in batches)
+            )
+        if query.order is not None:
+            raise BugzillaUsageError(
+                "limit or offset with order can't be applied to a search "
+                f"split into {len(batches)} batches"
+            )
+        end = None if query.limit is None else offset + query.limit
         results: list[RawBug] = []
-        for batch in query.batches(base_length=overhead):
+        for batch in batches:
+            batch = dataclasses.replace(batch, limit=None, offset=None)
             results.extend(self._paged_search(batch, base))
-        return tuple(results)
+            if end is not None and len(results) >= end:
+                break
+        return tuple(results[offset:end])
 
     def _paged_search(
         self, query: BugQuery, base: list[tuple[str, str]]