[devel] glibc 6:2.5-alt2 to git-push

[devel] glibc 6:2.5-alt2 to git-push

[Alexey Tourbin]

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

On Tue, Oct 10, 2006 at 03:06:59AM +0400, QA Team Robot wrote:
> glibc - The GNU libc libraries                                  	[20M]
> * Sun Oct 08 2006 Dmitry V. Levin <ldv@altlinux> 6:2.5-alt2
> - Updated to glibc-2_5-branch snapshot 20061006.
> - Build with -U_FORTIFY_SOURCE and -fno-stack-protector.
> - init.d/nscd: Removed obsolete nscd_nischeck.
> * Fri Sep 29 2006 Dmitry V. Levin <ldv@altlinux> 6:2.5-alt1

git-push бы сделать

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] glibc 6:2.5-alt2 to git-push

[Alexey Tourbin]

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

On Fri, Oct 13, 2006 at 07:07:15PM +0400, Alexey Tourbin wrote:
> On Tue, Oct 10, 2006 at 03:06:59AM +0400, QA Team Robot wrote:
> > glibc - The GNU libc libraries                                  	[20M]
> > * Sun Oct 08 2006 Dmitry V. Levin <ldv@altlinux> 6:2.5-alt2
> > - Updated to glibc-2_5-branch snapshot 20061006.
> > - Build with -U_FORTIFY_SOURCE and -fno-stack-protector.
> > - init.d/nscd: Removed obsolete nscd_nischeck.
> > * Fri Sep 29 2006 Dmitry V. Levin <ldv@altlinux> 6:2.5-alt1
> 
> git-push бы сделать

Спасибо!  Toolchain готов, как я понимаю?  Выжидаю момент, чтобы потом
избежать автоматической пересборки своих пакетов.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] glibc 6:2.5-alt2 to git-push

[Dmitry V. Levin]

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

On Fri, Oct 13, 2006 at 09:19:41PM +0400, Alexey Tourbin wrote:
> On Fri, Oct 13, 2006 at 07:07:15PM +0400, Alexey Tourbin wrote:
> > On Tue, Oct 10, 2006 at 03:06:59AM +0400, QA Team Robot wrote:
> > > glibc - The GNU libc libraries                                  	[20M]
> > > * Sun Oct 08 2006 Dmitry V. Levin <ldv@altlinux> 6:2.5-alt2
> > > - Updated to glibc-2_5-branch snapshot 20061006.
> > > - Build with -U_FORTIFY_SOURCE and -fno-stack-protector.
> > > - init.d/nscd: Removed obsolete nscd_nischeck.
> > > * Fri Sep 29 2006 Dmitry V. Levin <ldv@altlinux> 6:2.5-alt1
> > 
> > git-push бы сделать
> 
> Спасибо!  Toolchain готов, как я понимаю?  Выжидаю момент, чтобы потом
> избежать автоматической пересборки своих пакетов.

Да, toolchain готов.  Сборки gcc и glibc ещё будут, но никаких побочных
эффектов я от них не жду.  Дата отсечения как раз и была 9-го октября, всё
что было собрано после, судя по всему, в автоматической пересборке не
нуждается.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] glibc 6:2.5-alt2 to git-push

[Alexey Tourbin]

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

On Sat, Oct 14, 2006 at 12:32:27AM +0400, Dmitry V. Levin wrote:
> On Fri, Oct 13, 2006 at 09:19:41PM +0400, Alexey Tourbin wrote:
> > On Fri, Oct 13, 2006 at 07:07:15PM +0400, Alexey Tourbin wrote:
> > > On Tue, Oct 10, 2006 at 03:06:59AM +0400, QA Team Robot wrote:
> > > > glibc - The GNU libc libraries                                  	[20M]
> > > > * Sun Oct 08 2006 Dmitry V. Levin <ldv@altlinux> 6:2.5-alt2
> > > > - Updated to glibc-2_5-branch snapshot 20061006.
> > > > - Build with -U_FORTIFY_SOURCE and -fno-stack-protector.
> > > > - init.d/nscd: Removed obsolete nscd_nischeck.
> > > > * Fri Sep 29 2006 Dmitry V. Levin <ldv@altlinux> 6:2.5-alt1
> > > 
> > > git-push бы сделать
> > 
> > Спасибо!  Toolchain готов, как я понимаю?  Выжидаю момент, чтобы потом
> > избежать автоматической пересборки своих пакетов.
> 
> Да, toolchain готов.  Сборки gcc и glibc ещё будут, но никаких побочных
> эффектов я от них не жду.  Дата отсечения как раз и была 9-го октября, всё
> что было собрано после, судя по всему, в автоматической пересборке не
> нуждается.

