Re: [gentoo-dev] Re: [gentoo-commits] repo/gentoo:master commit in: dev-qt/qtmultimedia/

2016-04-18 Thread M. J. Everitt
On 18/04/16 16:47, Michał Górny wrote:
> This is invalid. Replacing invalid package names with other invalid
> names is no fix. It is ugly Gentoo-style hackery which proves that
> developers prefer trying randomly changing something until QA check
> doesn't trigger over reading the documentation. As a result, now
> a number packages with visible QA issues have been turned into a number
> of packages with QA issues that aren't caught by the schema due to
> regexp limitations.
>
> Please go over all the packages you've touched and fix the silent
> issues you introduced. I can't afford to waste more time writing more
> complex QA checks that are going to cover all the ways Gentoo
> developers can screw up such a simple thing as package *name*.
>
I can't help but want to write .. "If you want a job doing ... "

+1 monp.





signature.asc
Description: OpenPGP digital signature


[gentoo-dev] Re: [gentoo-commits] repo/gentoo:master commit in: dev-qt/qtmultimedia/

2016-04-18 Thread Michał Górny
On Mon, 18 Apr 2016 08:13:34 + (UTC)
"Patrice Clement"  wrote:

> commit: d43acba31bb5d4f07e59479cd5728abc9367a9ca
> Author: Patrice Clement  gentoo  org>
> AuthorDate: Mon Apr 18 06:46:09 2016 +
> Commit: Patrice Clement  gentoo  org>
> CommitDate: Mon Apr 18 07:58:25 2016 +
> URL:https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=d43acba3
> 
> dev-qt/qtmultimedia: Fix metadata.xml file.
> 
> Package-Manager: portage-2.2.26
> 
>  dev-qt/qtmultimedia/metadata.xml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/dev-qt/qtmultimedia/metadata.xml 
> b/dev-qt/qtmultimedia/metadata.xml
> index c0d6a4e..ab0a181 100644
> --- a/dev-qt/qtmultimedia/metadata.xml
> +++ b/dev-qt/qtmultimedia/metadata.xml
> @@ -9,8 +9,8 @@
>   Add support for exceptions - like 
> catching them
>   inside the event loop (recommended by upstream)
>   Enable EGL/GLES2 integration
> - Enable audio support via 
> media-libs/gstreamer:1.0
> - Enable audio support via 
> media-libs/gstreamer:0.10
> + Enable audio support via 
> media-libs/gstreamer-1.0
> + Enable audio support via 
> media-libs/gstreamer-0.10

This is invalid. Replacing invalid package names with other invalid
names is no fix. It is ugly Gentoo-style hackery which proves that
developers prefer trying randomly changing something until QA check
doesn't trigger over reading the documentation. As a result, now
a number packages with visible QA issues have been turned into a number
of packages with QA issues that aren't caught by the schema due to
regexp limitations.

Please go over all the packages you've touched and fix the silent
issues you introduced. I can't afford to waste more time writing more
complex QA checks that are going to cover all the ways Gentoo
developers can screw up such a simple thing as package *name*.

>   Build QML/QtQuick bindings and imports
>   Build the QtMultimediaWidgets module
>   
> 



-- 
Best regards,
Michał Górny



pgpGfbp5ofwKA.pgp
Description: OpenPGP digital signature