public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Martin Dummer <martin.dummer@gmx.net>
To: gentoo-dev@lists.gentoo.org, Sam James <sam@gentoo.org>
Subject: Re: [gentoo-dev] [PATCH] vdr-plugin-2.eclass: make qa warning conditional
Date: Thu, 9 May 2024 15:02:46 +0200	[thread overview]
Message-ID: <5354f18f-ee4a-4475-9b7e-6502886335ea@gmx.net> (raw)
In-Reply-To: <87le4ji3ox.fsf@gentoo.org>

[-- Attachment #1: Type: text/plain, Size: 1558 bytes --]


Am 09.05.24 um 14:13 schrieb Sam James:
> Martin Dummer<martin.dummer@gmx.net>  writes:
>
>> Maybe we can agree that the qa-warnings in vdr-eclass make more sense if i change them to "eawarn" or "einfo"?
>>
> Sure, make them eqawarn.

Hmm - current state of vdr-plugin-2.eclass is: there are many "eqawarn"
in there.



>> In my opinion, most plugins in the vdr context will practically not develop any further anyway. It is more important to
>> keep the current status of vdr-software in the ecosystem up to date as well as possible.
>>
>> So I need a practical useful approach instead of a fundamental discussion please.
> My point is that the QA warnings should exist, and you can worry about
> making them "developer-only" in future. Right now, they seem useful, and
> the things they flag need to be addressed.
>
Basically yes, but here (vdr-plugins) the qawarn are used more in a way
"to remind the plugin developers to adapt their sources to newer vdr
build environment" or "the way i18n implemented has changed"

The eclass fixes these problems with standardized quirks, the "equawarn"
messages show when these quirks are applied.

IMHO its not necessary to tell that to any user, only for interested
packagers when they are bored and look out for some extra work. That's
why I made the warnings conditional, printed out when the variable
"VDR_MAINTAINTER_MODE" is set to a not-empty value.

Finally, I am interested in an opinion whether this is acceptable or
not, otherwise I tend to remove the warnings at all.

[-- Attachment #2: Type: text/html, Size: 2577 bytes --]

  reply	other threads:[~2024-05-09 13:02 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-01 14:10 [gentoo-dev] [PATCH] vdr-plugin-2.eclass: make qa warning conditional Martin Dummer
2024-05-01 15:07 ` Eli Schwartz
2024-05-03  4:39   ` Sam James
2024-05-09 12:08     ` Martin Dummer
2024-05-09 12:13       ` Sam James
2024-05-09 13:02         ` Martin Dummer [this message]
2024-05-09 13:08           ` Sam James
2024-05-10 15:42             ` Martin Dummer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5354f18f-ee4a-4475-9b7e-6502886335ea@gmx.net \
    --to=martin.dummer@gmx.net \
    --cc=gentoo-dev@lists.gentoo.org \
    --cc=sam@gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox