Re: [gentoo-dev] Need help: Many patches for unmaintained packages

2021-12-31 Thread Ionen Wolkens
On Fri, Dec 31, 2021 at 07:27:28PM +0100, Martin Dummer wrote:
> Am 31.12.21 um 06:51 schrieb Jonas Stein:
> > I prepared a public Bugzilla filter "m-n PATCH"
> >
> > https://bugs.gentoo.org/buglist.cgi?cmdtype=dorem=run=m-n%20PATCH_id=85842
> >
> 
> Hello Jonas,
> 
> when I try to open that link I receive an error message
> 
> "The search named /m-n PATCH/ has not been made visible to you."
> 
> Can you check the settings again please?

Seems fine on my end although I see "shared to bz_canusewhineatothers"
rather than "saved-searches" group. Probably has something to do with
it for non-devs.

Alternatively you can do a normal quick search with:
"assignee:maintainer-nee...@gentoo.org kw:PATCH"[1]

Although jstein's command seem to omit the rather old bugs (seems to
limit >2019 or so), so you'll get different results with just that.

On another note, hope this will serve as a reminder for people to
add the PATCH keyword (although pull requests are still more likely
to see attention for a maintainer-needed package).

[1] 
https://bugs.gentoo.org/buglist.cgi?quicksearch=assignee%3Amaintainer-needed%40gentoo.org%20kw%3APATCH
-- 
ionen


signature.asc
Description: PGP signature


Re: [gentoo-dev] Need help: Many patches for unmaintained packages

2021-12-31 Thread Martin Dummer

Am 31.12.21 um 06:51 schrieb Jonas Stein:

I prepared a public Bugzilla filter "m-n PATCH"

https://bugs.gentoo.org/buglist.cgi?cmdtype=dorem=run=m-n%20PATCH_id=85842



Hello Jonas,

when I try to open that link I receive an error message

"The search named /m-n PATCH/ has not been made visible to you."

Can you check the settings again please?

Martin




[gentoo-dev] [PATCH] sgml-catalog-r1.eclass: Remove obsolete environment files

2021-12-31 Thread James Le Cuirot
These files are only regenerated when gensgmlenv is present, but this
tool was part of sgmltools-lite, which was last-rited over a year ago.

The presence of 93sgmltools-lite can break tools such as
asciidoc. When SGML_CATALOG_FILES is defined, it automatically passes
the --catalogs option to xmllint, which uses the obsolete variable
over the updated catalogs listed in /etc/sgml/catalog.

Signed-off-by: James Le Cuirot 
---
 eclass/sgml-catalog-r1.eclass | 18 +-
 1 file changed, 5 insertions(+), 13 deletions(-)

diff --git a/eclass/sgml-catalog-r1.eclass b/eclass/sgml-catalog-r1.eclass
index 1e1f17815ac4..9f8bb13d6095 100644
--- a/eclass/sgml-catalog-r1.eclass
+++ b/eclass/sgml-catalog-r1.eclass
@@ -1,4 +1,4 @@
-# Copyright 2019 Gentoo Authors
+# Copyright 2019-2021 Gentoo Authors
 # Distributed under the terms of the GNU General Public License v2
 
 # @ECLASS: sgml-catalog-r1.eclass
@@ -9,9 +9,8 @@
 # @SUPPORTED_EAPIS: 7
 # @BLURB: Functions for installing SGML catalogs
 # @DESCRIPTION:
-# This eclass regenerates /etc/sgml/catalog, /etc/sgml.{,c}env
-# and /etc/env.d/93sgmltools-lite as necessary for the DocBook tooling.
-# This is done via exported pkg_postinst and pkg_postrm phases.
+# This eclass regenerates /etc/sgml/catalog as necessary for the DocBook
+# tooling. This is done via exported pkg_postinst and pkg_postrm phases.
 
 case ${EAPI:-0} in
