Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QtWebEngine: new recipe #3040

Closed
wants to merge 3 commits into from
Closed

QtWebEngine: new recipe #3040

wants to merge 3 commits into from

Conversation

kenmays
Copy link
Contributor

@kenmays kenmays commented Sep 11, 2018

The Qt WebEngine submodule library to render web content. Qt WebEngine integrates Google Chromium's fast moving web capabilities into Qt.

Help wanted: Haiku integration of Qt WebEngine.
See: https://wiki.qt.io/QtWebEngine/Porting_from_QtWebKit

Ref: #2512
Support: https://wiki.qt.io/QtWebEngine

NOTE: W-I-P (no merge), help wanted

@kenmays kenmays changed the title QtWebEngine: Haiku integration needed QtWebEngine: new recipe - Haiku integration needed Sep 11, 2018
@kenmays kenmays changed the title QtWebEngine: new recipe - Haiku integration needed QtWebEngine: new recipe Sep 11, 2018
@extrowerk
Copy link
Member

extrowerk commented Sep 11, 2018

Can you please provide a buildlog to get a ~ feeling about it? To see, whats missing, where it breaks.
Maybe we can give you some hints during your journey.

@extrowerk
Copy link
Member

extrowerk commented Sep 12, 2018

Please do not add unrelated patches. If they arent about Haiku support, they shouldn’t be here at this stage. I mean, let us get it work first, then and only then should we start to pimp it.

@KapiX
Copy link
Member

KapiX commented Sep 13, 2018

And don't edit them by hand. They will likely not apply and the next person is left wondering why there is a problem.

My experiments: https://gist.github.com/KapiX/8502596ef66813195cf8d2bec7393cd7

@extrowerk
Copy link
Member

@kenmays: can you please incorporate the patches from KapiX and the cosmetic fixes from alaviss?
That would spare plenty time for the peope who are interested in this WIP recipe.

@KapiX
Copy link
Member

KapiX commented Sep 22, 2018

I have played with it a bit more and got some stuff to compile, but without various ports it's not feasible.
breakpad is the most important I think, since it's hard to rip out (if not impossible).

Will update the gist soon.

@waddlesplash
Copy link
Member

Why? AFAICT it's just used for automated crash reporting and the like; we should be able to just disable that stuff.

@KapiX
Copy link
Member

KapiX commented Sep 22, 2018

Well, we're not. I certainly didn't see any switches to disable it. If it's not enabled (and it is not by default) other targets complain about missing dependencies.

@KapiX
Copy link
Member

KapiX commented Sep 23, 2018

I stand corrected. I have managed to disable it. Updated gist makes it build up to QtWebEngineCore linking, but there are a bunch of unresolved references.

@return
Copy link
Contributor

return commented Oct 18, 2018

Is there any update on this? QtWebEngine is a required component for several Qt based browsers such as Falkon, Otter Browser, etc.

@extrowerk The source branch for this PR has been deleted (says unknown repository). So I don't think that the fixes can updated by the author and we might need a new PR for this one with the fixes included.

@extrowerk
Copy link
Member

@return : then we should close this. Please, guys, use the #2512 page then.

@extrowerk extrowerk closed this Oct 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

7 participants