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.

Sign up for GitHub

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

Jump to bottom

qt6-webengine: 6.8.0 [WIP] #21914

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

qt6-webengine: 6.8.0 [WIP] #21914

wants to merge 1 commit into from

Conversation

raedrizqie
Copy link
Contributor

mingw64 and ucrt64 will probably timed-out

@raedrizqie raedrizqie marked this pull request as ready for review September 13, 2024 15:56
@ZachBacon
Copy link
Contributor

As per what @raedrizqie said the other two did time out. I tested the clang64 build against rssguard. It builds, but seems to segfault. I'm gonna try and run it through lldb and see what's up. But in terms of linking webengine to apps that use webengine. It at least works.

@ZachBacon
Copy link
Contributor

ZachBacon commented Sep 13, 2024 •
edited
Loading

Update on the lldb stuff, here's the backtrace. I'll try and rerun it with the debug packages later.

* thread #20, name = 'Chrome_IOThread', stop reason = Exception 0x80000003 encountered at address 0x7ffd3a07ca32
  * frame #0: 0x00007ffd3a07ca33 ntdll.dll`RtlIsZeroMemory + 163
    frame #1: 0x00007ffd3a085b0a ntdll.dll`__misaligned_access + 1066
    frame #2: 0x00007ffd3a085dea ntdll.dll`__misaligned_access + 1802
    frame #3: 0x00007ffd3a091de5 ntdll.dll`__misaligned_access + 50949
    frame #4: 0x00007ffd39fac2dd ntdll.dll`RtlGetCurrentServiceSessionId + 4685
    frame #5: 0x00007ffd39faaff1 ntdll.dll`RtlFreeHeap + 81
    frame #6: 0x00007ffd3798364b ucrtbase.dll`_free_base + 27
    frame #7: 0x00007ffc121ab062 Qt6WebEngineCore.dll`base::TaskAnnotator::RunTaskImpl(base::PendingTask&) + 370
    frame #8: 0x00007ffc121c7571 Qt6WebEngineCore.dll`base::sequence_manager::internal::ThreadControllerWithMessagePumpImpl::DoWorkImpl(base::LazyNow*) + 817
    frame #9: 0x00007ffc121c70bb Qt6WebEngineCore.dll`base::sequence_manager::internal::ThreadControllerWithMessagePumpImpl::DoWork() + 107
    frame #10: 0x00007ffc121c7ae8 Qt6WebEngineCore.dll`non-virtual thunk to base::sequence_manager::internal::ThreadControllerWithMessagePumpImpl::DoWork() + 24
    frame #11: 0x00007ffc12225b90 Qt6WebEngineCore.dll`base::MessagePumpForIO::DoRunLoop() + 96
    frame #12: 0x00007ffc12223d72 Qt6WebEngineCore.dll`base::MessagePumpWin::Run(base::MessagePump::Delegate*) + 50
    frame #13: 0x00007ffc121c7ecb Qt6WebEngineCore.dll`base::sequence_manager::internal::ThreadControllerWithMessagePumpImpl::Run(bool, base::TimeDelta) + 267
    frame #14: 0x00007ffc121844c4 Qt6WebEngineCore.dll`base::RunLoop::Run(base::Location const&) + 388
    frame #15: 0x00007ffc121f0b89 Qt6WebEngineCore.dll`base::Thread::Run(base::RunLoop*) + 57
    frame #16: 0x00007ffc102fa2a3 Qt6WebEngineCore.dll`content::BrowserProcessIOThread::IOThreadRun(base::RunLoop*) + 83
    frame #17: 0x00007ffc121f0d1a Qt6WebEngineCore.dll`base::Thread::ThreadMain() + 362
    frame #18: 0x00007ffc1223c2c5 Qt6WebEngineCore.dll`base::win::AllowDarkModeForWindow(HWND__*, bool) + 437
    frame #19: 0x00007ffd397d257d kernel32.dll`BaseThreadInitThunk + 29
    frame #20: 0x00007ffd39fcaf28 ntdll.dll`RtlUserThreadStart + 40