7) ;;
@@ -50,16 +49,9 @@ sgml-catalog-r1_update_catalog() {
 
 # @FUNCTION: sgml-catalog-r1_update_env
 # @DESCRIPTION:
-# Regenerate environment variables and copy them to env.d.
+# Remove obsolete environment files. They can break tools such as asciidoc.
 sgml-catalog-r1_update_env() {
-   # gensgmlenv doesn't support overriding root
-   if [[ -z ${ROOT} && -x "${EPREFIX}/usr/bin/gensgmlenv" ]]; then
-   ebegin "Regenerating SGML environment variables"
-   gensgmlenv &&
-   grep -v export "${EPREFIX}/etc/sgml/sgml.env" > 
"${T}"/93sgmltools-lite &&
-   mv "${T}"/93sgmltools-lite 
"${EPREFIX}/etc/env.d/93sgmltools-lite"
-   eend "${?}"
-   fi
+   rm -f "${EROOT}"/etc/env.d/93sgmltools-lite 
"${EROOT}"/etc/sgml/sgml.{,c}env
 }
 
 sgml-catalog-r1_pkg_postinst() {
-- 
2.34.1




Re: [gentoo-dev] [PATCH] python-any-r1.eclass: Include proper python-exec dep in || deps

2021-12-31 Thread Michał Górny
On Fri, 2021-12-31 at 09:26 +0100, Michał Górny wrote:
> Enforce proper USE dependencies on dev-lang/python-exec from
> python-any-r1.  While we force all flags being enabled on it, there is
> still a possibility of temporary missync while a new target is being
> added.  The explicit dependency should be harmless and ensure correct
> package ordering, therefore avoiding errors like:
> 
> python-exec: EPYTHON value invalid (python3.10).
> 
> Closes: https://bugs.gentoo.org/830273
> 

Actually, this is wrong, as it adds the dep for Python 2 too.  I have
a better idea that avoids adding it to all packages' metadata.

-- 
Best regards,
Michał Górny




[gentoo-dev] [PATCH] python-any-r1.eclass: Include proper python-exec dep in || deps

2021-12-31 Thread Michał Górny
Enforce proper USE dependencies on dev-lang/python-exec from
python-any-r1.  While we force all flags being enabled on it, there is
still a possibility of temporary missync while a new target is being
added.  The explicit dependency should be harmless and ensure correct
package ordering, therefore avoiding errors like:

python-exec: EPYTHON value invalid (python3.10).

Closes: https://bugs.gentoo.org/830273
Signed-off-by: Michał Górny 
---
 eclass/python-any-r1.eclass | 15 +--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/eclass/python-any-r1.eclass b/eclass/python-any-r1.eclass
index 7af9474d9a1f..3d46a2eb4ca0 100644
--- a/eclass/python-any-r1.eclass
+++ b/eclass/python-any-r1.eclass
@@ -181,7 +181,12 @@ _python_any_set_globals() {
_python_export "${i}" PYTHON_PKG_DEP
 
# note: need to strip '=' slot operator for || deps
-   deps="${PYTHON_PKG_DEP/:0=/:0} ${deps}"
+   deps="
+   (
+   ${PYTHON_PKG_DEP/:0=/:0}
+   >=dev-lang/python-exec-2[python_targets_${i}]
+   )
+   ${deps}"
done
deps="|| ( ${deps})"
 
@@ -266,7 +271,13 @@ python_gen_any_dep() {
local i_depstr=${depstr//\$\{PYTHON_USEDEP\}/${PYTHON_USEDEP}}

i_depstr=${i_depstr//\$\{PYTHON_SINGLE_USEDEP\}/${PYTHON_SINGLE_USEDEP}}
# note: need to strip '=' slot operator for || deps
-   out="( ${PYTHON_PKG_DEP%=} ${i_depstr} ) ${out}"
+   out="
+   (
+   ${PYTHON_PKG_DEP%=}
+   >=dev-lang/python-exec-2[python_targets_${i}]
+   ${i_depstr}
+   )
+   ${out}"
done
echo "|| ( ${out})"
 }
-- 
2.34.1