-
Notifications
You must be signed in to change notification settings - Fork 866
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
Update to Chromium 132.0.6834.83 #3143
Conversation
I had to add a pylint exclusion section due to the length of the filename added in prune_binaries.py. Edit: There's a few paths that we shouldn't prune. If I get some time in the future I'll see if we can carve out exceptions for some files when pruning contingent paths. |
FYI, I just landed another related change. It should now be possible to build u-c using the lite tarball. I checked that the Windows build works with it (in my cross build, anyway), as well as the regular Debian build. If you have the time, please give it a try, and let me know if anything is missing.
I take it the (I'm preparing a change to tighten up that pruning code, so would not advise copying it directly) |
Yeah, contingent paths are completely removed. I forgot that I had set up file filtering in the clone script to deal with the test directories, so I've moved them there and added the paths removed from the lite tarball in their place. I'd like to eventually update how we prune those paths to account for that, consolidate the removal logic into a single location, and maybe make a new list for the paths instead of keeping them in the pruning script. The good news is that I was able to successfully build with the lite tarball! There was a section in the trk and unused prefs patch that needed to be removed since those files are now pruned. I'd like to eventually update our Cirrus container to use the lite tarball in order to keep it's size down but I'll wait to be sure there aren't any unforeseen issues with that. Also after looking at the changes to the lists I realized that there was a bug in update_lists.py that would prune paths that didn't need to be pruned. Before I moved |
The updated pruning script seems to break the windows build process:
I wonder why the I am not sure if we should not prune the |
Interesting, I guess that path will need to wait until I can update how we prune them. I just pushed a change with the ios path removed from the list. Hopefully that's the only one that's needed. |
I ran into some linking errors:
I fixed the first one with the following patch:
The following patch fixes the other linking errors:
If no one else faces the same errors I would simply add those patches to the windows safebrowing patch. |
I think it's reasonable to add that to core since there doesn't seem to be any platform-specific gating for those sections. |
on portable linux, i get the following error while generating build files with gn:
The file |
'chrome/android' is pruned now. |
@teeminus thanks, found it in maybe a stupid question, but why does that break my build but seemingly no others? Does anybody have an idea why this might be? |
There may be differences in the buildflags used or some platform-specific differences. |
portable linux builds and runs fine now |
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.
Thanks a lot, looking good on NixOS
Great to see the tarballs improvements upstream!
FYI, my change to the upstream pruning code went in. Relevant bits are here and here. Note that the code explicitly avoids deleting As for why stuff in |
Builds and runs fine on windows, too. |
The build on macOS failed. First, when building, the builder was looking for the Rust bindgen binary in the Second, this error is happening, and I am not sure what is causing it: [50718/50916] SOLINK 'obj/chrome/chrome_framework_shared_library/Chromium Framework' 'obj/chrome/chrome_framework_shared_library/Chromium Framework.TOC' 'Chromium Framework.dSYM/Contents/Info.plist' 'Chromium Framework.dSYM/Contents/Resources/DWARF/Chromium Framework'
FAILED: obj/chrome/chrome_framework_shared_library/Chromium Framework obj/chrome/chrome_framework_shared_library/Chromium Framework.TOC Chromium Framework.dSYM/Contents/Info.plist Chromium Framework.dSYM/Contents/Resources/DWARF/Chromium Framework
if [ ! -e "obj/chrome/chrome_framework_shared_library/Chromium Framework" -o ! -e "obj/chrome/chrome_framework_shared_library/Chromium Framework.TOC" ] || /Applications/Xcode-16.2.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/otool -l "obj/chrome/chrome_framework_shared_library/Chromium Framework" | grep -q LC_REEXPORT_DYLIB ; then TOOL_VERSION=1737235440 ../../build/toolchain/apple/linker_driver.py -Wcrl,driver,../../third_party/llvm-build/Release+Asserts/bin/clang++ -Wcrl,strippath,/Applications/Xcode-16.2.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/strip -Wcrl,installnametoolpath,../../third_party/llvm-build/Release+Asserts/bin/llvm-install-name-tool -shared -Wcrl,dsym,. -Wcrl,dsymutilpath,/usr/bin/dsymutil -compatibility_version 6834.0.83 -current_version 6834.0.83 -Wl,-install_name,@executable_path/../Frameworks/Chromium\ Framework.framework/Versions/132.0.6834.83/Chromium\ Framework -Wl,--no-warn-duplicate-rpath -Wl,-no_warn_duplicate_libraries -fuse-ld=lld -Wl,--icf=all -Wl,--color-diagnostics -Wl,-mllvm,-instcombine-lower-dbg-declare=0 -Wl,-mllvm,-split-threshold-for-reg-with-hint=0 -Wl,-cache_path_lto,thinlto-cache -Wl,--thinlto-cache-policy=cache_size=10\%:cache_size_bytes=40g:cache_size_files=100000 -flto=thin -Wl,--thinlto-jobs=all -Wl,-mllvm,-import-instr-limit=30 -Wcrl,object_path_lto -Wl,-mllvm,-disable-auto-upgrade-debug-info -Wl,-mllvm,-inlinehint-threshold=360 -fwhole-program-vtables -Wl,--strict-auto-link -Wl,--no-call-graph-profile-sort --target=arm64-apple-macos -Wl,-mllvm,-enable-machine-outliner=never -no-canonical-prefixes -Wl,-dead_strip -Wl,-no_data_in_code_info -Wl,-no_function_starts -nostdlib++ -isysroot ../../../../../../../Applications/Xcode-16.2.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX15.2.sdk -mmacos-version-min=11.0 -Wl,-ObjC -Wcrl,strip,-x,-S -Wl,--lto-O2 -Wl,-order_file -Wl,../../chrome/app/framework.order -Wl,-exported_symbols_list -Wl,../../chrome/app/framework.exports prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libstd.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/liballoc.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libcfg_if.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libcompiler_builtins.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libcore.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libgetopts.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libhashbrown.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libpanic_abort.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libpanic_unwind.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/librustc_demangle.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libstd_detect.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libtest.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libunicode_width.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libunwind.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libaddr2line.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libadler.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libgimli.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/liblibc.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libmemchr.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libminiz_oxide.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libobject.rlib -L/Applications/Xcode-16.2.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX15.2.sdk/usr/lib -o "obj/chrome/chrome_framework_shared_library/Chromium Framework" "@obj/chrome/chrome_framework_shared_library/Chromium Framework.rsp" obj/third_party/rust/cxx/v1/lib/libcxx_lib.rlib obj/base/libbase_crust_ulogger.rlib obj/base/libbase_clogging_ulog_useverity_ubindgen.rlib obj/build/rust/chromium_prelude/libchromium.rlib obj/base/libbase_clogging_urust_ulog_uintegration_ubindgen.rlib obj/third_party/rust/log/v0_4/lib/liblog_lib.rlib obj/third_party/rust/serde_json_lenient/v0_2/wrapper/libthird_uparty_srust_sserde_ujson_ulenient_sv0_u2_swrapper_cwrapper.rlib obj/third_party/rust/serde/v1/lib/libserde_lib.rlib obj/third_party/rust/serde_json_lenient/v0_2/lib/libserde_json_lenient_lib.rlib obj/third_party/rust/itoa/v1/lib/libitoa_lib.rlib obj/third_party/rust/ryu/v1/lib/libryu_lib.rlib obj/skia/libskia_cbridge_urust_uside.rlib obj/third_party/rust/font_types/v0_7/lib/libfont_types_lib.rlib obj/third_party/rust/bytemuck/v1/lib/libbytemuck_lib.rlib obj/third_party/rust/read_fonts/v0_23/lib/libread_fonts_lib.rlib obj/third_party/rust/skrifa/v0_24/lib/libskrifa_lib.rlib obj/third_party/blink/common/rust_crash/libthird_uparty_sblink_scommon_srust_ucrash_crs.rlib obj/third_party/crabbyavif/libthird_uparty_scrabbyavif_ccrabbyavif.rlib obj/third_party/crabbyavif/libdav1d_sys.rlib obj/third_party/crabbyavif/liblibyuv_sys.rlib obj/third_party/rust/libc/v0_2/lib/liblibc_lib.rlib obj/components/qr_code_generator/libcomponents_sqr_ucode_ugenerator_cqr_ucode_ugenerator_uffi_uglue.rlib obj/third_party/rust/qr_code/v2/lib/libqr_code_lib.rlib && { /Applications/Xcode-16.2.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/otool -l "obj/chrome/chrome_framework_shared_library/Chromium Framework" | grep LC_ID_DYLIB -A 5; /Applications/Xcode-16.2.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/nm -gPp "obj/chrome/chrome_framework_shared_library/Chromium Framework" | cut -f1-2 -d' ' | grep -v U$$; true; } > "obj/chrome/chrome_framework_shared_library/Chromium Framework.TOC"; else TOOL_VERSION=1737235440 ../../build/toolchain/apple/linker_driver.py -Wcrl,driver,../../third_party/llvm-build/Release+Asserts/bin/clang++ -Wcrl,strippath,/Applications/Xcode-16.2.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/strip -Wcrl,installnametoolpath,../../third_party/llvm-build/Release+Asserts/bin/llvm-install-name-tool -shared -Wcrl,dsym,. -Wcrl,dsymutilpath,/usr/bin/dsymutil -compatibility_version 6834.0.83 -current_version 6834.0.83 -Wl,-install_name,@executable_path/../Frameworks/Chromium\ Framework.framework/Versions/132.0.6834.83/Chromium\ Framework -Wl,--no-warn-duplicate-rpath -Wl,-no_warn_duplicate_libraries -fuse-ld=lld -Wl,--icf=all -Wl,--color-diagnostics -Wl,-mllvm,-instcombine-lower-dbg-declare=0 -Wl,-mllvm,-split-threshold-for-reg-with-hint=0 -Wl,-cache_path_lto,thinlto-cache -Wl,--thinlto-cache-policy=cache_size=10\%:cache_size_bytes=40g:cache_size_files=100000 -flto=thin -Wl,--thinlto-jobs=all -Wl,-mllvm,-import-instr-limit=30 -Wcrl,object_path_lto -Wl,-mllvm,-disable-auto-upgrade-debug-info -Wl,-mllvm,-inlinehint-threshold=360 -fwhole-program-vtables -Wl,--strict-auto-link -Wl,--no-call-graph-profile-sort --target=arm64-apple-macos -Wl,-mllvm,-enable-machine-outliner=never -no-canonical-prefixes -Wl,-dead_strip -Wl,-no_data_in_code_info -Wl,-no_function_starts -nostdlib++ -isysroot ../../../../../../../Applications/Xcode-16.2.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX15.2.sdk -mmacos-version-min=11.0 -Wl,-ObjC -Wcrl,strip,-x,-S -Wl,--lto-O2 -Wl,-order_file -Wl,../../chrome/app/framework.order -Wl,-exported_symbols_list -Wl,../../chrome/app/framework.exports prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libstd.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/liballoc.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libcfg_if.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libcompiler_builtins.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libcore.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libgetopts.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libhashbrown.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libpanic_abort.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libpanic_unwind.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/librustc_demangle.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libstd_detect.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libtest.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libunicode_width.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libunwind.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libaddr2line.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libadler.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libgimli.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/liblibc.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libmemchr.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libminiz_oxide.rlib prebuilt_rustc_sysroot/lib/rustlib/aarch64-apple-darwin/lib/libobject.rlib -L/Applications/Xcode-16.2.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX15.2.sdk/usr/lib -o "obj/chrome/chrome_framework_shared_library/Chromium Framework" "@obj/chrome/chrome_framework_shared_library/Chromium Framework.rsp" obj/third_party/rust/cxx/v1/lib/libcxx_lib.rlib obj/base/libbase_crust_ulogger.rlib obj/base/libbase_clogging_ulog_useverity_ubindgen.rlib obj/build/rust/chromium_prelude/libchromium.rlib obj/base/libbase_clogging_urust_ulog_uintegration_ubindgen.rlib obj/third_party/rust/log/v0_4/lib/liblog_lib.rlib obj/third_party/rust/serde_json_lenient/v0_2/wrapper/libthird_uparty_srust_sserde_ujson_ulenient_sv0_u2_swrapper_cwrapper.rlib obj/third_party/rust/serde/v1/lib/libserde_lib.rlib obj/third_party/rust/serde_json_lenient/v0_2/lib/libserde_json_lenient_lib.rlib obj/third_party/rust/itoa/v1/lib/libitoa_lib.rlib obj/third_party/rust/ryu/v1/lib/libryu_lib.rlib obj/skia/libskia_cbridge_urust_uside.rlib obj/third_party/rust/font_types/v0_7/lib/libfont_types_lib.rlib obj/third_party/rust/bytemuck/v1/lib/libbytemuck_lib.rlib obj/third_party/rust/read_fonts/v0_23/lib/libread_fonts_lib.rlib obj/third_party/rust/skrifa/v0_24/lib/libskrifa_lib.rlib obj/third_party/blink/common/rust_crash/libthird_uparty_sblink_scommon_srust_ucrash_crs.rlib obj/third_party/crabbyavif/libthird_uparty_scrabbyavif_ccrabbyavif.rlib obj/third_party/crabbyavif/libdav1d_sys.rlib obj/third_party/crabbyavif/liblibyuv_sys.rlib obj/third_party/rust/libc/v0_2/lib/liblibc_lib.rlib obj/components/qr_code_generator/libcomponents_sqr_ucode_ugenerator_cqr_ucode_ugenerator_uffi_uglue.rlib obj/third_party/rust/qr_code/v2/lib/libqr_code_lib.rlib && { /Applications/Xcode-16.2.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/otool -l "obj/chrome/chrome_framework_shared_library/ChromiumFramework" | grep LC_ID_DYLIB -A 5; /Applications/Xcode-16.2.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/nm -gPp "obj/chrome/chrome_framework_shared_library/Chromium Framework" | cut -f1-2 -d' ' | grep -v U$$; true; } > "obj/chrome/chrome_framework_shared_library/Chromium Framework.tmp" && if ! cmp -s "obj/chrome/chrome_framework_shared_library/Chromium Framework.tmp" "obj/chrome/chrome_framework_shared_library/Chromium Framework.TOC"; then mv "obj/chrome/chrome_framework_shared_library/Chromium Framework.tmp" "obj/chrome/chrome_framework_shared_library/Chromium Framework.TOC" ; fi; fi
PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and include the crash backtrace.
Stack dump:
0. Running pass "pgo-icall-prom" on module "obj/chrome/browser/libbrowser.a(render_view_context_menu.o)7813350"
LLVM ERROR: Failed to rename temporary file thinlto-cache/Thin-bf7677.tmp.o to thinlto-cache/llvmcache-74D26B8D7614E587C7756BC144172DE4EE560C8C: No such file or directory
clang++: error: unable to execute command: Abort trap: 6
clang++: error: linker command failed due to signal (use -v to see invocation) I am testing with the previous version of Chromium to see if this is some issue related to my building environment (I had to reconfigure it recently). |
The path issue is most likely caused by changes in your build environment, but the linking issue looks similar to what's described in (381444313). Try building with the gn arg |
I'm tempted to tag this soon (skipping the required second approval) and publish another revision if the issue on macOS turns out to be best fixed here. |
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.
Sorry for the delayed approval I was in flight-mode. Literally in every sense of the word :)
The path issue is not occuring in previous versions, so I am not sure what is happening, and I'll just use soft link as a walk around for now. I will be testing |
ungoogled-software/ungoogled-chromium#3143 (comment) Signed-off-by: Qian Qian "Cubik" <[email protected]>
See: ungoogled-software/ungoogled-chromium#3143 (comment) Signed-off-by: Qian Qian "Cubik" <[email protected]>
macOS builds fine with |
I remembered now, I had a very similar issue on Linux. But I cannot remember how I fixed it :( It was either that filesystem running out of i-nodes, or a bad RAM, or a polluted |
I don't think these apply to my case :( |
See: ungoogled-software/ungoogled-chromium#3143 (comment) Signed-off-by: Qian Qian "Cubik" <[email protected]>
This PR updates to Chromium 132.0.6834.83
Notes:
The rollup files are now duplicated for devtools-frontend, so another exclusion for the same file has been added to update_lists.py
The Inox safebrowsing patch had a small update for the changes in [5923123]
There was a new addition to the trk patch for [5964077]
disable-google-host-detection.patch had changes do to [5970653]
The pruned binaries patch had new additions for [5934367], [5998631], and [5917173]
The safebrowsing patch had changes for [5989132], [6001419], [5939832], [6006096], [6019823], [5958668], and [5990352]
remove-unused-preferences-fields.patch had removals for [5985667] and [5963058], changes for [5983026], and new additions for [5998292] and [6000926]
disable-capture-all-screens.patch had changes for [5998751]
Due to the change in [5926365] a new patch has been added: disable-chromelabs.patch
Unbranded chromium builds return their channel as UNKNOWN so this means that ChromeLabs is now permanently shown.
I had considered reimplementing the flag to keep that how it was previously, but would clash with the opt-in nature of the flags we add.
Disabling ChromeLabs entirely seems like the best route since we only publish stable versions and the expiriments are still available at chrome://flags
The UI removal patch had changes for [6003547]
The clone script has been updated to prevent removing the 'out' directories under node_modules since they now ship folders with that name. I've also removed the instrumented_libs custom dep since it has had it's condition updated to skip the download for small checkouts.
I've updated the list of contingent paths in prune_binaries.py
If you're not yet on llvm 19+ then you may need to apply the following change since CTAD isn't available yet:
Linux builds might also require removing the ELF CREL flag if you're not running the latest changes:
Change log
Blog post