devel@ where you _can_ ask
 help / color / mirror / Atom feed
* Re: [newbies] [#195123] EPERM ltrace.git=0.7.91.0.198.git82c6640-alt1
  @ 2017-11-22 10:02 ` Grigory Ustinov
  2017-11-24  9:29   ` Ivan Zakharyaschev
  0 siblings, 1 reply; 4+ messages in thread
From: Grigory Ustinov @ 2017-11-22 10:02 UTC (permalink / raw)
  To: devel-newbies

Всё пропатчил.


On 22.11.2017 12:41, Girar Builder awaiter robot wrote:
> http://git.altlinux.org/tasks/195123/logs/events.1.1.log
>
> 2017-Nov-22 09:32:48 :: task #195123 for sisyphus started by grenka:
> #100 build 0.7.91.0.198.git82c6640-alt1 from /people/grenka/packages/ltrace.git
> 2017-Nov-22 09:32:48 :: [x86_64] #100 ltrace.git 0.7.91.0.198.git82c6640-alt1: build start
> 2017-Nov-22 09:32:48 :: [i586] #100 ltrace.git 0.7.91.0.198.git82c6640-alt1: build start
> 2017-Nov-22 09:36:18 :: [x86_64] #100 ltrace.git 0.7.91.0.198.git82c6640-alt1: build OK
> 2017-Nov-22 09:36:27 :: [i586] #100 ltrace.git 0.7.91.0.198.git82c6640-alt1: build OK
> 2017-Nov-22 09:36:41 :: build check OK
> 2017-Nov-22 09:36:41 :: noarch check OK
> 2017-Nov-22 09:36:42 :: plan: src +1 -1 =17882, i586 +2 -2 =33457, x86_64 +2 -2 =33450
> 2017-Nov-22 09:36:42 :: version check OK
> 2017-Nov-22 09:36:43 :: ltrace: closes bugs: 33470
> 2017-Nov-22 09:38:55 :: generated apt indices
> 2017-Nov-22 09:38:56 :: created next repo
> 2017-Nov-22 09:39:25 :: dependencies check OK
> 2017-Nov-22 09:40:32 :: ELF symbols check OK
> 2017-Nov-22 09:41:01 :: [x86_64] #100 ltrace: install check OK
> 2017-Nov-22 09:41:02 :: [i586] #100 ltrace: install check OK
> 2017-Nov-22 09:41:28 :: [x86_64] #100 ltrace-debuginfo: install check OK
> 2017-Nov-22 09:41:29 :: [i586] #100 ltrace-debuginfo: install check OK
> 2017-Nov-22 09:41:47 :: [x86_64-i586] generated apt indices
> 2017-Nov-22 09:41:47 :: [x86_64-i586] created next repo
> 2017-Nov-22 09:41:56 :: [x86_64-i586] dependencies check OK
> 2017-Nov-22 09:41:56 :: gears inheritance check OK
> 2017-Nov-22 09:41:56 :: srpm inheritance check OK
> girar-check-perms: access to ltrace DENIED for grenka: does not belong to maintainers list yet
> ltrace: Operation not permitted
> 2017-Nov-22 09:41:56 :: acl check FAILED
> 2017-Nov-22 09:41:56 :: task #195123 for sisyphus EPERM



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [newbies] [#195123] EPERM ltrace.git=0.7.91.0.198.git82c6640-alt1
  2017-11-22 10:02 ` [newbies] [#195123] EPERM ltrace.git=0.7.91.0.198.git82c6640-alt1 Grigory Ustinov
@ 2017-11-24  9:29   ` Ivan Zakharyaschev
  2017-11-24 13:52     ` Ivan Zakharyaschev
  0 siblings, 1 reply; 4+ messages in thread
From: Ivan Zakharyaschev @ 2017-11-24  9:29 UTC (permalink / raw)
  To: devel@ where you _can_ ask; +Cc: grenka

[-- Attachment #1: Type: text/plain, Size: 3693 bytes --]

Hello!

On Wed, 22 Nov 2017, Grigory Ustinov wrote:

> Всё пропатчил.

Интересное разбирательство. Прокомментировал патчи, отправленные в 
upstream:

1. 
http://lists.alioth.debian.org/pipermail/ltrace-devel/2017-November/001390.html
2. 
http://lists.alioth.debian.org/pipermail/ltrace-devel/2017-November/001392.html
3. 
http://lists.alioth.debian.org/pipermail/ltrace-devel/2017-November/001393.html

С точки зрения Sisyphus по последним двум пунктам существенных замечаний 
нет, налицо исправления того, что было в upstream (и возможна дальнейшая 
работа по ещё лучшему оформлению этих проблем в upstream). Если коротко с 
точки зрения Sisyphus:

2. Нет поддержки long double, ну и тест не нужно запускать. (Когда 
поддержка появится, можно патч убрать.)

3. Исправлены тесты так, чтобы они давали разумный результат, выяснены 
причины которые на это влияют, коротко пояснены. Уже существенная ценность 
для пакета.

Что касается пункта 1 (тесты, которые опираются на наличие 
glibc-core-debuginfo): хорошо бы оставить лёгкий способ для человека, 
взявшего srpm, их всё же прогнать при своей локальной сборке, когда он 
может обеспечить наличие glibc-core-debuginfo. Кажется, удобно будет их 
оформить ручкой:

%def_without glibc_debuginfo

чем их вообще терять.

(enable/disable -- насколкьо я понимаю -- используют для фич самого 
проекта, а а with/without -- для зависимостей от всяких "сторонних" штук, 
иногда с выбором использовать внешнюю системную библиотеку или которую 
носим с собой в проекте. Поправьте, если я не так понимаю.)


> On 22.11.2017 12:41, Girar Builder awaiter robot wrote:
>>  http://git.altlinux.org/tasks/195123/logs/events.1.1.log
>>
>>  2017-Nov-22 09:32:48 :: task #195123 for sisyphus started by grenka:
>>  #100 build 0.7.91.0.198.git82c6640-alt1 from
>>  /people/grenka/packages/ltrace.git
>>  2017-Nov-22 09:32:48 :: [x86_64] #100 ltrace.git
>>  0.7.91.0.198.git82c6640-alt1: build start
>>  2017-Nov-22 09:32:48 :: [i586] #100 ltrace.git
>>  0.7.91.0.198.git82c6640-alt1: build start
>>  2017-Nov-22 09:36:18 :: [x86_64] #100 ltrace.git
>>  0.7.91.0.198.git82c6640-alt1: build OK
>>  2017-Nov-22 09:36:27 :: [i586] #100 ltrace.git
>>  0.7.91.0.198.git82c6640-alt1: build OK
>>  2017-Nov-22 09:36:41 :: build check OK
>>  2017-Nov-22 09:36:41 :: noarch check OK
>>  2017-Nov-22 09:36:42 :: plan: src +1 -1 =17882, i586 +2 -2 =33457, x86_64
>>  +2 -2 =33450
>>  2017-Nov-22 09:36:42 :: version check OK
>>  2017-Nov-22 09:36:43 :: ltrace: closes bugs: 33470
>>  2017-Nov-22 09:38:55 :: generated apt indices
>>  2017-Nov-22 09:38:56 :: created next repo
>>  2017-Nov-22 09:39:25 :: dependencies check OK
>>  2017-Nov-22 09:40:32 :: ELF symbols check OK
>>  2017-Nov-22 09:41:01 :: [x86_64] #100 ltrace: install check OK
>>  2017-Nov-22 09:41:02 :: [i586] #100 ltrace: install check OK
>>  2017-Nov-22 09:41:28 :: [x86_64] #100 ltrace-debuginfo: install check OK
>>  2017-Nov-22 09:41:29 :: [i586] #100 ltrace-debuginfo: install check OK
>>  2017-Nov-22 09:41:47 :: [x86_64-i586] generated apt indices
>>  2017-Nov-22 09:41:47 :: [x86_64-i586] created next repo
>>  2017-Nov-22 09:41:56 :: [x86_64-i586] dependencies check OK
>>  2017-Nov-22 09:41:56 :: gears inheritance check OK
>>  2017-Nov-22 09:41:56 :: srpm inheritance check OK
>>  girar-check-perms: access to ltrace DENIED for grenka: does not belong to
>>  maintainers list yet
>>  ltrace: Operation not permitted
>>  2017-Nov-22 09:41:56 :: acl check FAILED
>>  2017-Nov-22 09:41:56 :: task #195123 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] 4+ messages in thread

* Re: [newbies] [#195123] EPERM ltrace.git=0.7.91.0.198.git82c6640-alt1
  2017-11-24  9:29   ` Ivan Zakharyaschev
@ 2017-11-24 13:52     ` Ivan Zakharyaschev
  2017-11-24 21:38       ` Ivan Zakharyaschev
  0 siblings, 1 reply; 4+ messages in thread
From: Ivan Zakharyaschev @ 2017-11-24 13:52 UTC (permalink / raw)
  To: devel@ where you _can_ ask; +Cc: grenka

[-- Attachment #1: Type: text/plain, Size: 1286 bytes --]

Попробовал на своей машине, оно не очень приятно падает в этом режиме:

$ ltrace  -x \* /bin/ls /var/empty/
....
<... check_match resumed> ) 
= { 18799, '\022', '\0', 13, 221008, 220 }
<... do_lookup_x resumed> ) 
= 1
<... _dl_lookup_symbol_x resumed> ) 
= <void>
<... _dl_fixup resumed> ) 
= 140287210082128
getenv@libc.so.6("COLUMNS" <unfinished ...>
strlen@libc.so.6("COLUMNS") 
= 7
<... getenv resumed> ) 
= nil
ioctl( <unfinished ...>
--- SIGSEGV (Segmentation fault) ---
+++ killed by SIGSEGV +++

$ ltrace  -x \* /bin/echo a
....
<... check_match resumed> ) 
= { 20645, '"', '\0', 13, 471424, 151 }
<... do_lookup_x resumed> ) 
= 1
<... _dl_lookup_symbol_x resumed> ) 
= <void>
<... _dl_fixup resumed> ) 
= 140342807331200
fputs_unlocked@libc.so.6(0x7fffa221149b, 0x7fa400007fa4, 45, 32 
<unfinished ...>
strlen@libc.so.6("a") 
= 1
--- SIGSEGV (Segmentation fault) ---
+++ killed by SIGSEGV +++

$ rpm -q ltrace coreutils-debuginfo glibc-core-debuginfo
ltrace-0.7.91.0.198.git82c6640-alt1.x86_64
coreutils-debuginfo-8.27.0.23.f4570-alt1.x86_64
glibc-core-debuginfo-2.25-alt3.x86_64
$

Думаю, если это бага upstream-а, то это не препятствие для попадания 
пакета в Sisyphus. Можно будет в bugzill-е зафиксировать её просто.

Без debuginfo то же поведение.

-- 
Best regards,
Ivan

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [newbies] [#195123] EPERM ltrace.git=0.7.91.0.198.git82c6640-alt1
  2017-11-24 13:52     ` Ivan Zakharyaschev
@ 2017-11-24 21:38       ` Ivan Zakharyaschev
  0 siblings, 0 replies; 4+ messages in thread
From: Ivan Zakharyaschev @ 2017-11-24 21:38 UTC (permalink / raw)
  To: devel@ where you _can_ ask; +Cc: grenka

[-- Attachment #1: Type: text/plain, Size: 925 bytes --]

On Fri, 24 Nov 2017, Ivan Zakharyaschev wrote:

> Попробовал на своей машине, оно не очень приятно падает в этом режиме:
>
> $ ltrace  -x \* /bin/ls /var/empty/
> ....
> <... check_match resumed> ) = { 18799, '\022', '\0', 13, 221008, 220 }
> <... do_lookup_x resumed> ) = 1
> <... _dl_lookup_symbol_x resumed> ) = <void>
> <... _dl_fixup resumed> ) = 140287210082128
> getenv@libc.so.6("COLUMNS" <unfinished ...>
> strlen@libc.so.6("COLUMNS") = 7
> <... getenv resumed> ) = nil
> ioctl( <unfinished ...>
> --- SIGSEGV (Segmentation fault) ---
> +++ killed by SIGSEGV +++

> Думаю, если это бага upstream-а, то это не препятствие для попадания пакета в 
> Sisyphus. Можно будет в bugzill-е зафиксировать её просто.

https://bugzilla.altlinux.org/show_bug.cgi?id=34234

Можно поупражняться в git bisect (ведь прошлый релиз не падал при таком 
использовании), если кому интересно будет отловить багу. :)

-- 
Best regards,
Ivan

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-11-24 21:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-22 10:02 ` [newbies] [#195123] EPERM ltrace.git=0.7.91.0.198.git82c6640-alt1 Grigory Ustinov
2017-11-24  9:29   ` Ivan Zakharyaschev
2017-11-24 13:52     ` Ivan Zakharyaschev
2017-11-24 21:38       ` Ivan Zakharyaschev

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