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

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

[Vitaly Lipatov]

Заметил, что по какой-то причине в спеках задействуют макрос %name в URL 
или в Source (в элементах пути, не в %name-%version, конечно).

На мой взгляд, это сделано для того, чтобы затруднить доступ к сайту 
программы и исходным текстам. Этот путь указывается для человека, а 
пользоваться им невозможно.

Примеры:
q/qxmledit/qxmledit.spec:URL: https://github.com/lbellonda/%name
r/radeontop/radeontop.spec:Url: https://github.com/clbr/%name
r/radiusclient-ng/radiusclient-ng.spec:URL: 
http://developer.berlios.de/projects/%name/
r/rapidyaml/rapidyaml.spec:URL: https://github.com/biojppm/%name
r/recover/recover.spec:Url: http://recover.sourceforge.net/linux/%name
r/recoverdm/recoverdm.spec:Url: http://www.vanheusden.com/%name
и
j/jamin/jamin.spec:Source: http://prdownloads.sourceforge.net/%name/
j/js/js.spec:Source: ftp://ftp.mozilla.org/pub/mozilla.org/%name/
k/klavaro/klavaro.spec:Source: http://downloads.sourceforge.net/%name/

Количество:
$ git grep -i "^URL:" | grep "%name" | wc -l
342

$ git grep -i "^Source" | sed -e "s|%name-%version.*||" | grep "://" | 
grep "%name" | wc -l
427

Хотел бы попросить обратить на это внимание и не использовать макросы 
имени в минимально необходимых случаях.


-- 
С уважением,
Виталий Липатов,
ALT Linux Team

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

[Yuri Sedunov]

В Чт, 29/02/2024 в 14:58 +0300, Vitaly Lipatov пишет:
> Заметил, что по какой-то причине в спеках задействуют макрос %name в
> URL или в Source (в элементах пути, не в %name-%version, конечно).
> 
> На мой взгляд, это сделано для того, чтобы затруднить доступ к сайту 
> программы и исходным текстам. Этот путь указывается для человека, а 
> пользоваться им невозможно.
> 
> Примеры:
...
> k/klavaro/klavaro.spec:Source:
> http://downloads.sourceforge.net/%name/

У меня нет проблем с чтением спеков.

$ rpmbE klavaro.spec |grep 'Url\|Source' 
Url: http://klavaro.sourceforge.net/en/
Source: http://downloads.sourceforge.net/klavaro/klavaro-3.14.tar.bz2


-- 
Yuri N. Sedunov

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

[Anton Farygin]

On 29.02.2024 14:58, Vitaly Lipatov wrote:
> Хотел бы попросить обратить на это внимание и не использовать макросы 
> имени в минимально необходимых случаях. 

Поддерживаю. И здесь даже не столько проблема в чтении, которую можно 
решить выполнив парсер, но и проблема в изменениях, когда по каким-то 
причинам %name надо поменять на что-то другое.

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

[Ilya Kurdyukov]

Нашел сейчас в atop:

URL: http://www.%{name}tool.nl

Если мне захочется зайти на сайт, то я должен сначала вставить имя 
пакета в URL?

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

[Anton Farygin]

On 22.03.2024 06:11, Ilya Kurdyukov wrote:
> Нашел сейчас в atop:
>
> URL: http://www.%{name}tool.nl
>
> Если мне захочется зайти на сайт, то я должен сначала вставить имя 
> пакета в URL? 


Да конечно это ошибка, надо написать где-то на www.altlinux.org в 
правилах что так делать нельзя.

URL должен быть кликабельным из specfile из редактора, это удобно и полезно.

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

[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, например:

$ 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


-- 
ldv

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

[Grigory Ustinov]

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? Для людей или для 
сборочных скриптов?

"Любой дурак может написать код, понятный компьютеру. Хороший 
программист пишет код, понятный человеку." (с)

Тут выше было столько речей про качество спеков, что давать заднюю уже 
неприлично.

В случае с перлом, я думаю, что там действительно это может быть для 
скриптов.

>
> $ 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
>
>

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

[Denis Medvedev]

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


-- 

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

[Anton Farygin]

On 22.03.2024 09:54, Dmitry V. Levin wrote:
> 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, например:
>
Почти все пакеты с CPAN - генераты. Думаю что робот точно так-же может 
поправить сразу на чистый URL без тэгов.

А вообще таких пакетов действительно очень много, чуть больше 2000

там много "стандартного" вроде:

z/zathura/zathura.spec:Url: https://pwmt.org/projects/%name/
z/zlib-ng/zlib-ng.spec:Url: https://github.com/%name/%name
z/zn_poly/zn_poly.spec:Url: https://gitlab.com/sagemath/%name
z/zycore/zycore.spec:Url: https://github.com/zyantific/%name-c
z/zydis/zydis.spec:Url: https://%name.re/

Но есть и забавное:

l/libuhttpmock/libuhttpmock.spec:Url: https://gitlab.com/%_name/%_name
q/qxmpp-qt5/qxmpp-qt5.spec:Url: https://github.com/%_name-project/%_name
r/rpcsx/rpcsx.spec:Url: https://%name.github.io/%name-site/
r/rpmidentity/rpmidentity.spec:Url: http://git.altlinux.org/gears/%(echo 
%name |cut -b1)/%name.git
y/yecht/yecht.spec:URL:      http://github.com/%{cluster}/%{name}

Написать скрипт, который это всё превратит в нормальный URL не сложно, 
но это будет 2000 изменений в пакетах.

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

[Volodia Selezniov]

On Thu, Feb 29, 2024 at 02:58:08PM +0300, Vitaly Lipatov wrote:
> Заметил, что по какой-то причине в спеках задействуют макрос %name в URL 
> или в Source (в элементах пути, не в %name-%version, конечно).
> 
> На мой взгляд, это сделано для того, чтобы затруднить доступ к сайту 
> программы и исходным текстам. Этот путь указывается для человека, а 
> пользоваться им невозможно.

$ grep -i url *spec
URL: https://github.com/lbellonda/%name
$ rpmspec -q --qf '%{url}\n' *spec 2>/dev/null
https://github.com/lbellonda/qxmledit

> Примеры:
> q/qxmledit/qxmledit.spec:URL: https://github.com/lbellonda/%name
> r/radeontop/radeontop.spec:Url: https://github.com/clbr/%name
> r/radiusclient-ng/radiusclient-ng.spec:URL: 
> http://developer.berlios.de/projects/%name/
> r/rapidyaml/rapidyaml.spec:URL: https://github.com/biojppm/%name
> r/recover/recover.spec:Url: http://recover.sourceforge.net/linux/%name
> r/recoverdm/recoverdm.spec:Url: http://www.vanheusden.com/%name
> и
> j/jamin/jamin.spec:Source: http://prdownloads.sourceforge.net/%name/
> j/js/js.spec:Source: ftp://ftp.mozilla.org/pub/mozilla.org/%name/
> k/klavaro/klavaro.spec:Source: http://downloads.sourceforge.net/%name/
> 
> Количество:
> $ git grep -i "^URL:" | grep "%name" | wc -l
> 342
> 
> $ git grep -i "^Source" | sed -e "s|%name-%version.*||" | grep "://" | 
> grep "%name" | wc -l
> 427
> 
> Хотел бы попросить обратить на это внимание и не использовать макросы 
> имени в минимально необходимых случаях.

-- 
   WBR,
   Volodia

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

[Anton Farygin]

On 24.03.2024 00:45, Volodia Selezniov wrote:
> On Thu, Feb 29, 2024 at 02:58:08PM +0300, Vitaly Lipatov wrote:
>> Заметил, что по какой-то причине в спеках задействуют макрос %name в URL
>> или в Source (в элементах пути, не в %name-%version, конечно).
>>
>> На мой взгляд, это сделано для того, чтобы затруднить доступ к сайту
>> программы и исходным текстам. Этот путь указывается для человека, а
>> пользоваться им невозможно.
> $ grep -i url *spec
> URL:https://github.com/lbellonda/%name
> $ rpmspec -q --qf '%{url}\n' *spec 2>/dev/null
> https://github.com/lbellonda/qxmledit

URL из спек, по хорошему, должен быть кликабельным из редактора.

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

[Dmitry V. Levin]

On Sun, Mar 24, 2024 at 05:31:42PM +0300, Anton Farygin wrote:
> On 24.03.2024 00:45, Volodia Selezniov wrote:
> > On Thu, Feb 29, 2024 at 02:58:08PM +0300, Vitaly Lipatov wrote:
> >> Заметил, что по какой-то причине в спеках задействуют макрос %name в URL
> >> или в Source (в элементах пути, не в %name-%version, конечно).
> >>
> >> На мой взгляд, это сделано для того, чтобы затруднить доступ к сайту
> >> программы и исходным текстам. Этот путь указывается для человека, а
> >> пользоваться им невозможно.
> > $ grep -i url *spec
> > URL:https://github.com/lbellonda/%name
> > $ rpmspec -q --qf '%{url}\n' *spec 2>/dev/null
> > https://github.com/lbellonda/qxmledit
> 
> URL из спек, по хорошему, должен быть кликабельным из редактора.

URL в пакете - это в первую очередь метаинформация собранного пакета,
поэтому никто изначально не стремился к тому, чтобы URL в спеке был
записан без использования макросов.  Всегда хотелось, чтобы спеки были
поддерживаемыми и человекочитаемыми, но не все считают, что из этого
вытекает, что URL в спеке всегда следует записывать без использования
макросов.


-- 
ldv

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

[Sergey V Turchin]

On Monday, 25 March 2024 15:41:04 MSK Dmitry Levin wrote:

[...]
> не все считают, что из этого
> вытекает, что URL в спеке всегда следует записывать без использования
> макросов.
+1
Или нужно использовать спековый процессор.
Типа LO Writer, только для spec-файлов. ;-)

-- 
Regards, Sergey.

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

[Dmitry V. Levin]

On Mon, Mar 25, 2024 at 04:13:13PM +0300, Sergey V Turchin wrote:
> On Monday, 25 March 2024 15:41:04 MSK Dmitry Levin wrote:
> 
> [...]
> > не все считают, что из этого
> > вытекает, что URL в спеке всегда следует записывать без использования
> > макросов.
> +1
> Или нужно использовать спековый процессор.

Вроде бы есть уже, rpmspec называется. :)


-- 
ldv

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

[Sergey V Turchin]

On Monday, 25 March 2024 19:36:31 MSK Dmitry Levin wrote:
> On Mon, Mar 25, 2024 at 04:13:13PM +0300, Sergey V Turchin wrote:
> > On Monday, 25 March 2024 15:41:04 MSK Dmitry Levin wrote:
> > 
> > [...]
> > 
> > > не все считают, что из этого
> > > вытекает, что URL в спеке всегда следует записывать без использования
> > > макросов.
> > 
> > +1
> > Или нужно использовать спековый процессор.
> 
> Вроде бы есть уже, rpmspec называется. :)
Не! Надо WYSIWYG. ;-)

-- 
Regards, Sergey.

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

[Anton Farygin]

On 25.03.2024 15:41, Dmitry V. Levin wrote:
> On Sun, Mar 24, 2024 at 05:31:42PM +0300, Anton Farygin wrote:
>> On 24.03.2024 00:45, Volodia Selezniov wrote:
>>> On Thu, Feb 29, 2024 at 02:58:08PM +0300, Vitaly Lipatov wrote:
>>>> Заметил, что по какой-то причине в спеках задействуют макрос %name в URL
>>>> или в Source (в элементах пути, не в %name-%version, конечно).
>>>>
>>>> На мой взгляд, это сделано для того, чтобы затруднить доступ к сайту
>>>> программы и исходным текстам. Этот путь указывается для человека, а
>>>> пользоваться им невозможно.
>>> $ grep -i url *spec
>>> URL:https://github.com/lbellonda/%name
>>> $ rpmspec -q --qf '%{url}\n' *spec 2>/dev/null
>>> https://github.com/lbellonda/qxmledit
>> URL из спек, по хорошему, должен быть кликабельным из редактора.
> URL в пакете - это в первую очередь метаинформация собранного пакета,
> поэтому никто изначально не стремился к тому, чтобы URL в спеке был
> записан без использования макросов.  Всегда хотелось, чтобы спеки были
> поддерживаемыми и человекочитаемыми, но не все считают, что из этого
> вытекает, что URL в спеке всегда следует записывать без использования
> макросов.
>
>
Да, я тоже придерживался этого мнения и усложнял себе жизнь. Но можно же 
сделать хорошо сразу ?