В glibc 2.3.5-alt3 добавился warn_unused_result на asprintf и co,
так что софта с -Werror (elinks, который я недавно зафиксил), опять
перестала собираться.  Как-то хотелось бы, чтобы на время фриза новых
warning'ов не появлялось.

Кстати с asprintf результат можно не проверять, вот каким образом:

char *ptr = NULL;
asprintf(&ptr, fmt, ...);
if (ptr) {
	...
} else {
	// asprintf failed
}

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] glibc 6:2.5-alt2 to git-push

[Dmitry V. Levin]

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

On Sun, Oct 15, 2006 at 10:35:52AM +0400, Alexey Tourbin wrote:
> On Sat, Oct 14, 2006 at 12:32:27AM +0400, Dmitry V. Levin wrote:
> > On Fri, Oct 13, 2006 at 09:19:41PM +0400, Alexey Tourbin wrote:
[...]
> > > Спасибо!  Toolchain готов, как я понимаю?  Выжидаю момент, чтобы потом
> > > избежать автоматической пересборки своих пакетов.
> > 
> > Да, toolchain готов.  Сборки gcc и glibc ещё будут, но никаких побочных
> > эффектов я от них не жду.  Дата отсечения как раз и была 9-го октября, всё
> > что было собрано после, судя по всему, в автоматической пересборке не
> > нуждается.
> 
> В glibc 2.3.5-alt3 добавился warn_unused_result на asprintf и co,

Не проверять код возврата asprintf - это аналогично тому, чтобы не проверять
код возврата malloc.  Если бы Дреппер этого не сделал, я бы пропатчил сам.

> так что софта с -Werror (elinks, который я недавно зафиксил), опять
> перестала собираться.  Как-то хотелось бы, чтобы на время фриза новых
> warning'ов не появлялось.

Я постараюсь минимизировать неудобства.

> Кстати с asprintf результат можно не проверять, вот каким образом:
> 
> char *ptr = NULL;
> asprintf(&ptr, fmt, ...);
> if (ptr) {
> 	...
> } else {
> 	// asprintf failed
> }

Это не переносимо на другие дистрибутивы.  У меня когда-то давно была
растянутая во времени дискуссия в libc-alpha с Дреппером на тему
glibc-2.5-alt-asprintf.patch, можно нагуглить.  В конечном итоге меня
поддержали все кроме Дреппера. :)


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] glibc 6:2.5-alt2 to git-push

[Alexey Tourbin]

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

On Sun, Oct 15, 2006 at 02:42:44PM +0400, Dmitry V. Levin wrote:
> > Кстати с asprintf результат можно не проверять, вот каким образом:
> > 
> > char *ptr = NULL;
> > asprintf(&ptr, fmt, ...);
> > if (ptr) {
> > 	...
> > } else {
> > 	// asprintf failed
> > }
> 
> Это не переносимо на другие дистрибутивы.  У меня когда-то давно была
> растянутая во времени дискуссия в libc-alpha с Дреппером на тему
> glibc-2.5-alt-asprintf.patch, можно нагуглить.  В конечном итоге меня
> поддержали все кроме Дреппера. :)

Нет, ты с ним дискутировал о том, что asprintf должен ЗАНУЛЯТЬ ptr если
внутри пошло что-то не так (упал malloc).  Здесь же я НЕ рассчитываю на
то, что asprintf занулит prt -- он уже занулен с самого начала; а всего
лишь на то, что при ошибке asprintf не будет его модифицировать (или же
таки занулит).

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] glibc 6:2.5-alt2 to git-push

[Dmitry V. Levin]

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

