Bug 39046

Summary: Empty %_distro_version in p9_mipsel
Product: Sisyphus Reporter: Ivan A. Melnikov <iv>
Component: rpm-build-introAssignee: Vitaly Lipatov <lav>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P5 CC: lav, mike, sin
Version: unstable   
Hardware: mipsel   
OS: Linux   

Description Ivan A. Melnikov 2020-10-06 11:54:51 MSK 
В p9 на mipsel макрос %_distro_version оказывается пустым, что является ещё и синтаксической ошибкой:

error: Macro %_distro_version has empty body
warning: Macro %_distro_version not found
error: parse error in expression
Comment 1 Ivan A. Melnikov 2020-10-06 12:01:49 MSK 
$ cat /etc/altlinux-release
ALT p9_mipsel p9 (Vaccinium)

bin/distr_vendor с таким не знаком. Познакомить, в целом, не сложно:

http://git.altlinux.org/people/iv/packages/?p=rpm-build-altlinux-compat.git;a=commitdiff;h=99bba64f2abcb167c088dde3ce4ea82b0bc8810b

Могу отправить в Сизиф (и потом скопировать по p9) вот такой тег:

http://git.altlinux.org/people/iv/packages/?p=rpm-build-altlinux-compat.git;a=shortlog;h=refs/tags/2.2.1-alt1

lav@, посмотрите пожалуйста.
Comment 2 Vitaly Lipatov 2020-10-06 21:36:57 MSK 
Большое спасибо за подробную информацию. Давайте лучше я отправлю, заодно и кавычки добавлю.
и я собираюсь исправить так:
elif has "ALT p9 "
Не знаю, зачем там два раза p9 может быть.
Comment 3 Repository Robot 2020-10-07 01:32:31 MSK 
rpm-build-altlinux-compat-2.2.1-alt1 -> sisyphus:

 Tue Oct 06 2020 Vitaly Lipatov <lav@altlinux.ru> 2.2.1-alt1
 - fix license to GPLv2+ and add LICENSE file
 - install.sh: forbids empty distro related info
 - update distr_vendor from distro_info from distro_info package (ALT bug 39046)
Comment 4 Ivan A. Melnikov 2020-10-07 08:10:23 MSK 
(In reply to Vitaly Lipatov from comment #2)
> elif has "ALT p9 "

> ALT p9_mipsel p9 (Vaccinium)

Я конечно проверю, но по-моему не сработает: в p9_mipsel после p9 не пробел, а подчёркивание.
Comment 5 Vitaly Lipatov 2020-10-07 09:13:17 MSK 
(Ответ для Ivan A. Melnikov на комментарий #4)
> (In reply to Vitaly Lipatov from comment #2)
> > elif has "ALT p9 "
> 
> > ALT p9_mipsel p9 (Vaccinium)
> 
> Я конечно проверю, но по-моему не сработает: в p9_mipsel после p9 не пробел,
> а подчёркивание.
Ох, я самое главное-то проглядел :(
Comment 6 Vitaly Lipatov 2020-10-07 09:15:08 MSK 
(Ответ для Vitaly Lipatov на комментарий #5)
> > > ALT p9_mipsel p9 (Vaccinium)
> > 
> > Я конечно проверю, но по-моему не сработает: в p9_mipsel после p9 не пробел,
> > а подчёркивание.
Когда будете проверять, приведите сюда вывод
$ distro_info
пожалуйста.
Comment 7 Ivan A. Melnikov 2020-10-07 13:23:08 MSK 
(In reply to Vitaly Lipatov from comment #2)
[...]
> Не знаю, зачем там два раза p9 может быть.

Оно собирается из %distribution +  %distro.

Макрос %distribution переопределяется girar'ом:
%distribution     ALT ${dist/sisyphus/Sisyphus}

т.е. ALT + "настоящее имя" репозитория ($dist это содержимое task/repo). В официальной сборочнице это "настоящее имя" -- p9, но в догоняющих сборках оно  другое -- в mipsel это p9_mipsel, в e2k это p9_e2k и прочее.

%distro же определяется в p9 всегда, если я правильно разобрался, в самом пакете altlinux-release-p9, так как все эти сборочницы собирают, по сути, p9.
Comment 8 Ivan A. Melnikov 2020-10-07 13:25:41 MSK 
2 mike@: для эльбрусовских бранчей наверное тоже актуально.
Comment 9 Vitaly Lipatov 2020-10-07 21:56:30 MSK 
(Ответ для Ivan A. Melnikov на комментарий #1)
> $ cat /etc/altlinux-release
> ALT p9_mipsel p9 (Vaccinium)
> 
> bin/distr_vendor с таким не знаком. Познакомить, в целом, не сложно:
> 
> http://git.altlinux.org/people/iv/packages/?p=rpm-build-altlinux-compat.git;
> a=commitdiff;h=99bba64f2abcb167c088dde3ce4ea82b0bc8810b
> 
> Могу отправить в Сизиф (и потом скопировать по p9) вот такой тег:
В итоге изменил, как было предложено.
Comment 10 Vitaly Lipatov 2020-10-07 21:56:53 MSK 
259346 EPERM #3 p9 rpm-build-altlinux-compat.git=2.2.0-alt1 distro_info.git=1.4-alt1 eepm.git=3.2.6-alt1