По моему мнению, использование макроса %name должно быть оправдано. Ну, 
например, при переименовании проекта не надо будет вносить изменение в URL.

Но в большинстве случаев так не работает.

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

[Grigory Ustinov]

22.03.2024 12:25, Anton Farygin пишет:
> On 22.03.2024 09:54, Dmitry V. Levin wrote:
>> 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, например:
>>
> Почти все пакеты с CPAN - генераты. Думаю что робот точно так-же может 
> поправить сразу на чистый URL без тэгов.
>
> А вообще таких пакетов действительно очень много, чуть больше 2000
>
> там много "стандартного" вроде:
>
> z/zathura/zathura.spec:Url: https://pwmt.org/projects/%name/
> z/zlib-ng/zlib-ng.spec:Url: https://github.com/%name/%name
> z/zn_poly/zn_poly.spec:Url: https://gitlab.com/sagemath/%name
> z/zycore/zycore.spec:Url: https://github.com/zyantific/%name-c
> z/zydis/zydis.spec:Url: https://%name.re/
>
> Но есть и забавное:
>
> l/libuhttpmock/libuhttpmock.spec:Url: https://gitlab.com/%_name/%_name
> q/qxmpp-qt5/qxmpp-qt5.spec:Url: https://github.com/%_name-project/%_name
> r/rpcsx/rpcsx.spec:Url: https://%name.github.io/%name-site/
> r/rpmidentity/rpmidentity.spec:Url: 
> http://git.altlinux.org/gears/%(echo %name |cut -b1)/%name.git
> y/yecht/yecht.spec:URL:      http://github.com/%{cluster}/%{name}
>
> Написать скрипт, который это всё превратит в нормальный URL не сложно, 
> но это будет 2000 изменений в пакетах.
Напиши. Пусть хотя бы в @everybody пакетах будут нормальные урлы. А там 
глядишь может и кто-нибудь заапрувит в своих пакетах? Лучше конечно 
такое делать от имени какого-то бота, иначе repology с ума сойдёт:) Как 
вариант, могу запустить такое NMU для змеиных пакетов.
>
>
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel

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

[Ruslandh]

[-- Attachment #1.1.1: Type: text/plain, Size: 1247 bytes --]

26.03.2024 11:20, Anton Farygin пишет:
> Да, я тоже придерживался этого мнения и усложнял себе жизнь. Но можно же 
> сделать хорошо сразу ?
> 
> По моему мнению, использование макроса %name должно быть оправдано. Ну, 
> например, при переименовании проекта не надо будет вносить изменение в URL.
> 
> Но в большинстве случаев так не работает.
Мне как-то кажется, что такие подстановки создаются скритами, а не 
людьми, типа github2spec (не утверждаю что именно этим скриптом, но 
похожим по функциональности), дамаю так потому, что человеку было бы 
проще вставить url напрямую, без подстановки макроса, а вот скрипт мог 
спокойно "оптимизировать" такой спек.

---------------------------------------------------
С уважением, Хихин Руслан


[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 657 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

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

[Anton Farygin]

On 27.03.2024 09:26, Ruslandh wrote:
> 26.03.2024 11:20, Anton Farygin пишет:
>> Да, я тоже придерживался этого мнения и усложнял себе жизнь. Но можно 
>> же сделать хорошо сразу ?
>>
>> По моему мнению, использование макроса %name должно быть оправдано. 
>> Ну, например, при переименовании проекта не надо будет вносить 
>> изменение в URL.
>>
>> Но в большинстве случаев так не работает.
> Мне как-то кажется, что такие подстановки создаются скритами, а не 
> людьми, типа github2spec (не утверждаю что именно этим скриптом, но 
> похожим по функциональности), дамаю так потому, что человеку было бы 
> проще вставить url напрямую, без подстановки макроса, а вот скрипт мог 
> спокойно "оптимизировать" такой спек. 

ну если это так, то и источник такого поведения заодно было бы неплохо 
найти и поправить.