On Sun, Oct 15, 2006 at 02:58:00PM +0400, Alexey Tourbin wrote:
> On Sun, Oct 15, 2006 at 02:42:44PM +0400, Dmitry V. Levin wrote:
> > > Кстати с asprintf результат можно не проверять, вот каким образом:
> > > 
> > > char *ptr = NULL;
> > > asprintf(&ptr, fmt, ...);
> > > if (ptr) {
> > > 	...
> > > } else {
> > > 	// asprintf failed
> > > }
> > 
> > Это не переносимо на другие дистрибутивы.  У меня когда-то давно была
> > растянутая во времени дискуссия в libc-alpha с Дреппером на тему
> > glibc-2.5-alt-asprintf.patch, можно нагуглить.  В конечном итоге меня
> > поддержали все кроме Дреппера. :)
> 
> Нет, ты с ним дискутировал о том, что asprintf должен ЗАНУЛЯТЬ ptr если
> внутри пошло что-то не так (упал malloc).

Нет, ты слишком узко трактуешь.

> Здесь же я НЕ рассчитываю на
> то, что asprintf занулит prt -- он уже занулен с самого начала; а всего
> лишь на то, что при ошибке asprintf не будет его модифицировать (или же
> таки занулит).

Ванильный asprintf в случае ошибки оставит по адресу prt неизвестно что,
например, это может быть адрес памяти, которую он использовал во время
работы.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

[devel] [JT] Re: glibc 6:2.5-alt2 to git-push

[Michael Shigorin]

On Sun, Oct 15, 2006 at 02:42:44PM +0400, Dmitry V. Levin wrote:
> > В glibc 2.3.5-alt3 добавился warn_unused_result на asprintf и co,
> Не проверять код возврата asprintf - это аналогично тому, чтобы
> не проверять код возврата malloc.  Если бы Дреппер этого не
> сделал, я бы пропатчил сам.

Я, конечно, чайник, но припоминается утверждение касательно
малоосмысленности проверки кода возврата malloc() на линуксе
с включенным overcommit (что типично)...

> У меня когда-то давно была растянутая во времени дискуссия в
> libc-alpha с Дреппером на тему glibc-2.5-alt-asprintf.patch,
> можно нагуглить.  В конечном итоге меня поддержали все кроме
> Дреппера. :)

Соответственно толку ноль?

PS: а kerncheck туда часом не закидывал?  Во будет забавно,
если опять поддержат все, кроме :]

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] [JT] Re: glibc 6:2.5-alt2 to git-push

[Dmitry V. Levin]

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

On Sun, Oct 15, 2006 at 04:02:14PM +0300, Michael Shigorin wrote:
> On Sun, Oct 15, 2006 at 02:42:44PM +0400, Dmitry V. Levin wrote:
> > > В glibc 2.3.5-alt3 добавился warn_unused_result на asprintf и co,
> > Не проверять код возврата asprintf - это аналогично тому, чтобы
> > не проверять код возврата malloc.  Если бы Дреппер этого не
> > сделал, я бы пропатчил сам.
> 
> Я, конечно, чайник, но припоминается утверждение касательно
> малоосмысленности проверки кода возврата malloc() на линуксе
> с включенным overcommit (что типично)...

Тебе припомнилось какое-то наивное утверждение.
И overcommit не у всех включён, и ulimit -v бывает настроен, и ubc в ovz
может повлиять.

> > У меня когда-то давно была растянутая во времени дискуссия в
> > libc-alpha с Дреппером на тему glibc-2.5-alt-asprintf.patch,
> > можно нагуглить.  В конечном итоге меня поддержали все кроме
> > Дреппера. :)
> 
> Соответственно толку ноль?

Почему ноль?  Я патч прикладываю, поэтому я спокоен.
Просто я знаю, что Алексей как честный мантейнер отправляет патчи
upstream, поэтому ему надо знать про ALT specific.

> PS: а kerncheck туда часом не закидывал?  Во будет забавно,
> если опять поддержат все, кроме :]

Не закидывал и не буду закидывать.
Мне кажется, что дистрибутивной обвязке там не место.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [JT] Re: glibc 6:2.5-alt2 to git-push

[Michael Shigorin]

On Sun, Oct 15, 2006 at 06:39:17PM +0400, Dmitry V. Levin wrote:
> Тебе припомнилось какое-то наивное утверждение.

Давнее всё-таки.

> И overcommit не у всех включён, и ulimit -v бывает настроен,
> и ubc в ovz может повлиять.

Его тогда не было :)  Кажется, как и vz.

> > PS: а kerncheck туда часом не закидывал?
> Не закидывал и не буду закидывать.
> Мне кажется, что дистрибутивной обвязке там не место.