@raedrizqie
Copy link
Contributor Author

thanks for testing it @ZachBacon

@ZachBacon
Copy link
Contributor

thanks for testing it @ZachBacon

No worries, in interested in this as well.

@ZachBacon
Copy link
Contributor

the checksum for your modified patch caused it to fail to build. It'll need updating in the PKGBUILD when you get the chance

@ZachBacon
Copy link
Contributor

Flex should be added to the depends list since qtwebengine won't be built without it.

@ZachBacon
Copy link
Contributor

Another thing I noticed, it'll conflict with qtpdf as well

@raedrizqie
Copy link
Contributor Author

the checksum for your modified patch caused it to fail to build. It'll need updating in the PKGBUILD when you get the chance

new version (6.7.3) was released.. i need to rebased all patches

@raedrizqie
Copy link
Contributor Author

Another thing I noticed, it'll conflict with qtpdf as well

qtpdf is disabled for this package, so it should not be in conflict i think..

@ZachBacon
Copy link
Contributor

Another thing I noticed, it'll conflict with qtpdf as well

qtpdf is disabled for this package, so it should not be in conflict i think..

I remember when it was configuring it was showing that it was enabled.

@ZachBacon
Copy link
Contributor

Hmm, perhaps I missed it the first time, I'm at the moment rebuilding it against your latest change after I updated the sha checksum and added flex

@raedrizqie
Copy link
Contributor Author

Another thing I noticed, it'll conflict with qtpdf as well

yes, you were right..

error: failed to commit transaction (conflicting files)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindGn.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindGPerf.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindNinja.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindNodejs.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindPkgConfigHost.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindSnappy.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6BuildInternals/StandaloneTests/QtWebEngineTestsConfig.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/share/qt6/bin/gn.exe exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
Errors occurred, no packages were upgraded.

so, either fix qt6-pdf or mark this package as conflict

@ZachBacon
Copy link
Contributor

Another thing I noticed, it'll conflict with qtpdf as well

yes, you were right..

error: failed to commit transaction (conflicting files)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindGn.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindGPerf.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindNinja.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindNodejs.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindPkgConfigHost.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6/FindSnappy.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/lib/cmake/Qt6BuildInternals/StandaloneTests/QtWebEngineTestsConfig.cmake exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
mingw-w64-clang-x86_64-qt6-webengine: /clang64/share/qt6/bin/gn.exe exists in filesystem (owned by mingw-w64-clang-x86_64-qt6-pdf)
Errors occurred, no packages were upgraded.

so, either fix qt6-pdf or mark this package as conflict

I mean... Couldn't we just have this replace qtpdf once it's stable enough?

@raedrizqie
Copy link
Contributor Author

I mean... Couldn't we just have this replace qtpdf once it's stable enough?

yes we could.. but i think the build process will hit timeout (over 6 hours)

@ZachBacon
Copy link
Contributor

I mean... Couldn't we just have this replace qtpdf once it's stable enough?

yes we could.. but i think the build process will hit timeout (over 6 hours)

Hmmm, and I haven't been able to build it locally either, ninja keeps trying to clean things and it fails.

@raedrizqie
Copy link
Contributor Author

maybe because you have gn installed?

@ZachBacon
Copy link
Contributor

maybe because you have gn installed?

It shouldn't. I'll have to double check, on my way home from work

@realysy realysy mentioned this pull request Oct 6, 2024
Open
@Kreijstal
Copy link
Contributor

question mark, if you enable sccache, and just keep retrying builds... it will go below 6 hours on the second build, right?

@adeas31
Copy link

adeas31 commented Dec 3, 2024

Any update on this?

@Kreijstal
Copy link
Contributor

question mark, if you enable sccache, and just keep retrying builds... it will go below 6 hours on the second build, right?

Alternatively not sure how hard is it to split the package in 2... if anything

@raedrizqie raedrizqie changed the title qt6-webengine: 6.7.2 qt6-webengine: 6.8.0 [WIP] Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@raedrizqie @ZachBacon @Kreijstal @adeas31