Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 590920 - dev-qt/qtwebengine: build log not verbose
Summary: dev-qt/qtwebengine: build log not verbose
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: verbose_build
  Show dependency tree
 
Reported: 2016-08-10 10:58 UTC by .
Modified: 2017-06-04 13:20 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
qtwebengine-5.7.0-ninja-verbose.patch (qtwebengine-5.7.0-ninja-verbose.patch,559 bytes, patch)
2016-08-24 04:03 UTC, Mark Wright
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description . 2016-08-10 10:58:49 UTC
example:

```
make[3]: Entering directory '/tmp/portage/dev-qt/qtwebengine-5.6.1/work/qtwebengine-opensource-src-5.6.1/src/core'
/usr/bin/ninja  -C /tmp/portage/dev-qt/qtwebengine-5.6.1/work/qtwebengine-opensource-src-5.6.1/src/core/Release
ninja: Entering directory `/tmp/portage/dev-qt/qtwebengine-5.6.1/work/qtwebengine-opensource-src-5.6.1/src/core/Release'
[79/10936] CXX obj/src/3rdparty/chromium/base/base_static.base_switches.o
<command-line>:0:0: warning: "_FORTIFY_SOURCE" redefined
<built-in>: note: this is the location of the previous definition
[329/10936] CXX obj/src/3rdparty/chromium/third_party/webrtc/modules/audio_coding/codecs/audio_decoder_interface.audio_decoder.o
<command-line>:0:0: warning: "_FORTIFY_SOURCE" redefined
<built-in>: note: this is the location of the previous definition
[411/10936] COPY /tmp/portage/dev-qt/qtwebengine-5.6.1/work/qtwebengine-opensource-src-5.6.1/src/3rdparty/chromium/third_party/widevine/cdm/widevine_cdm_version.h gen[411/10936] COPY /tmp/portage/dev-qt/qtwebengine-5.6.1/work/qtwebengine-opensource-src-5.6.1/src/3rdparty/chromium/third_party/widevine/cdm/widevine_cdm_version.h gen[486/10936] CXX obj/src/3rdparty/chromium/third_party/webrtc/system_wrappers/source/system_wrappers.aligned_malloc.o
<command-line>:0:0: warning: "_FORTIFY_SOURCE" redefined
<built-in>: note: this is the location of the previous definition
[487/10936] CXX obj/src/3rdparty/chromium/third_party/webrtc/webrtc_common.common_types.o
<command-line>:0:0: warning: "_FORTIFY_SOURCE" redefined
<built-in>: note: this is the location of the previous definition
[489/10936] CXX obj/src/3rdparty/chromium/third_party/webrtc/webrtc_common.config.o
<command-line>:0:0: warning: "_FORTIFY_SOURCE" redefined
<built-in>: note: this is the location of the previous definition
[506/10936] CXX obj/src/3rdparty/chromium/components/scheduler/common/scheduler_common.scheduler_switches.o
<command-line>:0:0: warning: "_FORTIFY_SOURCE" redefined
```
Comment 1 Mark Wright gentoo-dev 2016-08-24 04:03:07 UTC
Created attachment 444006 [details, diff]
qtwebengine-5.7.0-ninja-verbose.patch

Proposed patch to add the ninja -v option to turn on verbose build.

It could be added to the ebuild like:

PATCHES=(
	"${FILESDIR}/${PN}-5.7.0-ninja-verbose.patch"
)

Or you can create a directory:

mkdir -p /etc/portage/patches/dev-qt/qtwebengine-5.7.0

and place the patch in the above directory.
Comment 2 . 2016-08-27 02:46:38 UTC
Why don't you just use NINJAFLAGS Makefile variable instead of patching?

Also it'd be better if it supported all NINJAOPTS as exported from cmake-utils.eclass. This would require to create a new ninja.eclass though.
Comment 3 Michael Palimaka (kensington) gentoo-dev 2017-06-04 11:43:01 UTC
In overlay:
https://gitweb.gentoo.org/proj/qt.git/commit/?id=cfbb33fabae2dd1ced8dbf4d83dfad1068571272
Comment 4 Michael Palimaka (kensington) gentoo-dev 2017-06-04 13:20:54 UTC
Thanks, fixed in git.

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=90f45d0172ad22d6b8a6e77025499156b704dcc9