[devel] Обновление libsoup (was: Re: mpdscribble-0.18.1-alt1: Sisyphus/i586 test rebuild failed)

[devel] Обновление libsoup (was: Re: mpdscribble-0.18.1-alt1: Sisyphus/i586 test rebuild failed)

[Slava Semushin]

2009/10/12 QA Team Robot <qa@altlinux.org>:
> Package: mpdscribble-0.18.1-alt1
[...]
> + make -j1 --silent --no-print-directory
> cc1: warnings being treated as errors
> src/http_client_soup.c: In function 'http_client_request':
> src/http_client_soup.c:140: error: cast discards qualifiers from pointer target type
> src/http_client_soup.c:163: error: cast discards qualifiers from pointer target type
> make[1]: *** [http_client_soup.o] Error 1

Это нормально, что после обновления libsoup у меня сломалась сборка
клиента (пусть даже и использующая -W -Wall -Werror)?


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

Re: [devel] Обновление libsoup (was: Re: mpdscribble-0.18.1-alt1: Sisyphus/i586 test rebuild failed)

[REAL]

Slava Semushin пишет:
>> Package: mpdscribble-0.18.1-alt1
> [...]
>> + make -j1 --silent --no-print-directory
>> cc1: warnings being treated as errors
>> src/http_client_soup.c: In function 'http_client_request':
>> src/http_client_soup.c:140: error: cast discards qualifiers from pointer target type
>> src/http_client_soup.c:163: error: cast discards qualifiers from pointer target type
>> make[1]: *** [http_client_soup.o] Error 1
> 
> Это нормально, что после обновления libsoup у меня сломалась сборка
> клиента (пусть даже и использующая -W -Wall -Werror)?

Насчёт конкретно данного случая не скажу, но -Werror сломает сборку 
даже при передаче параметра int, если ожидается const int.

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] шутки про -Werror

[Dmitry V. Levin]

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

On Mon, Oct 12, 2009 at 05:34:06PM +0800, REAL wrote:
[...]
> Насчёт конкретно данного случая не 
> скажу, но -Werror сломает сборку даже при 
> передаче параметра int, если ожидается const 
> int.

Я надеюсь, что это вы просто так пошутили?


-- 
ldv

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

Re: [devel] шутки про -Werror

[REAL]

Dmitry V. Levin пишет:
>> Насчёт конкретно данного случая не 
>> скажу, но -Werror сломает сборку даже при 
>> передаче параметра int, если ожидается const 
>> int.
> 
> Я надеюсь, что это вы просто так пошутили?

Утрировал, хотя от некоторого количества сообщений, которые только 
мешают, я бы с радостью избавился, и в таком случае сам бы использовал 
-Werror всегда.

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] шутки про -Werror

[Led]

On Tuesday, 13 October 2009 05:55:06 REAL wrote:
> Dmitry V. Levin пишет:
> >> Насчёт конкретно данного случая не
> >> скажу, но -Werror сломает сборку даже при
> >> передаче параметра int, если ожидается const
> >> int.
> >
> > Я надеюсь, что это вы просто так пошутили?
>
> Утрировал, хотя от некоторого количества сообщений, которые только
> мешают, я бы с радостью избавился, и в таком случае сам бы использовал
> -Werror всегда.

Как отсутствие -Werror мешает избавиться от "некоторого количества сообщений"?

-- 
Led

Re: [devel] шутки про -Werror

[REAL]

Led пишет:
>> Dmitry V. Levin пишет:
>>>> Насчёт конкретно данного случая не
>>>> скажу, но -Werror сломает сборку даже при
>>>> передаче параметра int, если ожидается const
>>>> int.
>>> Я надеюсь, что это вы просто так пошутили?
>> Утрировал, хотя от некоторого количества сообщений, которые только
>> мешают, я бы с радостью избавился, и в таком случае сам бы использовал
>> -Werror всегда.
> 
> Как отсутствие -Werror мешает избавиться от "некоторого количества сообщений"?

Не отсутствие, а присутствие. Я бы хотел включить _все_ (почти) 
сообщения не как warning, а как error, игнорируя только самые 
бесполезные (типа warn_unused_result). Это возможно?

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] шутки про -Werror

[Led]

On Tuesday, 13 October 2009 12:41:13 REAL wrote:
> Led пишет:
> >> Dmitry V. Levin пишет:
> >>>> Насчёт конкретно данного случая не
> >>>> скажу, но -Werror сломает сборку даже при
> >>>> передаче параметра int, если ожидается const
> >>>> int.
> >>>
> >>> Я надеюсь, что это вы просто так пошутили?
> >>
> >> Утрировал, хотя от некоторого количества сообщений, которые только
> >> мешают, я бы с радостью избавился, и в таком случае сам бы использовал
> >> -Werror всегда.
> >
> > Как отсутствие -Werror мешает избавиться от "некоторого количества
> > сообщений"?
>
> Не отсутствие, а присутствие. Я бы хотел включить _все_ (почти)
> сообщения не как warning, а как error,

Зачем? Что мешает просмотреть лог сборки и исправить то, что требуется?

> игнорируя только самые 
> бесполезные (типа warn_unused_result). Это возможно?



-- 
Led

Re: [devel] шутки про -Werror

[REAL]

Led пишет:
>> Не отсутствие, а присутствие. Я бы хотел включить _все_ (почти)
>> сообщения не как warning, а как error,
> 
> Зачем? Что мешает просмотреть лог сборки и исправить то, что требуется?

