* Re: [newbies] [#193026] EPERM vor.git=0.5.7-alt1 @ 2017-11-02 17:38 ` Grigory Ustinov 2017-11-02 18:09 ` Ivan Zakharyaschev 2017-11-02 18:57 ` Ivan Zakharyaschev 0 siblings, 2 replies; 7+ messages in thread From: Grigory Ustinov @ 2017-11-02 17:38 UTC (permalink / raw) To: devel-newbies Есть какие-нибудь претензии к этому пакету? 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [newbies] [#193026] EPERM vor.git=0.5.7-alt1 2017-11-02 17:38 ` [newbies] [#193026] EPERM vor.git=0.5.7-alt1 Grigory Ustinov @ 2017-11-02 18:09 ` Ivan Zakharyaschev 2017-11-02 18:57 ` Ivan Zakharyaschev 1 sibling, 0 replies; 7+ messages in thread From: Ivan Zakharyaschev @ 2017-11-02 18:09 UTC (permalink / raw) To: devel@ where you _can_ ask; +Cc: grenka, viy [-- 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 > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [newbies] [#193026] EPERM vor.git=0.5.7-alt1 2017-11-02 17:38 ` [newbies] [#193026] EPERM vor.git=0.5.7-alt1 Grigory Ustinov 2017-11-02 18:09 ` Ivan Zakharyaschev @ 2017-11-02 18:57 ` Ivan Zakharyaschev 2017-11-02 19:02 ` Ivan Zakharyaschev 2017-11-02 20:51 ` Igor Zubkov 1 sibling, 2 replies; 7+ messages in thread From: Ivan Zakharyaschev @ 2017-11-02 18:57 UTC (permalink / raw) To: Grigory Ustinov; +Cc: 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 > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [newbies] [#193026] EPERM vor.git=0.5.7-alt1 2017-11-02 18:57 ` Ivan Zakharyaschev @ 2017-11-02 19:02 ` Ivan Zakharyaschev 2017-11-02 20:51 ` Igor Zubkov 1 sibling, 0 replies; 7+ messages in thread From: Ivan Zakharyaschev @ 2017-11-02 19:02 UTC (permalink / raw) To: devel@ where you _can_ ask; +Cc: Grigory Ustinov [-- 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 >> >> > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [newbies] [#193026] EPERM vor.git=0.5.7-alt1 2017-11-02 18:57 ` Ivan Zakharyaschev 2017-11-02 19:02 ` Ivan Zakharyaschev @ 2017-11-02 20:51 ` Igor Zubkov 2017-11-02 21:31 ` Ivan Zakharyaschev 1 sibling, 1 reply; 7+ messages in thread From: Igor Zubkov @ 2017-11-02 20:51 UTC (permalink / raw) To: devel@ where you _can_ ask 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [newbies] [#193026] EPERM vor.git=0.5.7-alt1 2017-11-02 20:51 ` Igor Zubkov @ 2017-11-02 21:31 ` Ivan Zakharyaschev 2017-11-03 8:58 ` Igor Zubkov 0 siblings, 1 reply; 7+ messages in thread From: Ivan Zakharyaschev @ 2017-11-02 21:31 UTC (permalink / raw) To: devel@ where you _can_ ask [-- 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [newbies] [#193026] EPERM vor.git=0.5.7-alt1 2017-11-02 21:31 ` Ivan Zakharyaschev @ 2017-11-03 8:58 ` Igor Zubkov 0 siblings, 0 replies; 7+ messages in thread From: Igor Zubkov @ 2017-11-03 8:58 UTC (permalink / raw) To: devel@ where you _can_ ask 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-11-03 8:58 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-11-02 17:38 ` [newbies] [#193026] EPERM vor.git=0.5.7-alt1 Grigory Ustinov 2017-11-02 18:09 ` Ivan Zakharyaschev 2017-11-02 18:57 ` Ivan Zakharyaschev 2017-11-02 19:02 ` Ivan Zakharyaschev 2017-11-02 20:51 ` Igor Zubkov 2017-11-02 21:31 ` Ivan Zakharyaschev 2017-11-03 8:58 ` Igor Zubkov
devel@ where you _can_ ask This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel-newbies/0 devel-newbies/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 devel-newbies devel-newbies/ http://lore.altlinux.org/devel-newbies \ devel-newbies@lists.altlinux.org devel-newbies@lists.altlinux.ru devel-newbies@lists.altlinux.com public-inbox-index devel-newbies Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel-newbies AGPL code for this site: git clone https://public-inbox.org/public-inbox.git