Те, кто собирают glibc, в наши дни просто обязаны сперва думать?
Ну тебе видней.  Я склонен стараться подстелить соломку другим,
поскольку благодарен за то, когда другие предупреждали мои
глупейшие ошибки.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] [JT] Re: glibc 6:2.5-alt2 to git-push

[Dmitry V. Levin]

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

On Sun, Oct 15, 2006 at 09:35:50PM +0300, Michael Shigorin wrote:
> On Sun, Oct 15, 2006 at 06:39:17PM +0400, Dmitry V. Levin wrote:
[...]
> > > PS: а kerncheck туда часом не закидывал?
> > Не закидывал и не буду закидывать.
> > Мне кажется, что дистрибутивной обвязке там не место.
> 
> Те, кто собирают glibc, в наши дни просто обязаны сперва думать?

Несомненно.  Время от времени в libc-alpha@ пишут люди, которые не до
конца подумали, им в ответ регулярно советуют не собирать glibc
самостоятельно.

> Ну тебе видней.  Я склонен стараться подстелить соломку другим,
> поскольку благодарен за то, когда другие предупреждали мои
> глупейшие ошибки.

Эту конкретную проблему лучше решать другими средствами.
Я согласился применить kerncheck только потому, что эти другие средства
несколько сложнее.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [JT] Re: glibc 6:2.5-alt2 to git-push

[Michael Shigorin]

On Sun, Oct 15, 2006 at 11:40:34PM +0400, Dmitry V. Levin wrote:
> > Ну тебе видней.  Я склонен стараться подстелить соломку
> > другим, поскольку благодарен за то, когда другие
> > предупреждали мои глупейшие ошибки.
> Эту конкретную проблему лучше решать другими средствами.
> Я согласился применить kerncheck только потому, что эти другие
> средства несколько сложнее.

Ну у меня после первого шока от анонса (и второго --
от dist-upgrade не проснувшись :) действительно было
лёгкое изумление -- "неужто не получалось не ломать
так радикально, чтоб хотя бы ls работал?".

Но тут остаётся полагаться на апстрим и майнтейнера,
поскольку абсолютно вне моей компетенции.

(собсно ремарка "от твоих пакетов такого не ожидают"
значила примерно то, что я dist-upgrade когда делаю,
на обновлении именно твоих и ещё нескольких людей 
сборок не заостряюсь; думаю, не только я; увы и ах)

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] [JT] Re: glibc 6:2.5-alt2 to git-push

[Dmitry V. Levin]

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

On Sun, Oct 15, 2006 at 10:59:23PM +0300, Michael Shigorin wrote:
> On Sun, Oct 15, 2006 at 11:40:34PM +0400, Dmitry V. Levin wrote:
> > > Ну тебе видней.  Я склонен стараться подстелить соломку
> > > другим, поскольку благодарен за то, когда другие
> > > предупреждали мои глупейшие ошибки.
> > Эту конкретную проблему лучше решать другими средствами.
> > Я согласился применить kerncheck только потому, что эти другие
> > средства несколько сложнее.
> 
> Ну у меня после первого шока от анонса (и второго --
> от dist-upgrade не проснувшись :) действительно было
> лёгкое изумление -- "неужто не получалось не ломать
> так радикально, чтоб хотя бы ls работал?".

Поскольку ls слинкован с -lrt, то он всё равно перестал бы работать
из-за nptl.

Для зависимости на версии ядра можно попробовать реализовать в rpm
какой-нибудь динамический provides.

> Но тут остаётся полагаться на апстрим и майнтейнера,
> поскольку абсолютно вне моей компетенции.
> 
> (собсно ремарка "от твоих пакетов такого не ожидают"
> значила примерно то, что я dist-upgrade когда делаю,
> на обновлении именно твоих и ещё нескольких людей 
> сборок не заостряюсь; думаю, не только я; увы и ах)

Я не понимаю, чего ты ожидал, если всем известно, что для nptl нужно ядро 2.6.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] glibc 6:2.5-alt2 to git-push

[Alexey Tourbin]

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

