[devel] Сборочница не проверяет идентичность SRPM?

[devel] Сборочница не проверяет идентичность SRPM?

[Денис Назаров]

[-- Attachment #1.1: Type: text/plain, Size: 1005 bytes --]

Привет.

При обновлении OpenColorIO допустил ошибку и включил патч для i586 (в 
перечисление самих патчей, а не в секции setup) только для этой 
архитектуры, в следствие чего данный патч попал только в SRPM для i586, 
для остальных архитектур получился SRPM без этого патча. И при сборке 
пакета всё прошло удачно.

В репозиторий же как понимаю попадает SRPM из x86_64 и пересборка не 
удаётся - 
https://git.altlinux.org/beehive/logs/Sisyphus/i586/archive/2023/1125/error/opencolorio-2.3.0-alt2

Может стоит проверять дополнительно, что бы пакет с исходниками 
получился идентичным на всех собранных архитектурах?


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

Re: [devel] Сборочница не проверяет идентичность SRPM?

[Ilya Kurdyukov]

Просто не надо так делать:

%ifarch xxx
Patch1: xxx.patch
%endif

Перечисление патчей должно быть доступно для всех архитектур.

Если надо отключать патч, то делать это в %setup.


On 11/25/23 17:06, Денис Назаров wrote:
> Привет.
>
> При обновлении OpenColorIO допустил ошибку и включил патч для i586 (в 
> перечисление самих патчей, а не в секции setup) только для этой 
> архитектуры, в следствие чего данный патч попал только в SRPM для 
> i586, для остальных архитектур получился SRPM без этого патча. И при 
> сборке пакета всё прошло удачно.
>
> В репозиторий же как понимаю попадает SRPM из x86_64 и пересборка не 
> удаётся - 
> https://git.altlinux.org/beehive/logs/Sisyphus/i586/archive/2023/1125/error/opencolorio-2.3.0-alt2
>
> Может стоит проверять дополнительно, что бы пакет с исходниками 
> получился идентичным на всех собранных архитектурах?
>
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel

Re: [devel] Сборочница не проверяет идентичность SRPM?

[Денис Назаров]

[-- Attachment #1.1: Type: text/plain, Size: 2136 bytes --]

25.11.2023 15:03, Ilya Kurdyukov пишет:
> Просто не надо так делать:
>
> %ifarch xxx
> Patch1: xxx.patch
> %endif
>
> Перечисление патчей должно быть доступно для всех архитектур.
>
> Если надо отключать патч, то делать это в %setup.


Это как раз понятно - не заметил и по ошибке в 2 местах написал. Сейчас 
просто получается ситуация, что можно собрать пакет, который успешно 
проходит сборку, но не может пройти пересборку.


>
>
> On 11/25/23 17:06, Денис Назаров wrote:
>> Привет.
>>
>> При обновлении OpenColorIO допустил ошибку и включил патч для i586 (в 
>> перечисление самих патчей, а не в секции setup) только для этой 
>> архитектуры, в следствие чего данный патч попал только в SRPM для 
>> i586, для остальных архитектур получился SRPM без этого патча. И при 
>> сборке пакета всё прошло удачно.
>>
>> В репозиторий же как понимаю попадает SRPM из x86_64 и пересборка не 
>> удаётся - 
>> https://git.altlinux.org/beehive/logs/Sisyphus/i586/archive/2023/1125/error/opencolorio-2.3.0-alt2
>>
>> Может стоит проверять дополнительно, что бы пакет с исходниками 
>> получился идентичным на всех собранных архитектурах?
>>
>>
>> _______________________________________________
>> Devel mailing list
>> Devel@lists.altlinux.org
>> https://lists.altlinux.org/mailman/listinfo/devel
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel



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

Re: [devel] Сборочница не проверяет идентичность SRPM?

[Anton Farygin]

On 25.11.2023 15:18, Денис Назаров wrote:
> 25.11.2023 15:03, Ilya Kurdyukov пишет:
>> Просто не надо так делать:
>>
>> %ifarch xxx
>> Patch1: xxx.patch
>> %endif
>>
>> Перечисление патчей должно быть доступно для всех архитектур.
>>
>> Если надо отключать патч, то делать это в %setup.
>
>
> Это как раз понятно - не заметил и по ошибке в 2 местах написал. 
> Сейчас просто получается ситуация, что можно собрать пакет, который 
> успешно проходит сборку, но не может пройти пересборку.

Способов сделать такой пакет гораздо больше одного.

Вообще тут, как мне кажется, лучше повесить FR на girar о том, что бы он 
собирал для всех архитектур бинари из srpm, сделанного на какой-то одной 
архитектуре.

Re: [devel] Сборочница не проверяет идентичность SRPM?

[Денис Назаров]

[-- Attachment #1.1: Type: text/plain, Size: 1432 bytes --]

26.11.2023 12:29, Anton Farygin пишет:
> On 25.11.2023 15:18, Денис Назаров wrote:
>> 25.11.2023 15:03, Ilya Kurdyukov пишет:
>>> Просто не надо так делать:
>>>
>>> %ifarch xxx
>>> Patch1: xxx.patch
>>> %endif
>>>
>>> Перечисление патчей должно быть доступно для всех архитектур.
>>>
>>> Если надо отключать патч, то делать это в %setup.
>>
>>
>> Это как раз понятно - не заметил и по ошибке в 2 местах написал. 
>> Сейчас просто получается ситуация, что можно собрать пакет, который 
>> успешно проходит сборку, но не может пройти пересборку.
>
> Способов сделать такой пакет гораздо больше одного.
>
> Вообще тут, как мне кажется, лучше повесить FR на girar о том, что бы 
> он собирал для всех архитектур бинари из srpm, сделанного на какой-то 
> одной архитектуре.
>
https://bugzilla.altlinux.org/48592
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel



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

Re: [devel] Сборочница не проверяет идентичность SRPM?

[Ilya Kurdyukov]

On 11/26/23 20:27, Денис Назаров wrote:
> 26.11.2023 12:29, Anton Farygin пишет:
>> On 25.11.2023 15:18, Денис Назаров wrote:
>>> 25.11.2023 15:03, Ilya Kurdyukov пишет:
>>>> Просто не надо так делать:
>>>>
>>>> %ifarch xxx
>>>> Patch1: xxx.patch
>>>> %endif
>>>>
>>>> Перечисление патчей должно быть доступно для всех архитектур.
>>>>
>>>> Если надо отключать патч, то делать это в %setup.
>>>
>>>
>>> Это как раз понятно - не заметил и по ошибке в 2 местах написал. 
>>> Сейчас просто получается ситуация, что можно собрать пакет, который 
>>> успешно проходит сборку, но не может пройти пересборку.
>>
>> Способов сделать такой пакет гораздо больше одного.
>>
>> Вообще тут, как мне кажется, лучше повесить FR на girar о том, что бы 
>> он собирал для всех архитектур бинари из srpm, сделанного на какой-то 
>> одной архитектуре.
>>
> https://bugzilla.altlinux.org/48592

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

 > для поддерживаемых в задании архитектур

А изменятся поддерживаемые - изменится SRPM.

до: грабли

после: грабли прикрытые листьями

Re: [devel] Сборочница не проверяет идентичность SRPM?

[Anton Farygin]

On 26.11.2023 16:38, Ilya Kurdyukov wrote:
>
> On 11/26/23 20:27, Денис Назаров wrote:
>> 26.11.2023 12:29, Anton Farygin пишет:
>>> On 25.11.2023 15:18, Денис Назаров wrote:
>>>> 25.11.2023 15:03, Ilya Kurdyukov пишет:
>>>>> Просто не надо так делать:
>>>>>
>>>>> %ifarch xxx
>>>>> Patch1: xxx.patch
>>>>> %endif
>>>>>
>>>>> Перечисление патчей должно быть доступно для всех архитектур.
>>>>>
>>>>> Если надо отключать патч, то делать это в %setup.
>>>>
>>>>
>>>> Это как раз понятно - не заметил и по ошибке в 2 местах написал. 
>>>> Сейчас просто получается ситуация, что можно собрать пакет, который 
>>>> успешно проходит сборку, но не может пройти пересборку.
>>>
>>> Способов сделать такой пакет гораздо больше одного.
>>>
>>> Вообще тут, как мне кажется, лучше повесить FR на girar о том, что 
>>> бы он собирал для всех архитектур бинари из srpm, сделанного на 
>>> какой-то одной архитектуре.
>>>
>> https://bugzilla.altlinux.org/48592
>
> Только создаст ситуацию что кто-то сделает что-то в спеке для своей 
> архитектуры, и не сможет понять почему SRPM получается для другой. 
> Такое решение закладывает грабли на которые обязательно кто-нибудь 
> наступит. И когда наступит - ему будет сложно понять что происходит.
>
> > для поддерживаемых в задании архитектур
>
> А изменятся поддерживаемые - изменится SRPM.
>
> до: грабли
>
> после: грабли прикрытые листьями

На самом деле никто не знает, но реально у нас есть архитектуры в src.rpm.

И они бывают странные :)

лучше это привести в порядок.

Re: [devel] Сборочница не проверяет идентичность SRPM?

[Денис Назаров]

[-- Attachment #1.1: Type: text/plain, Size: 2581 bytes --]

26.11.2023 16:38, Ilya Kurdyukov пишет:
>
> On 11/26/23 20:27, Денис Назаров wrote:
>> 26.11.2023 12:29, Anton Farygin пишет:
>>> On 25.11.2023 15:18, Денис Назаров wrote:
>>>> 25.11.2023 15:03, Ilya Kurdyukov пишет:
>>>>> Просто не надо так делать:
>>>>>
>>>>> %ifarch xxx
>>>>> Patch1: xxx.patch
>>>>> %endif
>>>>>
>>>>> Перечисление патчей должно быть доступно для всех архитектур.
>>>>>
>>>>> Если надо отключать патч, то делать это в %setup.
>>>>
>>>>
>>>> Это как раз понятно - не заметил и по ошибке в 2 местах написал. 
>>>> Сейчас просто получается ситуация, что можно собрать пакет, который 
>>>> успешно проходит сборку, но не может пройти пересборку.
>>>
>>> Способов сделать такой пакет гораздо больше одного.
>>>
>>> Вообще тут, как мне кажется, лучше повесить FR на girar о том, что 
>>> бы он собирал для всех архитектур бинари из srpm, сделанного на 
>>> какой-то одной архитектуре.
>>>
>> https://bugzilla.altlinux.org/48592
>
> Только создаст ситуацию что кто-то сделает что-то в спеке для своей 
> архитектуры, и не сможет понять почему SRPM получается для другой. 
> Такое решение закладывает грабли на которые обязательно кто-нибудь 
> наступит. И когда наступит - ему будет сложно понять что происходит.
>
Тогда и в репозиторий надо класть SRPM отдельно для каждой архитектуры и 
пересобирать на каждой архитектуре из своего SRPM )

> > для поддерживаемых в задании архитектур
>
> А изменятся поддерживаемые - изменится SRPM.
>
> до: грабли
>
> после: грабли прикрытые листьями
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel



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

Re: [devel] Сборочница не проверяет идентичность SRPM?

[Danil Shein]

Вот все варианты "архитектуры" исходных пакетов за последние 10 лет:

SELECT groupUniqArray(pkg_arch)
FROM Packages
WHERE pkg_sourcepackage = 1

['pentium4','e2kv6','loongarch64','e2kv5','armv8l','armh','x86_64-i586','athlon','mipsel','i586','ppc64le','noarch','riscv64','x86_64','k6','e2kv4','i686','armv7l','aarch64','e2k','pentium3'] 


Ровно по этой причине в рамках p.a.o для исходных пакетов мы используем 
виртуальную архитектуру 'srpm'.

26.11.2023 16:45, Anton Farygin пишет:
> On 26.11.2023 16:38, Ilya Kurdyukov wrote:
>>
>> On 11/26/23 20:27, Денис Назаров wrote:
>>> 26.11.2023 12:29, Anton Farygin пишет:
>>>> On 25.11.2023 15:18, Денис Назаров wrote:
>>>>> 25.11.2023 15:03, Ilya Kurdyukov пишет:
>>>>>> Просто не надо так делать:
>>>>>>
>>>>>> %ifarch xxx
>>>>>> Patch1: xxx.patch
>>>>>> %endif
>>>>>>
>>>>>> Перечисление патчей должно быть доступно для всех архитектур.
>>>>>>
>>>>>> Если надо отключать патч, то делать это в %setup.
>>>>>
>>>>>
>>>>> Это как раз понятно - не заметил и по ошибке в 2 местах написал. 
>>>>> Сейчас просто получается ситуация, что можно собрать пакет, 
>>>>> который успешно проходит сборку, но не может пройти пересборку.
>>>>
>>>> Способов сделать такой пакет гораздо больше одного.
>>>>
>>>> Вообще тут, как мне кажется, лучше повесить FR на girar о том, что 
>>>> бы он собирал для всех архитектур бинари из srpm, сделанного на 
>>>> какой-то одной архитектуре.
>>>>
>>> https://bugzilla.altlinux.org/48592
>>
>> Только создаст ситуацию что кто-то сделает что-то в спеке для своей 
>> архитектуры, и не сможет понять почему SRPM получается для другой. 
>> Такое решение закладывает грабли на которые обязательно кто-нибудь 
>> наступит. И когда наступит - ему будет сложно понять что происходит.
>>
>> > для поддерживаемых в задании архитектур
>>
>> А изменятся поддерживаемые - изменится SRPM.
>>
>> до: грабли
>>
>> после: грабли прикрытые листьями
>
> На самом деле никто не знает, но реально у нас есть архитектуры в 
> src.rpm.
>
> И они бывают странные :)
>
> лучше это привести в порядок.
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel
-- 

*Данил Шеин / Danil Shein*

dshein@altlinux.org
dshein@basealt.ru