From b47bb4714ac8a77459a4656012fae2308b857670 Mon Sep 17 00:00:00 2001 From: Pino Toscano Date: Fri, 17 Sep 2021 06:40:51 +0200 Subject: [PATCH] spec: use nodejs-devel on Fedora According to the Fedora guidelines, nodejs-devel is needed for using nodejs modules: https://docs.fedoraproject.org/en-US/packaging-guidelines/Node.js/#_buildrequires Since this applies only on Fedora, and that nodejs-devel may not even exist on other distros (e.g. CentOS 9 Stream), then limit the nodejs-devel requirement only to Fedora, keep using nodejs unconditionally everywhere. --- packaging/cockpit-starter-kit.spec.in | 3 +++ 1 file changed, 3 insertions(+) diff --git a/packaging/cockpit-starter-kit.spec.in b/packaging/cockpit-starter-kit.spec.in index 72c9f41..ef8ea6b 100644 --- a/packaging/cockpit-starter-kit.spec.in +++ b/packaging/cockpit-starter-kit.spec.in @@ -7,6 +7,9 @@ License: LGPLv2+ Source0: https://github.com/cockpit-project/starter-kit/releases/download/%{version}/%{name}-%{version}.tar.xz Source1: https://github.com/cockpit-project/starter-kit/releases/download/%{version}/%{name}-node-%{version}.tar.xz BuildArch: noarch +%if 0%{?fedora} +BuildRequires: nodejs-devel +%endif BuildRequires: nodejs BuildRequires: make BuildRequires: libappstream-glib