On Sun, Oct 15, 2006 at 02:42:44PM +0400, Dmitry V. Levin wrote:
> On Sun, Oct 15, 2006 at 10:35:52AM +0400, Alexey Tourbin wrote:
> > On Sat, Oct 14, 2006 at 12:32:27AM +0400, Dmitry V. Levin wrote:
> > > On Fri, Oct 13, 2006 at 09:19:41PM +0400, Alexey Tourbin wrote:
> [...]
> > > > Спасибо!  Toolchain готов, как я понимаю?  Выжидаю момент, чтобы потом
> > > > избежать автоматической пересборки своих пакетов.
> > > 
> > > Да, toolchain готов.  Сборки gcc и glibc ещё будут, но никаких побочных
> > > эффектов я от них не жду.  Дата отсечения как раз и была 9-го октября, всё
> > > что было собрано после, судя по всему, в автоматической пересборке не
> > > нуждается.
> > 
> > В glibc 2.3.5-alt3 добавился warn_unused_result на asprintf и co,
> 
> Не проверять код возврата asprintf - это аналогично тому, чтобы не проверять
> код возврата malloc.  Если бы Дреппер этого не сделал, я бы пропатчил сам.

Вообще сейчас warn_unused_result не очень последовательно.  write() --
это warn_unused_result, а printf() -- нет; хотя printf() использует
write(), и фактически возвращает результат write().

С другой стороны, в нормальной ситуации stdout как бы должен быть живой
и нужно очень постараться, чтобы printf() отвалил, тем более по
дескриптору.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] glibc 6:2.5-alt2 to git-push

[Dmitry V. Levin]

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

On Tue, Oct 24, 2006 at 02:17:15AM +0400, Alexey Tourbin wrote:
> On Sun, Oct 15, 2006 at 02:42:44PM +0400, Dmitry V. Levin wrote:
> > On Sun, Oct 15, 2006 at 10:35:52AM +0400, Alexey Tourbin wrote:
> > > On Sat, Oct 14, 2006 at 12:32:27AM +0400, Dmitry V. Levin wrote:
> > > > On Fri, Oct 13, 2006 at 09:19:41PM +0400, Alexey Tourbin wrote:
> > [...]
> > > > > Спасибо!  Toolchain готов, как я понимаю?  Выжидаю момент, чтобы потом
> > > > > избежать автоматической пересборки своих пакетов.
> > > > 
> > > > Да, toolchain готов.  Сборки gcc и glibc ещё будут, но никаких побочных
> > > > эффектов я от них не жду.  Дата отсечения как раз и была 9-го октября, всё
> > > > что было собрано после, судя по всему, в автоматической пересборке не
> > > > нуждается.
> > > 
> > > В glibc 2.3.5-alt3 добавился warn_unused_result на asprintf и co,
> > 
> > Не проверять код возврата asprintf - это аналогично тому, чтобы не проверять
> > код возврата malloc.  Если бы Дреппер этого не сделал, я бы пропатчил сам.
> 
> Вообще сейчас warn_unused_result не очень последовательно.  write() --
> это warn_unused_result, а printf() -- нет; хотя printf() использует
> write(), и фактически возвращает результат write().

Проверить код возврата printf() в случае partial write бывает непросто,
тут скорее ferror() подойдёт.

> С другой стороны, в нормальной ситуации stdout как бы должен быть живой
> и нужно очень постараться, чтобы printf() отвалил, тем более по
> дескриптору.

Это бывает (и легко воспроизвести) в случае ENOSPC.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] glibc 6:2.5-alt2 to git-push

[Alexey Tourbin]

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

On Tue, Oct 24, 2006 at 02:25:18AM +0400, Dmitry V. Levin wrote:
> > Вообще сейчас warn_unused_result не очень последовательно.  write() --
> > это warn_unused_result, а printf() -- нет; хотя printf() использует
> > write(), и фактически возвращает результат write().
> 
> Проверить код возврата printf() в случае partial write бывает непросто,
> тут скорее ferror() подойдёт.

Ох, он же ещё и буферизирован по-чудному.  Я об этом забыл. :(

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

[devel] warn_unused_result портят жизнь мэйнтейнеру

[Slava Semushin]

--- Alexey Tourbin 2006-10-24 02:17:15 +0400
+++ Slava Semushin 2006-10-24 05:59:52 +0700

AT> Вообще сейчас warn_unused_result 

    Ох, задели больную тему...

    У меня три пакета (libtap, wmdiskmon и wmbday) с -Wall -Werror не
    пересобираются из-за этих проблем c неиспользуемым результатом. В
    частности из-за fgets()/vasprintf()/asprintf(). Также я не обновляю
    clines потому что там вылезли варнинги с write().

    Чесгря только час назад об этом всём думал. Я начал их фиксить.  Но
    не уверен, что правильно. Поэтому я думаю, как-то отключить эти
    предупреждения. Или вовсе тогда -Werror уберу.

    Пожалуйста, объясните, зачем были введены эти предупреждения? И почему
    аж в -Wall ? Почему бы не включать их по -W? Неужто всё так серьёзно?

    Также интересно возможно ли как-то заткнуть эти предупреждения?
    -Wno-что? Приведение к void не помогло. В случае с libtap, значение
    [v]asprintf() проверяется, только чуть иначе -- по первому аргументу.

-- 
+ Semushin Slava | php-coder @ ngs.ru
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] warn_unused_result портят жизнь мэйнтейнеру

