[devel] -Werror приводит к ошибкам в системных хедерах

[devel] -Werror приводит к ошибкам в системных хедерах

[Slava Semushin]

Здравствуйте!

Собираю новую версию mpdscribble. Включил --enable-werror и получаю:

gcc -std=gnu99 -DHAVE_CONFIG_H -I.  -I/usr/include/glib-2.0
-I/usr/lib/glib-2.0/include   -pthread -I/usr/include/glib-2.0
-I/usr/lib/glib-2.0/include   -I/usr/include/libsoup-2.4
-I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include
-I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include   -I./src
-I./src/libmpdclient -DFILE_CONF='"/usr/local/etc/mpdscribble.conf"'
-Werror -pedantic-errors -DNDEBUG -Wall -Wextra
-Wno-deprecated-declarations -Wmissing-prototypes
-Wdeclaration-after-statement -Wshadow -Wpointer-arith
-Wstrict-prototypes -Wcast-qual -Wwrite-strings -g -O2 -MT conn.o -MD
-MP -MF .deps/conn.Tpo -c -o conn.o `test -f 'src/conn.c' || echo
'./'`src/conn.c
In file included from /usr/include/libsoup-2.4/libsoup/soup.h:30,
                 from src/conn.c:26:
/usr/include/libsoup-2.4/libsoup/soup-session-feature.h:21: error:
struct has no members
make[1]: *** [conn.o] Ошибка 1

Решил обойти эту ошибку, включив только нужные программе хедеры вместо
обобщающего soup.h

Далее:

In file included from /usr/include/sys/syslog.h:207,
                 from /usr/include/syslog.h:1,
                 from src/log.c:32:
/usr/include/bits/syslog.h: In function 'syslog':
/usr/include/bits/syslog.h:32: error: ISO C forbids 'return' with
expression, in function returning void
/usr/include/bits/syslog.h: In function 'vsyslog':
/usr/include/bits/syslog.h:48: error: ISO C forbids 'return' with
expression, in function returning void
make[1]: *** [log.o] Ошибка 1

А тут как быть?

Не хотелось бы отключать -Werror флаг, к тому же очень похоже, что у
апстримного автора всё собирается и с ним, а вот у нас почему-то нет.
СтОит ли вешать баги на эти хедеры или.. что делать вообщем?

Спасибо за внимание.

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] -Werror приводит к ошибкам в системных хедерах

[Ivan A. Melnikov]

On Mon, 5 Jan 2009 15:36:54 +0600
"Slava Semushin" <slava.semushin@gmail.com> wrote:

> Здравствуйте!
> 
> Собираю новую версию mpdscribble. Включил --enable-werror и получаю:
[...]
> /usr/include/libsoup-2.4/libsoup/soup-session-feature.h:21: error:
> struct has no members

Это из-за опций gcc -pedantic-errors и/или -pedantic...

> 
> Решил обойти эту ошибку, включив только нужные программе хедеры вместо
> обобщающего soup.h
> 
> Далее:
> 
> In file included from /usr/include/sys/syslog.h:207,
>                  from /usr/include/syslog.h:1,
>                  from src/log.c:32:
> /usr/include/bits/syslog.h: In function 'syslog':
> /usr/include/bits/syslog.h:32: error: ISO C forbids 'return' with
> expression, in function returning void
> /usr/include/bits/syslog.h: In function 'vsyslog':
> /usr/include/bits/syslog.h:48: error: ISO C forbids 'return' with
> expression, in function returning void
> make[1]: *** [log.o] Ошибка 1
> 
> А тут как быть?
> 

Для меня это было сюрпризом, но действитедльно, в стандарте C99 пукт
6.8.6.4 параграф 1 запрещает return statement with an expression  в
function whose return type is void. Опять же, gcc обращает на это
внимание только с -pedantic{,-errors}.

JFI, в c++98  (пункт 6.6.3 параграф 3) и последующих (правда, посленего
драфта C++0x у меня под рукой нет) такоe разрешено. ЕМНИП такое можно
было и в C89.

> Не хотелось бы отключать -Werror флаг, к тому же очень похоже, что у
> апстримного автора всё собирается и с ним, а вот у нас почему-то нет.
> СтОит ли вешать баги на эти хедеры или.. что делать вообщем?
> 

Убирать -pedantic{,-errors}. Они ИМХО с -Werror не очень хорошо
сочетаются. Насчёт развешивания насекомых ничего не могу посоветовать.

> Спасибо за внимание.
> 

Пожалуйста.

-- 
Best Regards,
Ivan A. Melnikov

Re: [devel] -Werror приводит к ошибкам в системных хедерах

[Alexey I. Froloff]

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

* Slava Semushin <slava.semushin@> [090105 12:42]:
[..skip..]
> А тут как быть?
Перестать заниматься пыхнёй и продолжить празднование нового года
(и рождества).

-- 
Regards,
Sir Raorn.

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

Re: [devel] -Werror приводит к ошибкам в системных хедерах

[Ivan A. Melnikov]

On Mon, 5 Jan 2009 15:50:34 +0300
"Alexey I. Froloff" <raorn@altlinux.ru> wrote:

> * Slava Semushin <slava.semushin@> [090105 12:42]:
> [..skip..]
> > А тут как быть?
> Перестать заниматься пыхнёй и продолжить празднование нового года
> (и рождества).
> 

+inf

-- 
Best Regards,
Ivan A. Melnikov

Re: [devel] -Werror приводит к ошибкам в системных хедерах

[Dmitry V. Levin]

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

