From 172a082fe4c77204dc3f61a95d8b239e4a267c87 Mon Sep 17 00:00:00 2001 From: Marcelo Atie Date: Tue, 19 Dec 2017 16:50:59 -0500 Subject: [PATCH] fix libappindicator dependency in OpenSUSE Fix by @marceloatie, originally in https://github.com/keybase/client/pull/5226, squashed and merged by @oconnor663. --- packaging/linux/rpm/package_binaries.sh | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/packaging/linux/rpm/package_binaries.sh b/packaging/linux/rpm/package_binaries.sh index 3ad24023ec9a..578871897581 100755 --- a/packaging/linux/rpm/package_binaries.sh +++ b/packaging/linux/rpm/package_binaries.sh @@ -26,12 +26,16 @@ mode="$(cat "$build_root/MODE")" name="$("$here/../../binary_name.sh" "$mode")" -dependencies="Requires: at" +if [ "$rpm_arch" = "x86_64" ] ; then + dependencies="Requires: at, fuse, libappindicator1, 'libXss.so.1()(64bit)'" +else + dependencies="Requires: at, fuse, libappindicator1, 'libXss.so.1'" +fi + if [ "$mode" = "production" ] ; then repo_url="http://dist.keybase.io/linux/rpm/repo" elif [ "$mode" = "prerelease" ] ; then repo_url="http://prerelease.keybase.io/rpm" - dependencies="Requires: at, fuse, libXScrnSaver" elif [ "$mode" = "staging" ] ; then # Note: This doesn't exist yet. But we need to be distinct from the # production URL, because we're moving to a model where we build a clean