[Alexey Tourbin]

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

On Tue, Oct 24, 2006 at 06:07:52AM +0700, Slava Semushin wrote:
> 
> --- Alexey Tourbin 2006-10-24 02:17:15 +0400
> +++ Slava Semushin 2006-10-24 05:59:52 +0700
> 
> AT> Вообще сейчас warn_unused_result 
> 
>     Ох, задели больную тему...
> 
>     У меня три пакета (libtap, wmdiskmon и wmbday) с -Wall -Werror не
>     пересобираются из-за этих проблем c неиспользуемым результатом. В
>     частности из-за fgets()/vasprintf()/asprintf(). Также я не обновляю
>     clines потому что там вылезли варнинги с write().
> 
>     Чесгря только час назад об этом всём думал. Я начал их фиксить.  Но
>     не уверен, что правильно. Поэтому я думаю, как-то отключить эти
>     предупреждения. Или вовсе тогда -Werror уберу.
> 
>     Пожалуйста, объясните, зачем были введены эти предупреждения? И почему
>     аж в -Wall ? Почему бы не включать их по -W? Неужто всё так серьёзно?
> 
>     Также интересно возможно ли как-то заткнуть эти предупреждения?
>     -Wno-что? Приведение к void не помогло. В случае с libtap, значение
>     [v]asprintf() проверяется, только чуть иначе -- по первому аргументу.

-Wno-pointer-sign помогает в коде, в котором перемешано 'char *' и
'unsigned char *'.  Ещё помогает -fno-strict-aliasing, это какой-то
gcc'шный бзик, который я до конца не понимаю; к тому же в результате
компиляции этот strict aliasing может дать потенциально ошибочный код,
что однако на практике я пока краем уха слышал только про IA64.

В общем -Werror нужен только для "очень важных" пакетов или для куражу,
например собственный код поправить.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] warn_unused_result портят жизнь мэйнтейнеру

[Andrey Rahmatullin]

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

On Tue, Oct 24, 2006 at 07:43:35AM +0400, Alexey Tourbin wrote:
> В общем -Werror нужен только для "очень важных" пакетов или для куражу,
> например собственный код поправить.
Я давно об этом говорю.

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(8):

Мы работаем, чтобы зарабатывать деньги, или зарабатываем деньги, чтобы
работать? Я всегда стремился ко второму.
		-- aen in devel@

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] warn_unused_result портят жизнь мэйнтейнеру

[Alexey Tourbin]

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

On Tue, Oct 24, 2006 at 09:57:43AM +0600, Andrey Rahmatullin wrote:
> On Tue, Oct 24, 2006 at 07:43:35AM +0400, Alexey Tourbin wrote:
> > В общем -Werror нужен только для "очень важных" пакетов или для куражу,
> > например собственный код поправить.
> Я давно об этом говорю.

Ну тут как подумать.  -Wall всё же вылавливает потенциальные ошибки.
Если собирается с -Wall -Werror, значит потенциальных ошибок меньше,
совесть чище.  Причем есть ещё -W aka -Wextra, это уже совсем для
маньяков.

Тут ещё от maintainer'а зависит.  Если он просто заворачивает тарболл
в rpm как заворачивают селедку в газету, тогда конечно видал он все эти
-Werror у бабушки в деревне.  Есть у нас такие maintainer'ы.  А кто в
коде разбирается тому есть резон зафиксить.

Всё-таки с точки зрения индвидуального престижа maintainer'а и лица team
в целом, а также с точки зрения целесообразности нашей индивидуальной и
коллективной работы, -Werror в некоторых пакетах стоит фиксить.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] warn_unused_result портят жизнь мэйнтейнеру

[Andrey Rahmatullin]

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