On Mon, Jan 05, 2009 at 03:36:54PM +0600, Slava Semushin wrote:
> Здравствуйте!
> 
> Собираю новую версию mpdscribble. Включил --enable-werror и получаю:
> 
> gcc -std=gnu99 -DHAVE_CONFIG_H -I.  -I/usr/include/glib-2.0
> -I/usr/lib/glib-2.0/include   -pthread -I/usr/include/glib-2.0
> -I/usr/lib/glib-2.0/include   -I/usr/include/libsoup-2.4
> -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include
> -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include   -I./src
> -I./src/libmpdclient -DFILE_CONF='"/usr/local/etc/mpdscribble.conf"'
> -Werror -pedantic-errors -DNDEBUG -Wall -Wextra
> -Wno-deprecated-declarations -Wmissing-prototypes
> -Wdeclaration-after-statement -Wshadow -Wpointer-arith
> -Wstrict-prototypes -Wcast-qual -Wwrite-strings -g -O2

Сочетание -std=gnu99 -pedantic-errors -Wall -Wextra -Werror
в glibc не поддерживается.  Предлагаю -pedantic-errors убрать.

> In file included from /usr/include/sys/syslog.h:207,
>                  from /usr/include/syslog.h:1,
>                  from src/log.c:32:
> /usr/include/bits/syslog.h: In function 'syslog':
> /usr/include/bits/syslog.h:32: error: ISO C forbids 'return' with
> expression, in function returning void
> /usr/include/bits/syslog.h: In function 'vsyslog':
> /usr/include/bits/syslog.h:48: error: ISO C forbids 'return' with
> expression, in function returning void
> make[1]: *** [log.o] Ошибка 1
> 
> А тут как быть?

Это можно исправить, но вылезет где-нибудь в другом месте.


-- 
ldv

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

Re: [devel] -Werror приводит к ошибкам в системных хедерах

[Slava Semushin]

5 января 2009 г. 23:10 пользователь Dmitry V. Levin <ldv@altlinux> написал:
[...]
> Сочетание -std=gnu99 -pedantic-errors -Wall -Wextra -Werror
> в glibc не поддерживается.  Предлагаю -pedantic-errors убрать.

Хорошо, уберу.

Нужно ли сообщить об этом в апстрим? (чтобы --enable-werror не включал
-pedantic-errors, а только -Werror)

Если да, то как бы получше это объяснить/доказать? Может ссылочка
какая на Дреппера или ещё что?

[...]

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] -Werror приводит к ошибкам в системных хедерах

[Dmitry V. Levin]

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

On Tue, Jan 06, 2009 at 02:29:25AM +0600, Slava Semushin wrote:
> 5 января 2009 г. 23:10 пользователь Dmitry V. Levin <ldv@altlinux> написал:
> [...]
> > Сочетание -std=gnu99 -pedantic-errors -Wall -Wextra -Werror
> > в glibc не поддерживается.  Предлагаю -pedantic-errors убрать.
> 
> Хорошо, уберу.
> 
> Нужно ли сообщить об этом в апстрим? (чтобы --enable-werror не включал
> -pedantic-errors, а только -Werror)

Попробуйте исправить bits/syslog.h (убрать return в тех строчках, на
которые есть предупреждения).  Если в собираемом пакете больше ничего не
всплывёт, то проще исправить bits/syslog.h; если будет всплывать и
дальше, то сообщить апстриму, что системные файлы не готовы.


-- 
ldv

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

Re: [devel] -Werror приводит к ошибкам в системных хедерах

[Slava Semushin]

2009/1/5 Dmitry V. Levin <ldv@altlinux>:
> On Mon, Jan 05, 2009 at 03:36:54PM +0600, Slava Semushin wrote:
>> Здравствуйте!
>>
>> Собираю новую версию mpdscribble. Включил --enable-werror и получаю:
>>
>> gcc -std=gnu99 -DHAVE_CONFIG_H -I.  -I/usr/include/glib-2.0
>> -I/usr/lib/glib-2.0/include   -pthread -I/usr/include/glib-2.0
>> -I/usr/lib/glib-2.0/include   -I/usr/include/libsoup-2.4
>> -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include
>> -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include   -I./src
>> -I./src/libmpdclient -DFILE_CONF='"/usr/local/etc/mpdscribble.conf"'
>> -Werror -pedantic-errors -DNDEBUG -Wall -Wextra
>> -Wno-deprecated-declarations -Wmissing-prototypes
>> -Wdeclaration-after-statement -Wshadow -Wpointer-arith
>> -Wstrict-prototypes -Wcast-qual -Wwrite-strings -g -O2
>
> Сочетание -std=gnu99 -pedantic-errors -Wall -Wextra -Werror
> в glibc не поддерживается.  Предлагаю -pedantic-errors убрать.
>
>> In file included from /usr/include/sys/syslog.h:207,
>>                  from /usr/include/syslog.h:1,
>>                  from src/log.c:32:
>> /usr/include/bits/syslog.h: In function 'syslog':
>> /usr/include/bits/syslog.h:32: error: ISO C forbids 'return' with
>> expression, in function returning void
>> /usr/include/bits/syslog.h: In function 'vsyslog':
>> /usr/include/bits/syslog.h:48: error: ISO C forbids 'return' with
>> expression, in function returning void
>> make[1]: *** [log.o] Ошибка 1
>>
>> А тут как быть?
>
> Это можно исправить, но вылезет где-нибудь в другом месте.

Да, действительно:

http://git.altlinux.org/tasks/2258/build/1/i586/log

:-(

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru