Bug 40751

Summary: Erroneous use of %remove_optflags
Product: Sisyphus Reporter: Dmitry V. Levin <ldv>
Component: qt5-webkit-commonAssignee: Sergey V Turchin <zerg>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P5 CC: ilyakurdyukov, mike, zerg
Version: unstable   
Hardware: all   
OS: Linux   

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)