[devel] I: g++ -Werror=return-type by default

[devel] I: g++ -Werror=return-type by default

[Dmitry V. Levin]

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

On Sat, Feb 09, 2019 at 09:24:33AM +0000, Girar Builder pender robot wrote:
> http://git.altlinux.org/tasks/archive/done/_215/221014/logs/events.1.3.log
> 
> 2019-Feb-09 08:18:45 :: task #221014 for sisyphus started by ldv:
> #100 build 8.2.1-alt4 from /people/ldv/packages/gcc8.git fetched at 2019-Feb-08 10:31:28
[...]
> 2019-Feb-09 08:39:33 :: gcc8: closes bugs: 36038 36045
[...]
> 2019-Feb-09 09:24:15 :: updated /gears/g/gcc8.git branch `sisyphus'
> 2019-Feb-09 09:24:31 :: gears update OK
> 2019-Feb-09 09:24:31 :: task #221014 for sisyphus DONE

В этой сборке g++ (не gcc, только g++) по умолчанию включено
-Werror=return-type; это было сделано после того, как стало ясно,
что, начиная с версии 8, g++ при включённой оптимизации
в следующих двух случаях генерит код, практически непригодный
для выполнения:
1. no return statement in function returning non-void
2. control reaches end of non-void function

Согласно стандарту C++, компилятор при обработке такого кода праве делать
всё что угодно, и g++ теперь буквально не делает возврата из таких
функций.

Прежние версии g++ вели себя более гуманно, реализуя поведение,
свойственное компиляторам C: программы работали нормально, когда
не использовали возвращаемое значение таких функций.

Таким образом, при использовании g++ >= 8 все такие функции следует
исправить, либо меняя тип возвращаемого значения на void, либо реализуя
возвращение значения во всех случаях.

Я ожидаю, что исправлению подлежат около 80 пакетов, более точно можно
будет сказать по результатам тестовой пересборки.

Подробнее о проблеме см. https://bugzilla.altlinux.org/36038


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: [devel] I: g++ -Werror=return-type by default

[Dmitry V. Levin]

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

On Mon, Feb 11, 2019 at 03:39:27AM +0300, Dmitry V. Levin wrote:
[...]
> Я ожидаю, что исправлению подлежат около 80 пакетов, более точно можно
> будет сказать по результатам тестовой пересборки.

Их оказалось 95:

AlephOne-1.0.1-alt1.2
ananas-0.9.5-alt10
ann-1.1.2-alt4
antico-deluxe-0.1.96-alt1.2
arpage-0.3.3-alt3_24
asylum-0.3.2-alt1_16
bloboats-1.0.2-alt2
bmcontrol-1.1-alt3
canorus-0.6svn-alt1.qa2
ceferino-0.97.8-alt2.qa1
CGenius-1.9.9.6beta-alt1
cooldown-24-alt1.1
csmash-0.6.6-alt3_32
ctorrent-3.3.2-alt1.1.qa1.1
desmume-0.9.11-alt2
duel3-0.1-alt3_0.24.20060225.qa1
extrema-4.4.5-alt2
fbg-0.9.1-alt3_12
fbpager-0.1.4-alt8.qa1
fceux-2.2.3-alt1
fontmatrix-0.9.99-alt1
freehdl-0.0.8-alt4
gav-0.8.0-alt4.qa2
glob2-0.9.4.4-alt1.qa7.1
glsl-optimizer-0.1.0-alt1.git20140820
gnome-peercast-0.5.4-alt3.qa2
imule-1.4.6-alt5
intde-1-alt2
ipsumdump-1.86-alt1
java-1.8.0-openjdk-0:1.8.0.151-alt1_5.b12jpp8
jvgs-0.5-alt2.1.qa1
karbowanecwallet-1.3.1-alt1
kbookocr-2.1.0-alt2
kismet-2014.02.R1-alt1
kodi-addon-inputstream-adaptive-18.0-alt1
kumir-1.9.1.2810-alt2
lib2geom-20081103-alt1.5
libfreeimage-3.18.0-alt1
libplotmm-0.1.2-alt2
libprojectM-2.1.0-alt12
libqaccessibilityclient-0.1.1-alt1
libv-1.90-alt9.2
lopsus-1.4.0.1-alt1.qa1
love-11.1-alt1_2
lprof-1.11.4.1-alt7.20100921
mapsoft-20180722-alt1
marsyas-0.6.0-alt1.alpha.git20150301.3
megafuse-1.0.0-alt3.3
meshlab-2016.12-alt3
milkytracker-1.02.00-alt2_1
mirall-1.0.2-alt5
mpip-3.3-alt6.svn20140313
netgen-6.2.1808-alt1
nted-1.10.18-alt3
numptyphysics-0.3.160-alt1
octave-optim-1:1.5.2-alt3
opendx-4.4.4-alt7
openexr-2.2.0-alt1.1
openscada-0.9.0-alt2
paris-traceroute-0.92-alt1
perl-Data-Array2ArrayMap-SDB-0.15-alt1.2
perl-JavaScript-V8-0.070-alt7.1
pfstools-2.1.0-alt3
php7-pdo_sqlsrv-7.2.14-alt1
picmicrosim-1.0-alt3
polyml-5.4.1-alt3.1
praat-6.0.46-alt1
printer-driver-splix-2.0.1-alt1.svn315
procbench-0.9.0a-alt3.1
pure-doc-0.3-alt2
python-module-islpy-2014.2-alt2.1.1.1
python-module-meshpy-2014.1-alt3.git20140706.1.1
qcat-0.5-alt5.1.qa1
qkismet-0.3.1-alt1.qa1
qorganizer-3.1.5-alt1
qstopmotion-2.3.2-alt2
qt-fsarchiver-0.8.4.0-alt1
qtrainer-0.5.2-alt2.qa2
ring-project-20180826-alt3
rosa-media-player-1.6.11-alt2
rxclient-1.0.0-alt1
scilab-6.0.1-alt3
scim-1.4.17-alt2
sfxr-sdl-1.1-alt1
sidplayfp-1.4.3-alt1
sidplay-libs-2.1.1-alt1_23
SimGear-2018.2.2-alt1
sprng-4.4-alt1
texlive-2018-alt2_7
verlihub-0.9.8e-alt2
verlihub-plugin-python-1.1-alt2.1.1.1
verlihub-plugins-0.1-alt2.qa2.1
verlihub-plugins-lua51-1.8.1-alt1.qa2
xevil-2.02r2-alt2
yaafe-0.64-alt2.git20130420


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: [devel] I: g++ -Werror=return-type by default

[Антон Мидюков]

11.02.2019 17:23, Dmitry V. Levin пишет:
> On Mon, Feb 11, 2019 at 03:39:27AM +0300, Dmitry V. Levin wrote:
> [...]
>> Я ожидаю, что исправлению подлежат около 80 пакетов, более точно можно
>> будет сказать по результатам тестовой пересборки.
> Их оказалось 95:
>
> meshlab-2016.12-alt3
Я его буду обновлять. Скорее всего в новой версии (бета) всё ок.
> openscada-0.9.0-alt2
Тут бы дождаться релиза 0.9.1. Его надо с qt5 собирать, а сизифный qt5 
пока не поддерживается. Поддержка gcc8 скорее всего тоже добавится. В 
крайнем случае, можно временно удалить, если будет мешать бранчеванию. 
Соберу, как выйдет 0.9.1 вновь.
> python-module-islpy-2014.2-alt2.1.1.1
> python-module-meshpy-2014.1-alt3.git20140706.1.1
Я занялся также обновлением этих двух крайне устаревших пакетов.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>

Re: [devel] I: g++ -Werror=return-type by default

[Sergey V Turchin]

On Monday, 11 February 2019 16:06:22 MSK Антон Мидюков wrote:

[...]
> Его надо с qt5 собирать, а сизифный qt5 пока не поддерживается.
Странно. Он почти не старый.

[...]

-- 
Regards, Sergey.

Re: [devel] I: g++ -Werror=return-type by default

[Антон Мидюков]

11.02.2019 20:20, Sergey V Turchin пишет:
> On Monday, 11 February 2019 16:06:22 MSK Антон Мидюков wrote:
>
> [...]
>> Его надо с qt5 собирать, а сизифный qt5 пока не поддерживается.
> Странно. Он почти не старый.
>
> [...]
>
qt5 в Сизифе для openscada слишком новый.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>

Re: [devel] I: g++ -Werror=return-type by default

[Sergey V Turchin]

On Monday, 11 February 2019 16:29:23 MSK Антон Мидюков wrote:
> 11.02.2019 20:20, Sergey V Turchin пишет:
> > On Monday, 11 February 2019 16:06:22 MSK Антон Мидюков wrote:
> > 
> > [...]
> > 
> >> Его надо с qt5 собирать, а сизифный qt5 пока не поддерживается.
> > 
> > Странно. Он почти не старый.
> > 
> > [...]
> 
> qt5 в Сизифе для openscada слишком новый.
А какой Qt5 поддерживается? Оно должно быть совместимо сверху вниз.

-- 
Regards, Sergey.

Re: [devel] I: g++ -Werror=return-type by default

[Антон Мидюков]

11.02.2019 21:00, Sergey V Turchin пишет:
> On Monday, 11 February 2019 16:29:23 MSK Антон Мидюков wrote:
>> 11.02.2019 20:20, Sergey V Turchin пишет:
>>> On Monday, 11 February 2019 16:06:22 MSK Антон Мидюков wrote:
>>>
>>> [...]
>>>
>>>> Его надо с qt5 собирать, а сизифный qt5 пока не поддерживается.
>>> Странно. Он почти не старый.
>>>
>>> [...]
>> qt5 в Сизифе для openscada слишком новый.
> А какой Qt5 поддерживается? Оно должно быть совместимо сверху вниз.
5.9 точно поддерживается.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>

[devel] openscada qt-5.11

[Sergey V Turchin]

On Monday, 11 February 2019 17:06:41 MSK Антон Мидюков wrote:
> 11.02.2019 21:00, Sergey V Turchin пишет:
> > On Monday, 11 February 2019 16:29:23 MSK Антон Мидюков wrote:
> >> 11.02.2019 20:20, Sergey V Turchin пишет:
> >>> On Monday, 11 February 2019 16:06:22 MSK Антон Мидюков wrote:
> >>> 
> >>> [...]
> >>> 
> >>>> Его надо с qt5 собирать, а сизифный qt5 пока не поддерживается.
> >>> 
> >>> Странно. Он почти не старый.
> >>> 
> >>> [...]
> >> 
> >> qt5 в Сизифе для openscada слишком новый.
> > 
> > А какой Qt5 поддерживается? Оно должно быть совместимо сверху вниз.
> 
> 5.9 точно поддерживается.
А что не так c 5.11?

-- 
Regards, Sergey.

Re: [devel] openscada qt-5.11

[Антон Мидюков]

11.02.2019 21:15, Sergey V Turchin пишет:
> On Monday, 11 February 2019 17:06:41 MSK Антон Мидюков wrote:
>> 11.02.2019 21:00, Sergey V Turchin пишет:
>>> On Monday, 11 February 2019 16:29:23 MSK Антон Мидюков wrote:
>>>> [...]
>>>> qt5 в Сизифе для openscada слишком новый.
>>> А какой Qt5 поддерживается? Оно должно быть совместимо сверху вниз.
>> 5.9 точно поддерживается.
> А что не так c 5.11?
http://git.altlinux.org/tasks/221272/logs/events.1.1.log

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>

Re: [devel] openscada qt-5.11

[Dmitry V. Levin]

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

On Mon, Feb 11, 2019 at 10:36:33PM +0700, Антон Мидюков wrote:
> 11.02.2019 21:15, Sergey V Turchin пишет:
> > On Monday, 11 February 2019 17:06:41 MSK Антон Мидюков wrote:
> >> 11.02.2019 21:00, Sergey V Turchin пишет:
> >>> On Monday, 11 February 2019 16:29:23 MSK Антон Мидюков wrote:
> >>>> [...]
> >>>> qt5 в Сизифе для openscada слишком новый.
> >>> А какой Qt5 поддерживается? Оно должно быть совместимо сверху вниз.
> >> 5.9 точно поддерживается.
> > А что не так c 5.11?
> http://git.altlinux.org/tasks/221272/logs/events.1.1.log

А какое отношение имеет qt5 к ошибкам
no return statement in function returning non-void
в пакете openscada?


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: [devel] openscada qt-5.11

[Антон Мидюков]

11.02.2019 23:40, Dmitry V. Levin пишет:
> On Mon, Feb 11, 2019 at 10:36:33PM +0700, Антон Мидюков wrote:
>> 11.02.2019 21:15, Sergey V Turchin пишет:
>>> On Monday, 11 February 2019 17:06:41 MSK Антон Мидюков wrote:
>>>> 11.02.2019 21:00, Sergey V Turchin пишет:
>>>>> On Monday, 11 February 2019 16:29:23 MSK Антон Мидюков wrote:
>>>>>> [...]
>>>>>> qt5 в Сизифе для openscada слишком новый.
>>>>> А какой Qt5 поддерживается? Оно должно быть совместимо сверху вниз.
>>>> 5.9 точно поддерживается.
>>> А что не так c 5.11?
>> http://git.altlinux.org/tasks/221272/logs/events.1.1.log
> А какое отношение имеет qt5 к ошибкам
> no return statement in function returning non-void
> в пакете openscada?
Ой. Туплю. Сейчас до того места не соберётся.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>

Re: [devel] openscada qt-5.11

[Антон Мидюков]

11.02.2019 23:56, Антон Мидюков пишет:
> 11.02.2019 23:40, Dmitry V. Levin пишет:
>> On Mon, Feb 11, 2019 at 10:36:33PM +0700, Антон Мидюков wrote:
>>> 11.02.2019 21:15, Sergey V Turchin пишет:
>>>> On Monday, 11 February 2019 17:06:41 MSK Антон Мидюков wrote:
>>>>> 11.02.2019 21:00, Sergey V Turchin пишет:
>>>>>> On Monday, 11 February 2019 16:29:23 MSK Антон Мидюков wrote:
>>>>>>> [...]
>>>>>>> qt5 в Сизифе для openscada слишком новый.
>>>>>> А какой Qt5 поддерживается? Оно должно быть совместимо сверху вниз.
>>>>> 5.9 точно поддерживается.
>>>> А что не так c 5.11?
>>> http://git.altlinux.org/tasks/221272/logs/events.1.1.log
>> А какое отношение имеет qt5 к ошибкам
>> no return statement in function returning non-void
>> в пакете openscada?
> Ой. Туплю. Сейчас до того места не соберётся.
>
Вот теперь собирается до того места:

http://git.altlinux.org/tasks/221272/logs/events.4.1.log

Возможно, qt5 лишь как-то косвенно причастен.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>

Re: [devel] openscada qt-5.11

[Vladislav Zavjalov]

On Mon, Feb 18, 2019 at 02:12:16PM +0700, Антон Мидюков wrote:
> 11.02.2019 23:56, Антон Мидюков пишет:
> > 11.02.2019 23:40, Dmitry V. Levin пишет:
> >> On Mon, Feb 11, 2019 at 10:36:33PM +0700, Антон Мидюков wrote:
> >>> 11.02.2019 21:15, Sergey V Turchin пишет:
> >>>> On Monday, 11 February 2019 17:06:41 MSK Антон Мидюков wrote:
> >>>>> 11.02.2019 21:00, Sergey V Turchin пишет:
> >>>>>> On Monday, 11 February 2019 16:29:23 MSK Антон Мидюков wrote:
> >>>>>>> [...]
> >>>>>>> qt5 в Сизифе для openscada слишком новый.
> >>>>>> А какой Qt5 поддерживается? Оно должно быть совместимо сверху вниз.
> >>>>> 5.9 точно поддерживается.
> >>>> А что не так c 5.11?
> >>> http://git.altlinux.org/tasks/221272/logs/events.1.1.log
> >> А какое отношение имеет qt5 к ошибкам
> >> no return statement in function returning non-void
> >> в пакете openscada?
> > Ой. Туплю. Сейчас до того места не соберётся.
> >
> Вот теперь собирается до того места:
> 
> http://git.altlinux.org/tasks/221272/logs/events.4.1.log
> 
> Возможно, qt5 лишь как-то косвенно причастен.

Похоже, что в openscada/src/tmess.h:48 определяется макро message,
и это ломает вызов __ec.message() в /usr/include/c++/8/future

Re: [devel] openscada qt-5.11

[Vladislav Zavjalov]

On Mon, Feb 18, 2019 at 10:38:30AM +0300, Vladislav Zavjalov wrote:
> On Mon, Feb 18, 2019 at 02:12:16PM +0700, Антон Мидюков wrote:
> > Вот теперь собирается до того места:
> > 
> > http://git.altlinux.org/tasks/221272/logs/events.4.1.log
> > 
> > Возможно, qt5 лишь как-то косвенно причастен.
> 
> Похоже, что в openscada/src/tmess.h:48 определяется макро message,
> и это ломает вызов __ec.message() в /usr/include/c++/8/future

Исправить можно, например, переставив #include в файлах
  openscada/src/moduls/ui/QTCfg/qtcfg.cpp
  openscada/src/moduls/ui/QTCfg/tuimod.cpp
так чтобы "qtcfg.h" шел перед <tmess.h>

Re: [devel] openscada qt-5.11

[Антон Мидюков]

18.02.2019 14:59, Vladislav Zavjalov пишет:
> On Mon, Feb 18, 2019 at 10:38:30AM +0300, Vladislav Zavjalov wrote:
>> On Mon, Feb 18, 2019 at 02:12:16PM +0700, Антон Мидюков wrote:
>>> Вот теперь собирается до того места:
>>>
>>> http://git.altlinux.org/tasks/221272/logs/events.4.1.log
>>>
>>> Возможно, qt5 лишь как-то косвенно причастен.
>> Похоже, что в openscada/src/tmess.h:48 определяется макро message,
>> и это ломает вызов __ec.message() в /usr/include/c++/8/future
> Исправить можно, например, переставив #include в файлах
>    openscada/src/moduls/ui/QTCfg/qtcfg.cpp
>    openscada/src/moduls/ui/QTCfg/tuimod.cpp
> так чтобы "qtcfg.h" шел перед <tmess.h>
Спасибо, попробую.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>