devel@ where you _can_ ask
 help / color / mirror / Atom feed
* 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