Есть какие-нибудь претензии к этому пакету?
On 02.11.2017 20:21, Girar Builder awaiter robot wrote:
> http://git.altlinux.org/tasks/193026/logs/events.1.1.log
>
> 2017-Nov-02 17:15:23 :: task #193026 for sisyphus started by grenka:
> #100 build 0.5.7-alt1 from /people/grenka/packages/vor.git
> 2017-Nov-02 17:15:23 :: [i586] #100 vor.git 0.5.7-alt1: build start
> 2017-Nov-02 17:15:23 :: [x86_64] #100 vor.git 0.5.7-alt1: build start
> 2017-Nov-02 17:17:07 :: [x86_64] #100 vor.git 0.5.7-alt1: build OK
> 2017-Nov-02 17:17:15 :: [i586] #100 vor.git 0.5.7-alt1: build OK
> 2017-Nov-02 17:17:20 :: build check OK
> 2017-Nov-02 17:17:20 :: noarch check OK
> 2017-Nov-02 17:17:21 :: plan: src +1 -1 =17968, i586 +2 -2 =33735, x86_64 +2 -2 =33728
> 2017-Nov-02 17:17:21 :: version check OK
> 2017-Nov-02 17:18:31 :: generated apt indices
> 2017-Nov-02 17:18:31 :: created next repo
> 2017-Nov-02 17:18:51 :: dependencies check OK
> 2017-Nov-02 17:19:20 :: ELF symbols check OK
> 2017-Nov-02 17:19:40 :: [x86_64] #100 vor: install check OK
> 2017-Nov-02 17:19:42 :: [i586] #100 vor: install check OK
> 2017-Nov-02 17:20:01 :: [x86_64] #100 vor-debuginfo: install check OK
> 2017-Nov-02 17:20:04 :: [i586] #100 vor-debuginfo: install check OK
> 2017-Nov-02 17:20:31 :: [x86_64-i586] generated apt indices
> 2017-Nov-02 17:20:31 :: [x86_64-i586] created next repo
> 2017-Nov-02 17:20:58 :: [x86_64-i586] dependencies check OK
> 2017-Nov-02 17:20:59 :: gears inheritance check OK
> 2017-Nov-02 17:20:59 :: srpm inheritance check OK
> girar-check-perms: access to vor DENIED for grenka: does not belong to maintainers list yet
> vor: Operation not permitted
> 2017-Nov-02 17:20:59 :: acl check FAILED
> 2017-Nov-02 17:21:00 :: task #193026 for sisyphus EPERM
[-- Attachment #1: Type: text/plain, Size: 2376 bytes --] On Thu, 2 Nov 2017, Grigory Ustinov wrote: > Есть какие-нибудь претензии к этому пакету? 2viy: На самом деле, если в пакете всё устраивает, то любой член team может пропустить в Sisyphus пакет из задания, сделав approve. Ведь важно только то, полностью ли нормальный пакет. А grenka@ не прошёл ещё до конца процедуру join. На нынешнем этапе он может создавать задания, но их не пропускают без чьего-нибудь одобрения. > On 02.11.2017 20:21, Girar Builder awaiter robot wrote: >> http://git.altlinux.org/tasks/193026/logs/events.1.1.log >> >> 2017-Nov-02 17:15:23 :: task #193026 for sisyphus started by grenka: >> #100 build 0.5.7-alt1 from /people/grenka/packages/vor.git >> 2017-Nov-02 17:15:23 :: [i586] #100 vor.git 0.5.7-alt1: build start >> 2017-Nov-02 17:15:23 :: [x86_64] #100 vor.git 0.5.7-alt1: build start >> 2017-Nov-02 17:17:07 :: [x86_64] #100 vor.git 0.5.7-alt1: build OK >> 2017-Nov-02 17:17:15 :: [i586] #100 vor.git 0.5.7-alt1: build OK >> 2017-Nov-02 17:17:20 :: build check OK >> 2017-Nov-02 17:17:20 :: noarch check OK >> 2017-Nov-02 17:17:21 :: plan: src +1 -1 =17968, i586 +2 -2 =33735, x86_64 >> +2 -2 =33728 >> 2017-Nov-02 17:17:21 :: version check OK >> 2017-Nov-02 17:18:31 :: generated apt indices >> 2017-Nov-02 17:18:31 :: created next repo >> 2017-Nov-02 17:18:51 :: dependencies check OK >> 2017-Nov-02 17:19:20 :: ELF symbols check OK >> 2017-Nov-02 17:19:40 :: [x86_64] #100 vor: install check OK >> 2017-Nov-02 17:19:42 :: [i586] #100 vor: install check OK >> 2017-Nov-02 17:20:01 :: [x86_64] #100 vor-debuginfo: install check OK >> 2017-Nov-02 17:20:04 :: [i586] #100 vor-debuginfo: install check OK >> 2017-Nov-02 17:20:31 :: [x86_64-i586] generated apt indices >> 2017-Nov-02 17:20:31 :: [x86_64-i586] created next repo >> 2017-Nov-02 17:20:58 :: [x86_64-i586] dependencies check OK >> 2017-Nov-02 17:20:59 :: gears inheritance check OK >> 2017-Nov-02 17:20:59 :: srpm inheritance check OK >> girar-check-perms: access to vor DENIED for grenka: does not belong to >> maintainers list yet >> vor: Operation not permitted >> 2017-Nov-02 17:20:59 :: acl check FAILED >> 2017-Nov-02 17:21:00 :: task #193026 for sisyphus EPERM > > _______________________________________________ > devel-newbies mailing list > devel-newbies@lists.altlinux.org > https://lists.altlinux.org/mailman/listinfo/devel-newbies > >
[-- Attachment #1: Type: text/plain, Size: 3245 bytes --] On Thu, 2 Nov 2017, Grigory Ustinov wrote: > Есть какие-нибудь претензии к этому пакету? Конечно, писать Requires: libXXX -- лишнее, потому что автоматически генерятся зависимости более точные -- на библиотеки собственно: $ rpm -qp RPMS.task/vor-0.5.7-alt1.x86_64.rpm --requires libSDL libSDL_mixer libSDL_image /lib64/ld-linux-x86-64.so.2 libSDL-1.2.so.0()(64bit) >= set:lg0OuAA2ogGpZAppCjlYyzPVXuZLduYuHzt0ZFNi46YcvnZpL2yZuPWoaf1 rpmlib(SetVersions) libSDL_image-1.2.so.0()(64bit) >= set:jifK3 libSDL_mixer-1.2.so.0()(64bit) >= set:kgaFOJ3QjXIdaOAMzlryrImk4 libc.so.6(GLIBC_2.14)(64bit) libc.so.6(GLIBC_2.2.5)(64bit) libc.so.6(GLIBC_2.3.4)(64bit) libc.so.6(GLIBC_2.4)(64bit) libm.so.6(GLIBC_2.2.5)(64bit) libpthread.so.0(GLIBC_2.2.5)(64bit) rtld(GNU_HASH) rpmlib(PayloadIsLzma) $ Это небольшой недостаток, потому что пакет, который содержит эти библиотеки, может поменять имя когда-то. А эти вручную записанные зависимости будут мешать такому переименованию. (И начать вытягивать лишние пакеты, если в пакете со старым именем будет лежать бибилиотека новой версии.) Также ещё замечание: в BuildRequires стоит писать только lib*-devel, без дублирования просто lib*. (Бессмысленное дублирование -- их напрямую использовать при сборке не получится без соответствующего *-devel.) > On 02.11.2017 20:21, Girar Builder awaiter robot wrote: >> http://git.altlinux.org/tasks/193026/logs/events.1.1.log >> >> 2017-Nov-02 17:15:23 :: task #193026 for sisyphus started by grenka: >> #100 build 0.5.7-alt1 from /people/grenka/packages/vor.git >> 2017-Nov-02 17:15:23 :: [i586] #100 vor.git 0.5.7-alt1: build start >> 2017-Nov-02 17:15:23 :: [x86_64] #100 vor.git 0.5.7-alt1: build start >> 2017-Nov-02 17:17:07 :: [x86_64] #100 vor.git 0.5.7-alt1: build OK >> 2017-Nov-02 17:17:15 :: [i586] #100 vor.git 0.5.7-alt1: build OK >> 2017-Nov-02 17:17:20 :: build check OK >> 2017-Nov-02 17:17:20 :: noarch check OK >> 2017-Nov-02 17:17:21 :: plan: src +1 -1 =17968, i586 +2 -2 =33735, x86_64 >> +2 -2 =33728 >> 2017-Nov-02 17:17:21 :: version check OK >> 2017-Nov-02 17:18:31 :: generated apt indices >> 2017-Nov-02 17:18:31 :: created next repo >> 2017-Nov-02 17:18:51 :: dependencies check OK >> 2017-Nov-02 17:19:20 :: ELF symbols check OK >> 2017-Nov-02 17:19:40 :: [x86_64] #100 vor: install check OK >> 2017-Nov-02 17:19:42 :: [i586] #100 vor: install check OK >> 2017-Nov-02 17:20:01 :: [x86_64] #100 vor-debuginfo: install check OK >> 2017-Nov-02 17:20:04 :: [i586] #100 vor-debuginfo: install check OK >> 2017-Nov-02 17:20:31 :: [x86_64-i586] generated apt indices >> 2017-Nov-02 17:20:31 :: [x86_64-i586] created next repo >> 2017-Nov-02 17:20:58 :: [x86_64-i586] dependencies check OK >> 2017-Nov-02 17:20:59 :: gears inheritance check OK >> 2017-Nov-02 17:20:59 :: srpm inheritance check OK >> girar-check-perms: access to vor DENIED for grenka: does not belong to >> maintainers list yet >> vor: Operation not permitted >> 2017-Nov-02 17:20:59 :: acl check FAILED >> 2017-Nov-02 17:21:00 :: task #193026 for sisyphus EPERM > > _______________________________________________ > devel-newbies mailing list > devel-newbies@lists.altlinux.org > https://lists.altlinux.org/mailman/listinfo/devel-newbies > >
[-- Attachment #1: Type: text/plain, Size: 3131 bytes --] On Thu, 2 Nov 2017, Ivan Zakharyaschev wrote: > On Thu, 2 Nov 2017, Grigory Ustinov wrote: > >> Есть какие-нибудь претензии к этому пакету? Вообще не жалко (сборочницу) собрать ещё один релиз с учётом этих замечаний. Зато замечания будут на деле зафиксированы коммитом и, может быть, в сознании будущего мейнтейнера. Думаю, было бы нелпохо/полезно для нашего дела (join) сделать ещё один релиз. > Конечно, писать Requires: libXXX -- лишнее, потому что автоматически > генерятся зависимости более точные -- на библиотеки собственно: > Это небольшой недостаток, потому что пакет, который содержит эти библиотеки, > может поменять имя когда-то. А эти вручную записанные зависимости будут > мешать такому переименованию. (И начать вытягивать лишние пакеты, если в > пакете со старым именем будет лежать бибилиотека новой версии.) > > Также ещё замечание: в BuildRequires стоит писать только lib*-devel, без > дублирования просто lib*. (Бессмысленное дублирование -- их напрямую > использовать при сборке не получится без соответствующего *-devel.) > >> On 02.11.2017 20:21, Girar Builder awaiter robot wrote: >> > http://git.altlinux.org/tasks/193026/logs/events.1.1.log >> > >> > 2017-Nov-02 17:15:23 :: task #193026 for sisyphus started by grenka: >> > #100 build 0.5.7-alt1 from /people/grenka/packages/vor.git >> > 2017-Nov-02 17:15:23 :: [i586] #100 vor.git 0.5.7-alt1: build start >> > 2017-Nov-02 17:15:23 :: [x86_64] #100 vor.git 0.5.7-alt1: build start >> > 2017-Nov-02 17:17:07 :: [x86_64] #100 vor.git 0.5.7-alt1: build OK >> > 2017-Nov-02 17:17:15 :: [i586] #100 vor.git 0.5.7-alt1: build OK >> > 2017-Nov-02 17:17:20 :: build check OK >> > 2017-Nov-02 17:17:20 :: noarch check OK >> > 2017-Nov-02 17:17:21 :: plan: src +1 -1 =17968, i586 +2 -2 =33735, >> > x86_64 >> > +2 -2 =33728 >> > 2017-Nov-02 17:17:21 :: version check OK >> > 2017-Nov-02 17:18:31 :: generated apt indices >> > 2017-Nov-02 17:18:31 :: created next repo >> > 2017-Nov-02 17:18:51 :: dependencies check OK >> > 2017-Nov-02 17:19:20 :: ELF symbols check OK >> > 2017-Nov-02 17:19:40 :: [x86_64] #100 vor: install check OK >> > 2017-Nov-02 17:19:42 :: [i586] #100 vor: install check OK >> > 2017-Nov-02 17:20:01 :: [x86_64] #100 vor-debuginfo: install check OK >> > 2017-Nov-02 17:20:04 :: [i586] #100 vor-debuginfo: install check OK >> > 2017-Nov-02 17:20:31 :: [x86_64-i586] generated apt indices >> > 2017-Nov-02 17:20:31 :: [x86_64-i586] created next repo >> > 2017-Nov-02 17:20:58 :: [x86_64-i586] dependencies check OK >> > 2017-Nov-02 17:20:59 :: gears inheritance check OK >> > 2017-Nov-02 17:20:59 :: srpm inheritance check OK >> > girar-check-perms: access to vor DENIED for grenka: does not belong to >> > maintainers list yet >> > vor: Operation not permitted >> > 2017-Nov-02 17:20:59 :: acl check FAILED >> > 2017-Nov-02 17:21:00 :: task #193026 for sisyphus EPERM >> >> _______________________________________________ >> devel-newbies mailing list >> devel-newbies@lists.altlinux.org >> https://lists.altlinux.org/mailman/listinfo/devel-newbies >> >> >
2017-11-02 20:57 GMT+02:00 Ivan Zakharyaschev: >> Есть какие-нибудь претензии к этому пакету? > > Конечно, писать Requires: libXXX -- лишнее, потому что автоматически > генерятся зависимости более точные -- на библиотеки собственно: > > Это небольшой недостаток, потому что пакет, который содержит эти библиотеки, > может поменять имя когда-то. А эти вручную записанные зависимости будут > мешать такому переименованию. (И начать вытягивать лишние пакеты, если в > пакете со старым именем будет лежать бибилиотека новой версии.) > > Также ещё замечание: в BuildRequires стоит писать только lib*-devel, без > дублирования просто lib*. (Бессмысленное дублирование -- их напрямую > использовать при сборке не получится без соответствующего *-devel.) BuildRequires лучше вообще руками не писать без надобности. Проще натравить buildreq на спек без каких либо BuildRequires и получить список того что требуется для сборки. И делать так при каждой сборке. -- Igor Zubkov http://hi.im/ice
[-- Attachment #1: Type: text/plain, Size: 725 bytes --] On Thu, 2 Nov 2017, Igor Zubkov wrote: > 2017-11-02 20:57 GMT+02:00 Ivan Zakharyaschev: >> Также ещё замечание: в BuildRequires стоит писать только lib*-devel, без >> дублирования просто lib*. (Бессмысленное дублирование -- их напрямую >> использовать при сборке не получится без соответствующего *-devel.) > > BuildRequires лучше вообще руками не писать без надобности. Проще > натравить buildreq на спек без каких либо BuildRequires и получить > список того что требуется для сборки. И делать так при каждой сборке. Всё же если делать это в hasher-е, то сначала нужно (точнее проще всего) написать избыточный список BuildRequires, а потом buildreq перепишет его на "оптимальный", убрав лишнее. -- Best regards, Ivan
2017-11-02 23:31 GMT+02:00 Ivan Zakharyaschev: >>> Также ещё замечание: в BuildRequires стоит писать только lib*-devel, без >>> дублирования просто lib*. (Бессмысленное дублирование -- их напрямую >>> использовать при сборке не получится без соответствующего *-devel.) >> >> BuildRequires лучше вообще руками не писать без надобности. Проще >> натравить buildreq на спек без каких либо BuildRequires и получить >> список того что требуется для сборки. И делать так при каждой сборке. > > Всё же если делать это в hasher-е, то сначала нужно (точнее проще всего) > написать избыточный список BuildRequires, а потом buildreq перепишет его на > "оптимальный", убрав лишнее. Ну так, это и надо делать в hasher. А если собирать пакеты которых даже нет в репозитории, то тут получается самое оно. Ты ставишь в hasher зависимости, собираешь и запускаешь buildreq. Из основной системы тебе не залетает в зависимости всякий мусор. buildreq умеет фильтровать лишнии зависимости, но не все пакеты правильно настраивают buildreq что бы он фильтровал лишнее. Ну и ещё есть случаи когда в BuildRequires залетают всякие glibc-devel-static. Зачем, не понятно. Вот только тут лучше всего сделать какой-то repocop тест что бы проверять, не потерялся ли функционал после сборки. Можно это сделать путём сравнения зависимостей и файлов. -- Igor Zubkov http://hi.im/ice