On Tue, Oct 24, 2006 at 08:16:02AM +0400, Alexey Tourbin wrote:
> Ну тут как подумать.  -Wall всё же вылавливает потенциальные ошибки.
> Если собирается с -Wall -Werror, значит потенциальных ошибок меньше,
> совесть чище. 
Но когда несистемный пакет уходит в орфанед из-за варнинга - это
неправильно.

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(8):

> > На то он и Sisyphus, что бы после обновления получать массу удовольствия
>А я думал что он для разработки нужен...
А кто сказал, что от разработки нельзя получать массу удовольствия ??? ;-)
		-- rider in devel@

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] [JT] warn_unused_result портят жизнь мэйнтейнеру

[Pavlov Konstantin]

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

On Tuesday 24 October 2006 08:16, Alexey Tourbin wrote:
>  Если он просто заворачивает тарболл
> в rpm как заворачивают селедку в газету, тогда конечно видал он все эти
> -Werror у бабушки в деревне.  Есть у нас такие maintainer'ы.  А кто в
> коде разбирается тому есть резон зафиксить.

А еще есть апстрим. Который типа:

This software module was originally developed by

    Ming-Chieh Lee (mingcl@microsoft.com), Microsoft Corporation
    Wei-ge Chen (wchen@microsoft.com), Microsoft Corporation
    Bruce Lin (blin@microsoft.com), Microsoft Corporation
    Chuang Gu (chuanggu@microsoft.com), Microsoft Corporation
    Simon Winder (swinder@microsoft.com), Microsoft Corporation
    (date: March, 1996)
and edited by
        Wei Wu (weiwu@stallion.risc.rockwell.com) Rockwell Science Center

and also edited by
    Yoshihiro Kikuchi (TOSHIBA CORPORATION)
    Takeshi Nagai (TOSHIBA CORPORATION)
    Toshiaki Watanabe (TOSHIBA CORPORATION)
    Noboru Yamaguchi (TOSHIBA CORPORATION)

and also edited by
    David B. Shu (dbshu@hrl.com), Hughes Electronics/HRL Laboratories

    Marc Mongenet (Marc.Mongenet@epfl.ch), Swiss Federal Institute of 
Technology, Lausanne (EPFL)

    Mathias Wien (wien@ient.rwth-aachen.de) RWTH Aachen / Robert BOSCH GmbH

and also edited by
    Yoshinori Suzuki (Hitachi, Ltd.)

and also edited by
    Hideaki Kimata (NTT)

and also edited by
    Fujitsu Laboratories Ltd. (contact: Eishi Morimatsu)

and also edited by
    Massimo Ravasi (Massimo.Ravasi@epfl.ch), Swiss Federal Institute of 
Technology, Lausanne (EPFL)

and also edited by
    Takefumi Nagumo (nagumo@av.crl.sony.co.jp), Sony Corporation
    Sehoon Son (shson@unitel.co.kr) Samsung AIT

in the course of development of the MPEG-4 Video (ISO/IEC 14496-2).
This software module is an implementation of a part of one or more MPEG-4 
Video tools
as specified by the MPEG-4 Video.

Вот и копайся потом в таком выясняя чего они там хотели сказать...
find / subst улучшает жизнь.

-- 
Pavlov Konstantin,
ALT Linux Team,
VideoLAN Team,
jid: thresh@altlinux.org

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [JT] warn_unused_result портят жизнь мэйнтейнеру

[Alexey Tourbin]

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

