From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 7FB6A158064 for ; Thu, 9 May 2024 13:02:53 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id B5983E2C5B; Thu, 9 May 2024 13:02:49 +0000 (UTC) Received: from mout.gmx.net (mout.gmx.net [212.227.15.19]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (prime256v1) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 2B807E2C57 for ; Thu, 9 May 2024 13:02:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmx.net; s=s31663417; t=1715259767; x=1715864567; i=martin.dummer@gmx.net; bh=QZ7N/RZLlApk61oVMuEgOP3jUQoep1aIacqtRstknxw=; h=X-UI-Sender-Class:Content-Type:Message-ID:Date:MIME-Version: Subject:To:References:From:In-Reply-To:cc: content-transfer-encoding:content-type:date:from:message-id: mime-version:reply-to:subject:to; b=iW7S8KWW8X8wLJSP4fPby+ncnaElllL3ypQkf2eZ1+KvPRE7X6MlGc+biVgip+MV ACVPwMvQasI5H1T4Cbs7kdDCk3BrOlK278SaVIKJjZKQ0ImtDlel4k7+9CfJqlC6P xBK2UFUSBOhkrvx+QV288uwRYcznL1vi/m9qWKcsbEkJ/IFM2yrlOq+1CNVTM7td0 3A308MiFOYWsvs1e4C8Anv+ANF0CAqcAEQxYfVX65FKHZYaiG+ewGeO+qqiyGbmBE cZEhtMgR7/y0IrP+IleaGQ4nGsYnTzOSX3amfpgYOreSdMzbtNxQkAcq7qLtQV+hZ 2QHVQJKI0ZixmqWpOg== X-UI-Sender-Class: 724b4f7f-cbec-4199-ad4e-598c01a50d3a Received: from [192.168.166.209] ([146.52.8.170]) by mail.gmx.net (mrgmx004 [212.227.17.190]) with ESMTPSA (Nemesis) id 1M4JqV-1s5KP13hUf-003g2P; Thu, 09 May 2024 15:02:46 +0200 Content-Type: multipart/alternative; boundary="------------SxYYX0hc2zXAO2ZUj4PSumBd" Message-ID: <5354f18f-ee4a-4475-9b7e-6502886335ea@gmx.net> Date: Thu, 9 May 2024 15:02:46 +0200 Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-dev@lists.gentoo.org Reply-to: gentoo-dev@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [gentoo-dev] [PATCH] vdr-plugin-2.eclass: make qa warning conditional To: gentoo-dev@lists.gentoo.org, Sam James References: <7e5a29e3-43e4-4af9-b7c5-660501a027a8@gmx.net> <4dc6df27-4efa-47b1-8ddd-4bdd08a08b21@gmail.com> <87cyq3ldah.fsf@gentoo.org> <5803c021-601d-48b2-bbfa-ce5de555e1cf@gmx.net> <87le4ji3ox.fsf@gentoo.org> Content-Language: en-US From: Martin Dummer Autocrypt: addr=martin.dummer@gmx.net; keydata= xsFNBFyXOk0BEAC0SrM62OIarfdzmzMLdbTBiWU2hFyLxVD2NHTe2fRnwtH1ergBLyrjuXx3 xM529HDCVcIJoKGgfm5TbmhDN2o5TVeamh9YOzLG60424R+u0tjEh6py9Bjg3VcKT4u11VOU KQUkOVRkwSo5lsWu6JZ+CkpS+pOD42wl3gSOO6ZvtMF0YHudW8NKSaXNz7khMakIhiWqU1oe 1q3XBZswpb5i1x415Wx7SPovBTs7Aw8xDb5qSndBts1lE56mq0i77P0u3TUT1/yE5kzEh/Nv 0VwcFfs2VAQTKAfrZrFVY2PVis8uhqQrA6a74YSkHUEDL/6A3RoLE2w6ZbC/bIPhcpHnvTXa 1cm++Wp5mg9ivAul22Lxc1d2TqK2nAjqEu5BpdIjdhYtXDYR5rLF9fA5OBy2ycUOK5QXtr9n BbzljWrnlWp4pJBdBNH3UqwrLJi1qfGpbK0Qr0NYYfcrt+iGkvmWxdu/CmrNEz/DsBuWvHMY W0G0mtopwZ4M8jPaYjdwCV75DifjByidEIoy1W9MMsI/KA7e5ZHNjk34bJUV8cuxT3nyttCV +XZjXkQb3YuypKOdFuerUYSx9cGQSXZysdkdny/B4XB9cMEqrwcKw7sD8dlFO5Hqe8xxJC40 xRY5/HT1jzf/ZLiH9iJgX55t8ncLmJDBKeySRjEfLqajsj9scwARAQABzSVNYXJ0aW4gRHVt bWVyIDxtYXJ0aW4uZHVtbWVyQGdteC5uZXQ+wsGUBBMBCAA+AhsDBQsJCAcCBhUKCQgLAgQW AgMBAh4BAheAFiEEh6On4PuhCLLmLjy5zl8+m23gXRIFAmYoG3kFCQxiGZMACgkQzl8+m23g XRLiCRAAl5n0n2TrCSgp10Mk0qljxplf/iP3U/+mWq7ZeXl6De23jTGKCtne4e+DVfiiEf8N d+9H7UsdHxfEg6ajQcbTcwPb1NrGxxwJVEhPB3dTg2jgLJaI7wSt8LJcTHwbHt9GDbOxhXRH 8jt+Cq3lmpFesa5j5Z7KUazI+yXrMirjdB4Oj32jjv+Uj0S0GDEXQzZGjWoE2r87KXjV5VQ8 PYpdSD6jIiVa+BAG5x6JgKHUNahB8IDk5Tf3SMUW7qyfssSUmOZDgR0mGOaSxSJj0CJxp9e2 JIHoD4z9VhaNDeeHkzgsNXZtqvTue9JteQTOM8/pwfdPRng2Aoux87U9D0SaUDuqO6WFOhRQ j2O76U55LFj4s+d4XjGVp3pF13iTPEgZ9sS5cvlYQ7V4Dh2aNJxY+Ubx3azzPDlvgUsyYpeZ 1r3hBDHjx8KBe04dyeOjOPOl8atIBmGIk9tSyQUE3qtRIycoFw/UZ/D7yiMcMygWuxJ0KLtl Hpqv1UJXRFNTNYW400I9P41cYz0LM7Nr1j005bIyGkhqBqnUapi9uXicPBj3+xEcfGPAd/OA 38qKPUPtg0hrQs9KzkGtYHtTommeApkuZOKHOBAwTQmEzEga0B56AT63dzKnqadF9Bttac8A GjIZfQOjhAqThWw/7Sko5PmplJMcadLU5hwCVPllVnbOwE0EXJc6TQEIALUoGvhnsqO76OLt dysas6bLdEFjqsU8AKqyPHAL7b9Fvd2d9J5JPED0aGRAuR6GPpRhLHNUWTxzeKGgCl/xMay1 Kuo+A0LOACHBXMFHHArl+WVS6bCbFKJyYGGyxik9bfdYfm8gMvMNmJUjdGyWGLJ6+Pp4Q1Iy 8KkT/JcbdhcJ2y2fnNOrAu+ba2C5Oc0lJwDzZ59PtvhtuxFlFdOGOulDDNjLHPWP0vXr8yjQ X7b54uVxUFSBqo+1rzUX6Bkz3TSjNpfMDAzC/MgwYVOlpTDAbKNl5yABGfjER0DueyQogA5K o2A1Tk410zc5uiOg0Y4jjgA6tBPZ/NH4kr1FWzsAEQEAAcLBfAQYAQgAJgIbDBYhBIejp+D7 oQiy5i48uc5fPptt4F0SBQJmKBt5BQkMYhmTAAoJEM5fPptt4F0SDEoP/1ooRd8JS4nMYtWU QvqcucVl7V/hw/G65OmdU504jbP5K3xgV3/nMBkVmPYqi/Gtp7LfNkMSflRZXyFGsFyhUIr4 0Eya3w+mS3EnbaRh1ODOe/wOcdMJq+SzrAPY+9AbbIXhRCNvrFpTWUpsGfR3Cz+V+sCZQ7fy sQF86JpEyR+Stm22wXsuvfIzIZID3SKz3IYC+iYgt5h46v/xHid4vVyRPDypOtZ7giYn4HC7 XArdyXoWf0Id5OtzZCWGcNlZqaKZEzfz1IPzmAN7b/vgwpVnMD708lqa7zuv2Dv138WmkIgV EzgAvwCIB6B9jrL28PFI5ocs+EK2aAQj6x8Ite8sk/ozs219X3An+G1Rt+JHHWZc5DOxQFBs faddCZcffEmrrCdzUKV1/ba40donE8l6CSWC9mWGb1z58PH+gfpiTFvOgANGmQMpa71gjSLW j2dd3MXzIs6cXz2KyzfCm0SVYvkpNWS1l9FGE9m0BO2Mc7+wvcsFHptQSSCg+lMK+FzgwN9M wFTdxQllmdseKG5OkntdRLbQiHYqEz/H+Enwh+3HfrCsumLavXvOwuXQV3Z7QXxrtzzIdykb TXyBqji3zxdNt4YNSTelO1JhuSlv+XOdEKTBdmX8zZLT4OpZM7jGHC0SPdzqLjAbjUgXzg+U aFP1F3EqAqsljh7JsazD In-Reply-To: <87le4ji3ox.fsf@gentoo.org> X-Provags-ID: V03:K1:+yLDYObEf7LuA1VEfdWV7hRnvN7vbIVm5ff50PjXFQwtFccrBJ/ za1FTs267SWtNoNB7v6krwOzSwqD8ZU+uX9JDLXmwVomjmLeJxWT/ciwCct35Mxr+iIExxy qOLE5ZDcyaFEhV2bGxkBJFKcr2CglNUQiSCoRmnpkizWHjk4iGW/YE+2vRQrPh7B7FSp8V7 B3dJDonb0RpaHc/9RJkBA== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:Tro9FmsEGEo=;phzEYLRr+huo168QatMEQ/iX96G +kpxABTsy1PfUEzg/R/cMWnnjMGPSVk01NWpu+eNlXiEBk5q21X6wCHgVAt2chWUvBqNGkknf 8+hrmIAujWG82lkYSVe44mit/p1QC+mAyB+n4insQVrnxsYOeqkvU+J06YTh2qeQ7a3XvRQRb W/fv/89iLsB5zO9gJgLC7MQdB2yFiHcy31GQEo/cdwN0RRys6o3p0c8Xh/IdtHXGtZ8avvu9f PyeJvZyxdmfEhUpV4KSJfo9in+eUyMleWd3xlfEZipZHXwLMUTnC81ZLN0c+SLUxt80htB0jT C/C6vwtncs+duGLq5iLENGlglNjACaX4mIGpzlvd83b7+/fJh2aD0v+tVUbfrn9m4xn5XZmml MJjHq+qSkFeOV9hSgCQUu4jdbEOZ5wLNFT7erXaNXSs48KfbCC2WaG0Bg2uAgFUzIp2UlQDY4 gIstn1Zr4mRh71HqCWIrBbUG+OU6mG+hRCLB0wZXKxdk6wGHl2CY9Aha+d1UtCkvX3sYyZ+z0 E4Dyf4oCwycjoT9MFLSIFc72h5i2WuX8gsN/IlIAWsV/0sBuN68dJhDLI38wiikuoqPTQY505 Oxf7SycUpGRYFBw+qxvj1idmhCDs/0oLusU9TJDegTymrfu8ZXIQVviA3KRAv3Jx5fS/KgvZL YTry0IFET9J1xllSjKy9osgwU92M4QbqAgpPM700oBYqdxCIihbtqRvaHGu2pLevrw3wc9AH6 uFNpB5dUUR0vGne4iGPRVVSKY8XSJUx8MVQ4NQQbqoQwsMUtfm49Ag+WZqoTGpHbnSSC2sIip iaxf2gNd6qEaFIMY/SAmuymuzDeoQx/UU5RzynxHEbnkA= X-Archives-Salt: 62f246d6-179e-421e-a402-2902f16c3733 X-Archives-Hash: eb9c522a4f87ff173771d695e345f912 This is a multi-part message in MIME format. --------------SxYYX0hc2zXAO2ZUj4PSumBd Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable Am 09.05.24 um 14:13 schrieb Sam James: > Martin Dummer writes: > >> Maybe we can agree that the qa-warnings in vdr-eclass make more sense i= f 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 dev= elop 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 discussi= on 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. --------------SxYYX0hc2zXAO2ZUj4PSumBd Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 7bit


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.

    

  


--------------SxYYX0hc2zXAO2ZUj4PSumBd--