Bug ID | 5114 |
---|---|
Summary | Review request: ungoogled-chromium - Chromium, sans integration with Google |
Product | Package Reviews |
Version | Current |
Hardware | x86_64 |
OS | GNU/Linux |
Status | NEW |
Severity | enhancement |
Priority | P1 |
Component | Review Request |
Assignee | rpmfusion-package-review@rpmfusion.org |
Reporter | dotqvint@gmail.com |
CC | rpmfusion-package-review@rpmfusion.org |
Blocks | 2, 30 |
namespace | free |
Full URLs to the spec file, source RPM and binary RPM: - https://ungoogled.blob.core.windows.net/ungoogled/review/2018-12-18_1/ungoogled-chromium.spec - https://ungoogled.blob.core.windows.net/ungoogled/review/2018-12-18_1/ungoogled-chromium-71.0.3578.98-2.fc29.src.rpm - https://ungoogled.blob.core.windows.net/ungoogled/review/2018-12-18_1/ungoogled-chromium-71.0.3578.98-2.fc29.x86_64.rpm A short description for the package: ungoogled-chromium is Chromium, sans integration with Google. It also features some tweaks to enhance privacy, control, and transparency (almost all of which require manual activation or enabling). ungoogled-chromium retains the default Chromium experience as closely as possible. Unlike other Chromium forks that have their own visions of a web browser, ungoogled-chromium is essentially a drop-in replacement for Chromium. This package is not eligible to be included in Fedora because of proprietary codecs. This is my first RPM Fusion package. I wish to keep on improving it. I also seek a sponsor as I'm not a Fedora sponsored packager nor an RPM Fusion sponsored packager. ----------------------- rpmlint on source RPM ----------------------- This package is primarily based on Tom Callaway's <spot@fedoraproject.org> work. All rpmlint warnings and errors are also reproducible for the source RPM of the Fedora `chromium` package. * E: hardcoded-library-path in /usr/lib/libc.so Caused by this line: BuildRequires: /lib/libc.so.6 /usr/lib/libc.so Tom Callaway commented it as follows: # Really, this is what we want: # BuildRequires: glibc-devel(x86-32) libgcc(x86-32) # But, koji only offers glibc32. Maybe that's enough. # This BR will pull in either glibc.i686 or glibc32. * W: unversioned-explicit-provides bundled(boringssl) * W: unversioned-explicit-provides bundled(bspatch) * W: unversioned-explicit-provides bundled(crashpad) * W: unversioned-explicit-provides bundled(dmg_fp) * W: unversioned-explicit-provides bundled(iccjpeg) * W: unversioned-explicit-provides bundled(mozc) * W: unversioned-explicit-provides bundled(re2) * W: unversioned-explicit-provides bundled(skia) * W: unversioned-explicit-provides bundled(xdg-mime) * W: unversioned-explicit-provides bundled(xdg-user-dirs) These warnings are caused by `Provides: bundled(...)` lines. Chromium uses its own forks of third-party libraries -- unbundling or versioning seem to be impossible. * E: hardcoded-library-path in /usr/lib/%{chromium_browser_channel} * E: hardcoded-library-path in /usr/lib/%{chromium_browser_channel}/%{chromium_browser_channel}.sh * E: hardcoded-library-path in /usr/lib/chrome-sandbox These errors are caused by `semanage` invocations: semanage fcontext -a -t bin_t /usr/lib/%{chromium_browser_channel} semanage fcontext -a -t bin_t /usr/lib/%{chromium_browser_channel}/%{chromium_browser_channel}.sh semanage fcontext -a -t chrome_sandbox_exec_t /usr/lib/chrome-sandbox Tom Callaway commented that semanage itself adjusts the lib directory naming. * W: mixed-use-of-spaces-and-tabs (spaces: line 60, tab: line 98) * W: invalid-url Source2: depot_tools.git-master.tar.gz Revision of the file is accessible only from the Fedora Lookaside Cache. ----------------------- rpmlint on x86_64 RPM ----------------------- The most of the following warnings and errors are reproducible for the Fedora `chromium` package. * E: explicit-lib-dependency libcanberra-gtk3(x86-64) Chromium expects this GTK module for some reason. * W: unstripped-binary-or-object /usr/lib64/ungoogled-chromium/chrome-sandbox * W: unstripped-binary-or-object /usr/lib64/ungoogled-chromium/chromedriver * W: unstripped-binary-or-object /usr/lib64/ungoogled-chromium/headless_shell * W: unstripped-binary-or-object /usr/lib64/ungoogled-chromium/protoc * W: unstripped-binary-or-object /usr/lib64/ungoogled-chromium/swiftshader/libEGL.so * W: unstripped-binary-or-object /usr/lib64/ungoogled-chromium/swiftshader/libGLESv2.so * W: unstripped-binary-or-object /usr/lib64/ungoogled-chromium/ungoogled-chromium I'm not sure how to resolve this. Fedora package has similar warnings. * W: conffile-without-noreplace-flag /etc/ungoogled-chromium/master_preferences Contains distro-specific settings, noreplace flag seems to be needless. * E: missing-call-to-setgroups-before-setuid /usr/lib64/ungoogled-chromium/chrome-sandbox * E: missing-call-to-chdir-with-chroot /usr/lib64/ungoogled-chromium/ungoogled-chromium * E: missing-call-to-chdir-with-chroot /usr/lib64/ungoogled-chromium/headless_shell * E: setuid-binary /usr/lib64/ungoogled-chromium/chrome-sandbox root 4755 * E: non-standard-executable-perm /usr/lib64/ungoogled-chromium/chrome-sandbox 4755 I'm not sure what can I do here. Both Fedora package and chromium-vaapi have similar errors. * E: script-without-shebang /usr/lib64/ungoogled-chromium/pyproto/google/protobuf/internal/__init__.py * E: htaccess-file /usr/lib64/ungoogled-chromium/resources/inspector/.htaccess * E: zero-length /usr/lib64/ungoogled-chromium/resources/inspector/emulated_devices/emulated_devices_module.js * E: zero-length /usr/lib64/ungoogled-chromium/resources/inspector/js_profiler/js_profiler_module.js * E: zero-length /usr/lib64/ungoogled-chromium/resources/inspector/node_debugger/node_debugger_module.js Resource files.