* 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