Re: [devel] Излишнее увлечение макросом %name в спеке

[Denis Medvedev <nbr@altlinux.org>]

On Fri, 22 Mar 2024 10:16:50 +0300
Grigory Ustinov <grenka@altlinux.org> wrote:

> 22.03.2024 9:54, Dmitry V. Levin пишет:
> > On Fri, Mar 22, 2024 at 09:10:19AM +0300, Anton Farygin wrote:
> >> On 22.03.2024 06:11, Ilya Kurdyukov wrote:
> >>> Нашел сейчас в atop:
> >>>
> >>> URL: http://www.%{name}tool.nl
> >>>
> >>> Если мне захочется зайти на сайт, то я должен сначала вставить имя
> >>> пакета в URL?
> >> Да конечно это ошибка, надо написать где-то на www.altlinux.org в
> >> правилах что так делать нельзя.
> >>
> >> URL должен быть кликабельным из specfile из редактора, это удобно
> >> и полезно.
> > Я бы не утверждал так уж категорично.  У нас в Сизифе, например,
> > много таких вполне валидных конструкций, как %CPAN, например:
> 
> Давайте определимся для кого существует тэг URL? Для людей или для 
> сборочных скриптов?
Может стоит требовать добавлять тег HumanReadableURL: в котором обязан
быть человекочитаемый URL в случае URL для скриптов? 
Потому что люди должны иметь возможность посмотреть на исходники явно.
> 
> "Любой дурак может написать код, понятный компьютеру. Хороший 
> программист пишет код, понятный человеку." (с)
> 
> Тут выше было столько речей про качество спеков, что давать заднюю
> уже неприлично.
> 
> В случае с перлом, я думаю, что там действительно это может быть для 
> скриптов.
Добавить тег HumanSourceURL: в котором должен быть человекочитаемый URL
? 
> 
> >
> > $ git --git-dir=/people/specbot/public/specs.git grep -i -h '^URL:
> > %CPAN' @ |sort |uniq -c |sort -nr 581 URL: %CPAN %dist
> >      278 Url: %CPAN %module_name
> >       70 Url: %CPAN %module
> >       43 Url: %CPAN %dist
> >       41 Url: %CPAN %m_distro
> >       35 URL: %CPAN %module
> >        3 URL: %CPAN %module_name
> >        2 Url: %CPAN %oname
> >        1 Url: %CPAN X11-Xlib
> >        1 Url: %CPAN URI-Find
> >        1 Url: %CPAN Twiggy
> >        1 Url: %CPAN TOML
> >        1 Url: %CPAN Text-Xslate-Bridge-TT2Like
> >        1 Url: %CPAN Text-Xslate
> >        1 Url: %CPAN Text-TestBase
> >        1 Url: %CPAN Text-Markdown-Hoedown
> >        1 Url: %CPAN Test-Trap
> >        1 Url: %CPAN Test-TinyMocker
> >        1 Url: %CPAN Test-TCP
> >        1 Url: %CPAN Test-Spec
> >        1 Url: %CPAN Test-SharedFork
> >        1 Url: %CPAN Test-Requires
> >        1 Url: %CPAN Test-Most
> >        1 Url: %CPAN System-Command
> >        1 Url: %CPAN Sub-Exporter-Progressive
> >        1 Url: %CPAN strictures
> >        1 Url: %CPAN Stream-Buffered
> >        1 Url: %CPAN Starman
> >        1 Url: %CPAN Specio
> >        1 Url: %CPAN Socket-Netlink
> >        1 Url: %CPAN Server-Starter
> >        1 Url: %CPAN RRDTool-OO
> >        1 Url: %CPAN Router-Simple-Sinatraish
> >        1 Url: %CPAN Router-Simple
> >        1 Url: %CPAN Router-Boom
> >        1 Url: %CPAN Role-Tiny
> >        1 URL: %CPAN %real_name
> >        1 Url: %CPAN Protocol-WebSocket
> >        1 Url: %CPAN Proc-FastSpawn
> >        1 Url: %CPAN Plack-Middleware-Session
> >        1 Url: %CPAN Plack-Middleware-ReverseProxy
> >        1 Url: %CPAN Plack-Middleware-ETag
> >        1 Url: %CPAN Plack
> >        1 Url: %CPAN Parse-Functions
> >        1 Url: %CPAN OpenGL
> >        1 Url: %CPAN Object-Event
> >        1 Url: %CPAN Object-Container
> >        1 Url: %CPAN Net-Twitter
> >        1 Url: %CPAN Net-OpenSSH
> >        1 URL: %CPAN %name
> >        1 Url: %CPAN MouseX-Foreign
> >        1 Url: %CPAN MooseX-Clone
> >        1 Url: %CPAN Moo
> >        1 Url: %CPAN MojoX-Renderer-Xslate
> >        1 Url: %CPAN Mojolicious
> >        1 Url: %CPAN Module-Install-XSUtil
> >        1 Url: %CPAN Module-Install-ReadmeFromPod
> >        1 Url: %CPAN Module-Install-CheckLib
> >        1 Url: %CPAN Module-Implementation
> >        1 Url: %CPAN(%module)
> >        1 Url: %CPAN Minilla
> >        1 Url: %CPAN MetaCPAN-API
> >        1 Url: %CPAN Mcrypt
> >        1 Url: %CPAN Math-FFT
> >        1 Url: %CPAN Math-Complex
> >        1 Url: %CPAN Log-Any
> >        1 Url: %CPAN Lingua-JA-Regular-Unicode
> >        1 Url: %CPAN JavaScript-Value-Escape
> >        1 Url: %CPAN JavaScript-V8
> >        1 Url: %CPAN JavaScript-Minifier
> >        1 Url: %CPAN JavaScript-Beautifier
> >        1 Url: %CPAN IO-Socket-IP
> >        1 Url: %CPAN IO-FDPass
> >        1 Url: %CPAN HTTP-Session
> >        1 Url: %CPAN HTML-StickyQuery
> >        1 Url: %CPAN HTML-Shakan
> >        1 Url: %CPAN HTML-FillInForm-Lite
> >        1 Url: %CPAN Hash-MultiValue
> >        1 Url: %CPAN Gtk3
> >        1 Url: %CPAN Glib-Object-Introspection
> >        1 URL: %CPAN GD
> >        1 Url: %CPAN Future
> >        1 Url: %CPAN FormValidator-Lite
> >        1 Url: %CPAN Filesys-Notify-Simple
> >        1 Url: %CPAN File-Path-Tiny
> >        1 Url: %CPAN FCGI-Client
> >        1 Url: %CPAN ExtUtils-H2PM
> >        1 Url: %CPAN ExtUtils-CChecker
> >        1 Url: %CPAN Eval-Context
> >        1 Url: %CPAN Error
> >        1 Url: %CPAN DynaLoader-Functions
> >        1 URl: %CPAN %dist
> >        1 Url: %CPAN Devel-PatchPerl
> >        1 Url: %CPAN Devel-CallParser
> >        1 Url: %CPAN Devel-CallChecker
> >        1 Url: %CPAN Debug-Client
> >        1 Url: %CPAN Data-Section-Simple
> >        1 URL: %CPAN Data-MessagePack
> >        1 Url: %CPAN Data-Localize
> >        1 Url: %CPAN Dancer-Template-Xslate
> >        1 Url: %CPAN Dancer-Session-Cookie
> >        1 Url: %CPAN Dancer-Plugin-Database
> >        1 Url: %CPAN Dancer
> >        1 Url: %CPAN CSS-Compressor
> >        1 Url: %CPAN CPAN-Perl-Releases
> >        1 Url: %CPAN Convert-Bencode
> >        1 Url: %CPAN Config-Identity
> >        1 Url: %CPAN Class-Load-XS
> >        1 Url: %CPAN CHI
> >        1 Url: %CPAN CGI-Emulate-PSGI
> >        1 Url: %CPAN CGI-Compile
> >        1 Url: %CPAN Cairo-GObject
> >        1 Url: %CPAN Boulder
> >        1 URL: %CPAN B-Flags
> >        1 Url: %CPAN B-Debug
> >        1 URL: %CPAN B-C
> >        1 Url: %CPAN App-perlbrew
> >        1 Url: %CPAN App-cpanminus
> >        1 Url: %CPAN Apache-LogFormat-Compiler
> >        1 Url: %CPAN AnyEvent-HTTP
> >        1 Url: %CPAN AnyEvent-Gearman
> >        1 Url: %CPAN AnyEvent-Fork
> >        1 Url: %CPAN Amon2-Lite
> >        1 Url: %CPAN Amon2
> >
> >
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel


--