On Tue, Oct 24, 2006 at 09:15:55AM +0400, Pavlov Konstantin wrote:
> On Tuesday 24 October 2006 08:16, Alexey Tourbin wrote:
> >  Если он просто заворачивает тарболл
> > в rpm как заворачивают селедку в газету, тогда конечно видал он все эти
> > -Werror у бабушки в деревне.  Есть у нас такие maintainer'ы.  А кто в
> > коде разбирается тому есть резон зафиксить.
> 
> А еще есть апстрим. Который типа:
> 
> This software module was originally developed by
> 
>     Ming-Chieh Lee (mingcl@microsoft.com), Microsoft Corporation
>     Wei-ge Chen (wchen@microsoft.com), Microsoft Corporation
>     Bruce Lin (blin@microsoft.com), Microsoft Corporation
>     Chuang Gu (chuanggu@microsoft.com), Microsoft Corporation
>     Simon Winder (swinder@microsoft.com), Microsoft Corporation
>     (date: March, 1996)
> and edited by
>         Wei Wu (weiwu@stallion.risc.rockwell.com) Rockwell Science Center
> 
> and also edited by
>     Yoshihiro Kikuchi (TOSHIBA CORPORATION)
>     Takeshi Nagai (TOSHIBA CORPORATION)
>     Toshiaki Watanabe (TOSHIBA CORPORATION)
>     Noboru Yamaguchi (TOSHIBA CORPORATION)
> 
> and also edited by
>     David B. Shu (dbshu@hrl.com), Hughes Electronics/HRL Laboratories
> 
>     Marc Mongenet (Marc.Mongenet@epfl.ch), Swiss Federal Institute of 
> Technology, Lausanne (EPFL)
> 
>     Mathias Wien (wien@ient.rwth-aachen.de) RWTH Aachen / Robert BOSCH GmbH
> 
> and also edited by
>     Yoshinori Suzuki (Hitachi, Ltd.)
> 
> and also edited by
>     Hideaki Kimata (NTT)
> 
> and also edited by
>     Fujitsu Laboratories Ltd. (contact: Eishi Morimatsu)
> 
> and also edited by
>     Massimo Ravasi (Massimo.Ravasi@epfl.ch), Swiss Federal Institute of 
> Technology, Lausanne (EPFL)
> 
> and also edited by
>     Takefumi Nagumo (nagumo@av.crl.sony.co.jp), Sony Corporation
>     Sehoon Son (shson@unitel.co.kr) Samsung AIT
> 
> in the course of development of the MPEG-4 Video (ISO/IEC 14496-2).
> This software module is an implementation of a part of one or more MPEG-4 
> Video tools
> as specified by the MPEG-4 Video.
> 
> Вот и копайся потом в таком выясняя чего они там хотели сказать...
> find / subst улучшает жизнь.

И что, код совсем плохой?  Вам же хуже. :)

По-моему, нежелание смотреть на код серьезно снижает уровень
maintainer'а и, косвенно, team в целом.  -Werror в этом отношении --
не самоцель, а повод посмотреть на код.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [JT] warn_unused_result портят жизнь мэйнтейнеру

[Pavlov Konstantin]

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

On Tuesday 24 October 2006 09:45, Alexey Tourbin wrote:
> > Вот и копайся потом в таком выясняя чего они там хотели сказать...
> > find / subst улучшает жизнь.
>
> И что, код совсем плохой?  Вам же хуже. :)
>
> По-моему, нежелание смотреть на код серьезно снижает уровень
> maintainer'а и, косвенно, team в целом.  -Werror в этом отношении --
> не самоцель, а повод посмотреть на код.

На код можно глянуть в git.alt:thresh/packages/mpeg4ip.git, если уж так 
интересно ;)

Я ничего против кода в общем и целом не имею, смотреть туда приходится и т.п. 
Но пытаться править именно в этом случае как-то ну сооовсем не хочется. Есть 
более интересные проекты.

-- 
Pavlov Konstantin,
ALT Linux Team,
VideoLAN Team,
jid: thresh@altlinux.org

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [JT] warn_unused_result портят жизнь мэйнтейнеру

[Michael Shigorin]

On Tue, Oct 24, 2006 at 09:45:00AM +0400, Alexey Tourbin wrote:
> По-моему, нежелание смотреть на код серьезно снижает уровень
> maintainer'а и, косвенно, team в целом.  -Werror в этом
> отношении -- не самоцель, а повод посмотреть на код.

Вот.  А цели бывают разные.

-- 
майнтейнер,
который предпочитает
не смотреть...

Re: [devel] [JT] warn_unused_result портят жизнь мэйнтейнеру

[Alexey Tourbin]

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

On Tue, Oct 24, 2006 at 03:24:56PM +0300, Michael Shigorin wrote:
> On Tue, Oct 24, 2006 at 09:45:00AM +0400, Alexey Tourbin wrote:
> > По-моему, нежелание смотреть на код серьезно снижает уровень
> > maintainer'а и, косвенно, team в целом.  -Werror в этом
> > отношении -- не самоцель, а повод посмотреть на код.
> 
> Вот.  А цели бывают разные.

Цели не безусловно разные.  Это всё равно как сказать, что и люди все
разные -- кто поумнее, кто поглупее, а кто и вовсе дурак.  Дайвёрсити
здесь плохое оправдание.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]