Bug 40751 - Erroneous use of %remove_optflags
Summary: Erroneous use of %remove_optflags
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: qt5-webkit-common (show other bugs)
Version: unstable
Hardware: all Linux
: P5 normal
Assignee: Sergey V Turchin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-16 17:56 MSK by Dmitry V. Levin
Modified: 2021-08-17 13:24 MSK (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dmitry V. Levin 2021-08-16 17:56:49 MSK
В qtwebkit.spec есть такая конструкция:

%remove_optflags '-g'
%ifarch %e2k
# because of this error on linking:
# "relocation truncated to fit: R_E2K_32_ABS"
%add_optflags -g0 -fpermissive
%else
%add_optflags -g1 -fpermissive
%endif

Не вдаваясь здесь в осмысленность этого действия, обращаю внимание, что
она работает неправильно, когда в %optflags появляются ещё какие-нибудь
параметры, начинающиеся на -g, мы это увидели, например, тут:
/beehive/logs/icarus/x86_64/latest/error/qt5-webkit-5.212.0-alt21
Другими словами, пакет скоро может стать FTBFS.

При этом -g в %optflags берётся из макроса %optflags_debug,
так что гораздо надёжнее было бы просто переопределить его.

Например, в llvm12.spec есть такие строки:

# Decrease debuginfo verbosity to reduce memory consumption during final library linking
%ifarch %ix86 %arm
%define optflags_debug -g0
%else
%define optflags_debug -g1
%endif
Comment 1 Sergey V Turchin 2021-08-17 10:57:06 MSK
Хотя, проще будет самому сделать.
Comment 2 Sergey V Turchin 2021-08-17 11:34:44 MSK
> При этом -g в %optflags берётся из макроса %optflags_debug,
> так что гораздо надёжнее было бы просто переопределить его.
http://git.altlinux.org/tasks/283022/build/100/armh/log
Не работает.
Comment 3 Dmitry V. Levin 2021-08-17 12:12:41 MSK
(In reply to Sergey V Turchin from comment #2)
> > При этом -g в %optflags берётся из макроса %optflags_debug,
> > так что гораздо надёжнее было бы просто переопределить его.
> http://git.altlinux.org/tasks/283022/build/100/armh/log
> Не работает.

%optflags_debug имеет смысл менять только до первого %add_optflags.
Comment 4 Repository Robot 2021-08-17 13:24:09 MSK
qt5-webkit-5.212.0-alt23 -> sisyphus:

 Tue Aug 17 2021 Sergey V Turchin <zerg@altlinux> 5.212.0-alt23
 - define optflags_debug when build (closes: 40751)