-
Notifications
You must be signed in to change notification settings - Fork 186
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
chromium: Backport missing dependency in NewTabPage #815
chromium: Backport missing dependency in NewTabPage #815
Conversation
Dependency is missing in NewTabPage module. This can cause random build errors. Error occurrence depends on the order in which packages were build. It seems to occur more frequently with lower BB_NUMBER_THREADS values. Signed-off-by: Daniel Semkowicz <dse@thaumatec.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks!
If that's ok for you, I can create a PR for master once #806 is merged. |
That's fine for me. Thank you. |
Cherry-pick of OSSystems#815. Dependency is missing in NewTabPage module. This can cause random build errors. Error occurrence depends on the order in which packages were build. It seems to occur more frequently with lower BB_NUMBER_THREADS values. Signed-off-by: Daniel Semkowicz <dse@thaumatec.com>
Cherry-pick of OSSystems#815. Build and patch changes: ------------------------ Add one backported patch to fix a build race condition that would sometimes lead to build errors. License changes: ---------------- Added licenses: none. Removed licenses: none. Updated licenses: none. Test-built: ----------- * chromium-x11: - scarthgap, clang, MACHINE=qemux86-64 Signed-off-by: Max Ihlenfeldt <max@igalia.com>
Cherry-pick of #815. Build and patch changes: ------------------------ Add one backported patch to fix a build race condition that would sometimes lead to build errors. License changes: ---------------- Added licenses: none. Removed licenses: none. Updated licenses: none. Test-built: ----------- * chromium-x11: - scarthgap, clang, MACHINE=qemux86-64 Signed-off-by: Max Ihlenfeldt <max@igalia.com> Co-authored-by: Daniel Semkowicz <158140967+dsemkowicz-tt@users.noreply.github.com>
Dependency is missing in NewTabPage module. This can cause random build errors. Error occurrence depends on the order in which packages were build. It seems to occur more frequently with lower BB_NUMBER_THREADS values.
Build failed in my case usually with following error:
This issue most likely occurs also on master (Chromium 122 at the moment of writing), because fix was merged in version 125. I experienced this problem only on kirkstone branch and didn't test the master. If you think this fix is also beneficial on master, I can do a rebase on 122 and push the second MR.