From 6588e7bdd3acbad84209cd63daa4fe7738d2858a Mon Sep 17 00:00:00 2001
From: Tom Callaway <spot@fedoraproject.org>
Date: Wed, 28 Mar 2018 10:01:28 -0400
Subject: [PATCH 1/2] check that there is no system google module, shadowing
 bundled ones :: conditionalize api keys (on by default)

---
 chromium.spec | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/chromium.spec b/chromium.spec
index 845e712..4f76103 100644
--- a/chromium.spec
+++ b/chromium.spec
@@ -6,6 +6,10 @@
 %global lsuffix fedora
 %endif
 
+# Some people wish not to use the Fedora Google API keys. Mmkay.
+# Expect stuff to break in weird ways if you disable.
+%global useapikeys 1
+
 # Leave this alone, please.
 %global target out/Release
 %global headlesstarget out/Headless
@@ -110,10 +114,17 @@ BuildRequires:  libicu-devel >= 5.4
 ### Note: These are for Fedora use ONLY.
 ### For your own distribution, please get your own set of keys.
 ### http://lists.debian.org/debian-legal/2013/11/msg00006.html
+%if %{useapikeys}
 %global api_key AIzaSyDUIXvzVrt5OkVsgXhQ6NFfvWlA44by-aw
 %global default_client_id 449907151817.apps.googleusercontent.com
 %global default_client_secret miEreAep8nuvTdvLums6qyLK
 %global chromoting_client_id 449907151817-8vnlfih032ni8c4jjps9int9t86k546t.apps.googleusercontent.com 
+%else
+%global api_key %nil
+%global default_client_id %nil
+%global default_client_secret %nil
+%global chromoting_client_id %nil
+%endif
 
 %global majorversion 65
 
@@ -1130,6 +1141,12 @@ sed -i 's|exec "${THIS_DIR}/ninja-linux${LONG_BIT}"|exec "/usr/bin/ninja-build"|
 . /opt/rh/devtoolset-7/enable
 %endif
 
+# Check that there is no system 'google' module, shadowing bundled ones:
+if python -c 'import google ; print google.__path__' 2> /dev/null ; then \
+    echo "Python 'google' module is defined, this will shadow modules of this build"; \
+    exit 1 ; \
+fi
+
 tools/gn/bootstrap/bootstrap.py -v --gn-gen-args "$CHROMIUM_CORE_GN_DEFINES $CHROMIUM_BROWSER_GN_DEFINES"
 %{target}/gn gen --args="$CHROMIUM_CORE_GN_DEFINES $CHROMIUM_BROWSER_GN_DEFINES" %{target}
 
@@ -1597,6 +1614,10 @@ getent group chrome-remote-desktop >/dev/null || groupadd -r chrome-remote-deskt
 
 
 %changelog
+* Wed Mar 28 2018 Tom Callaway <spot@fedoraproject.org>
+- check that there is no system 'google' module, shadowing bundled ones
+- conditionalize api keys (on by default)
+
 * Wed Mar 21 2018 Tom Callaway <spot@fedoraproject.org> 65.0.3325.181-1
 - update to 65.0.3325.181
 

From 11492d003030b34ed8a8dd57b3845d97ee051c7d Mon Sep 17 00:00:00 2001
From: Tom Callaway <spot@fedoraproject.org>
Date: Wed, 4 Apr 2018 11:22:06 -0400
Subject: [PATCH 2/2] add explicit dependency on minizip (bz 1534282)

---
 chromium.spec | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/chromium.spec b/chromium.spec
index 4f76103..8bbca8a 100644
--- a/chromium.spec
+++ b/chromium.spec
@@ -134,7 +134,7 @@ Name:		chromium%{chromium_channel}%{?freeworld:-freeworld}
 Name:		chromium%{chromium_channel}
 %endif
 Version:	%{majorversion}.0.3325.181
-Release:	1%{?dist}
+Release:	2%{?dist}
 Summary:	A WebKit (Blink) powered web browser
 Url:		http://www.chromium.org/Home
 License:	BSD and LGPLv2+ and ASL 2.0 and IJG and MIT and GPLv2+ and ISC and OpenSSL and (MPLv1.1 or GPLv2 or LGPLv2)
@@ -584,6 +584,9 @@ Chromium is an open-source web browser, powered by WebKit (Blink).
 
 %package common
 Summary: Files needed for both the headless_shell and full Chromium
+# Chromium needs an explicit Requires: minizip
+# We put it here to cover headless too.
+Requires: minizip%{_isa}
 
 %description common
 %{summary}.
@@ -1614,6 +1617,9 @@ getent group chrome-remote-desktop >/dev/null || groupadd -r chrome-remote-deskt
 
 
 %changelog
+* Wed Apr  4 2018 Tom Callaway <spot@fedoraproject.org> 65.0.3325.181-2
+- add explicit dependency on minizip (bz 1534282)
+
 * Wed Mar 28 2018 Tom Callaway <spot@fedoraproject.org>
 - check that there is no system 'google' module, shadowing bundled ones
 - conditionalize api keys (on by default)