Мой вариант мне представляется более удобным.

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] шутки про -Werror

[Денис Смирнов]

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

On Tue, Oct 13, 2009 at 05:41:13PM +0800, REAL wrote:

R> Не отсутствие, а присутствие. Я бы хотел включить _все_ (почти) 
R> сообщения не как warning, а как error, игнорируя только самые 
R> бесполезные (типа warn_unused_result). Это возможно?

Ничего сееб бесполезные...

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

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

Re: [devel] шутки про -Werror

[REAL]

Денис Смирнов пишет:
> R> Не отсутствие, а присутствие. Я бы хотел включить _все_ (почти) 
> R> сообщения не как warning, а как error, игнорируя только самые 
> R> бесполезные (типа warn_unused_result). Это возможно?
> 
> Ничего сееб бесполезные...

Ну пока ещё ни разу не было причин обращать на эти сообщения внимания. 
А что, были прецеденты? Хотелось бы ознакомиться, если так, и в чём 
заключалась проблема.

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] шутки про -Werror

[Денис Смирнов]

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

On Thu, Oct 15, 2009 at 10:53:22AM +0800, REAL wrote:

R> Ну пока ещё ни разу не было причин обращать на эти сообщения внимания. 
R> А что, были прецеденты? Хотелось бы ознакомиться, если так, и в чём 
R> заключалась проблема.

К примеру многие не проверяют результат работы fwrite -- это почти
гарантирует грабли в случае работы с NFS :)

Дело даже не в прецедентах. Не проверять код возврата некоторых функций
уже само по себе может гарантировать проблемы в некоторых ситуациях.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

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

Re: [devel] шутки про -Werror

[Kharitonov A. Dmitry]

Денис Смирнов пишет:
> On Thu, Oct 15, 2009 at 10:53:22AM +0800, REAL wrote:
>
> R> Ну пока ещё ни разу не было причин обращать на эти сообщения внимания. 
> R> А что, были прецеденты? Хотелось бы ознакомиться, если так, и в чём 
> R> заключалась проблема.
>
> К примеру многие не проверяют результат работы fwrite -- это почти
> гарантирует грабли в случае работы с NFS :)
>
> Дело даже не в прецедентах. Не проверять код возврата некоторых функций
> уже само по себе может гарантировать проблемы в некоторых ситуациях.
>   
Из-за кеширования записи ошибка часто обнаруживается только при закрытии 
файла, когда сделать уже мало чего можно (выводимые данные уже 
уничтожены -- обычно файл закрывают в самом конце программы), и это 
классическая проблема, которую разрешить не все умеют, по этому и не 
проверяют.

Re: [devel] шутки про -Werror

[Max Ivanov]

> Из-за кеширования записи ошибка часто обнаруживается только при закрытии
> файла, когда сделать уже мало чего можно (выводимые данные уже уничтожены --
> обычно файл закрывают в самом конце программы), и это классическая проблема,
> которую разрешить не все умеют, по этому и не проверяют.

А как её правильно разрешать? Научите.

Re: [devel] шутки про -Werror

[REAL]

Денис Смирнов пишет:
> R> Ну пока ещё ни разу не было причин обращать на эти сообщения внимания. 
> R> А что, были прецеденты? Хотелось бы ознакомиться, если так, и в чём 
> R> заключалась проблема.
> 
> К примеру многие не проверяют результат работы fwrite -- это почти
> гарантирует грабли в случае работы с NFS :)

Ну тут я ничего не могу сказать: с NFS ни разу не имел дело, ни прямо, 
ни косвенно.

> Дело даже не в прецедентах. Не проверять код возврата некоторых функций
> уже само по себе может гарантировать проблемы в некоторых ситуациях.

Это теория. Меня же интересуют (желательно, количеством больше 1) 
конретные примеры: когда апстрим игнорирует возвращаемое значение, а в 
ALT это дело патчат. Желательно, со ссылками на git.alt (коммиты я уж 
готов сам поискать).

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

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] Обновление libsoup (was: Re: mpdscribble-0.18.1-alt1: Sisyphus/i586 test rebuild failed)

[Slava Semushin]

2009/10/12 Slava Semushin <slava.semushin@gmail.com>:
> 2009/10/12 QA Team Robot <qa@altlinux.org>:
>> Package: mpdscribble-0.18.1-alt1
> [...]
>> + make -j1 --silent --no-print-directory
>> cc1: warnings being treated as errors
>> src/http_client_soup.c: In function 'http_client_request':
>> src/http_client_soup.c:140: error: cast discards qualifiers from pointer target type
>> src/http_client_soup.c:163: error: cast discards qualifiers from pointer target type
>> make[1]: *** [http_client_soup.o] Error 1
>
> Это нормально, что после обновления libsoup у меня сломалась сборка
> клиента (пусть даже и использующая -W -Wall -Werror)?

BTW, проблема всё ещё актуальна и проблема не в моём пакете, а в
libsoup. (В идеале хотелось бы чтобы это выявлялось ещё на стадии
сборки.)

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


-- 
Slava Semushin

Re: [devel] Обновление libsoup (was: Re: mpdscribble-0.18.1-alt1: Sisyphus/i586 test rebuild failed)

[Slava Semushin]

1 ноября 2009 г. 20:50 пользователь Slava Semushin
<slava.semushin@gmail.com> написал:
[...]
> В ближайшее время напишу об этом авторам libsoup.

=> https://bugzilla.gnome.org/show_bug.cgi?id=600315

-- 
Slava Semushin