Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 1358918 (qtwebbrowser)

Summary:	Review Request: qtwebbrowser - QtWebEngine based browser for embedded touch devices
Product:	[Fedora] Fedora	Reporter:	Raphael Groner <projects.rg>
Component:	Package Review	Assignee:	Nobody's working on this, feel free to take it <nobody>
Status:	CLOSED NOTABUG	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	package-review, rdieter
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-12-07 22:24:24 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	928937

Description Raphael Groner 2016-07-21 18:49:29 UTC

Spec URL: https://raphgro.fedorapeople.org/review/qt/qtwebbrowser/qtwebbrowser.spec
SRPM URL: https://raphgro.fedorapeople.org/review/qt/qtwebbrowser/qtwebbrowser-1.0-1.fc24.src.rpm
Description: QtWebEngine based browser for embedded touch devices
The Qt WebBrowser (codename Roadtrip) is a browser
for embedded devices developed using the capabilities
of Qt and Qt WebEngine. Using recent Chromium, it
features up-to-date HTML technologies behind a minimal
but slick touch-friendly user interface from Qt Quick.

Fedora Account System Username: raphgro

Test builds:
https://copr.fedorainfracloud.org/coprs/raphgro/review/package/qtwebbrowser

Note: Upload to koji for scratch builds is slow from here, so I use copr.

Comment 1 Raphael Groner 2016-07-21 18:56:58 UTC

DEBUG util.py:421:  No matching package to install: 'pkgconfig(Qt5WebEngine)'

… for epel7 and ppc64le (all Fedoras)

Maybe add ExcludeArch?

Comment 2 Kevin Kofler 2016-07-21 23:10:40 UTC

You should copy the ExclusiveArch line from qt5-qtwebengine.

Comment 3 Raphael Groner 2016-07-22 09:27:24 UTC

(In reply to Kevin Kofler from comment #2)
> You should copy the ExclusiveArch line from qt5-qtwebengine.

See https://bugzilla.redhat.com/show_bug.cgi?id=1298011#c7 .

Comment 4 Rex Dieter 2016-08-01 14:56:26 UTC

That may be a good long-term goal.  But in the short-term, I'd recommend you follow Kevin's advice and copy the ExclusiveArch line from qt5-qtwebengine.spec

Comment 5 Rex Dieter 2016-08-16 14:09:09 UTC

I can help review this today...

Comment 6 Raphael Groner 2016-10-21 15:05:16 UTC

(In reply to Raphael Groner from comment #1)
> DEBUG util.py:421:  No matching package to install: 'pkgconfig(Qt5WebEngine)'
> 
> … for epel7 and ppc64le (all Fedoras)
> 
> Maybe add ExcludeArch?

(In reply to Kevin Kofler from comment #2)
> You should copy the ExclusiveArch line from qt5-qtwebengine.

It's fixed for qt5-qtwebengine (see rhbz#1298011). I assume ExcludeArch is not needed any more for qtwebbrowser, too.

Comment 7 Raphael Groner 2016-10-21 15:07:02 UTC

https://bugzilla.redhat.com/show_bug.cgi?id=1298011#c10

Comment 8 Rex Dieter 2016-10-21 15:19:36 UTC

sorry for the delay, I won't have time in the immediate future (removing myself as reviewer to allow someone else to take it for now)

Comment 9 Raphael Groner 2016-10-29 13:45:24 UTC

I'm confused. Is ExclusiveArch needed or not? There's now a patch for qtwebengine introducing a macro for ExclusiveArch (as I originally suggested).

Comment 10 Rex Dieter 2016-10-31 11:53:05 UTC

Yes, the macro simply makes using ExclusiveArch easier

Comment 11 Raphael Groner 2016-12-07 22:24:24 UTC

Lost interest in this package. Feel free to use this request as a base for another review.