From 5ecc7d08c4fc4fa74925a6252291cc50b52b3325 Mon Sep 17 00:00:00 2001 From: "Brian J. Murrell" Date: Tue, 1 Nov 2022 16:14:06 -0400 Subject: [PATCH 1/2] Set minimum version of npm To imply a minimum version of NodeJS which is now 16 and prevent people from trying to build with a NodeJS/npm that is too old. --- fedora/jellyfin-web.spec | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/fedora/jellyfin-web.spec b/fedora/jellyfin-web.spec index 2c48737ed5..69d04e7709 100644 --- a/fedora/jellyfin-web.spec +++ b/fedora/jellyfin-web.spec @@ -2,7 +2,7 @@ Name: jellyfin-web Version: 10.8.0 -Release: 1%{?dist} +Release: 2%{?dist} Summary: The Free Software Media System web client License: GPLv2 URL: https://jellyfin.org @@ -14,7 +14,8 @@ BuildArch: noarch BuildRequires: nodejs %else BuildRequires: git -BuildRequires: npm +# Nodejs 16 is required and npm >= 8 should bring in NodeJS 16 +BuildRequires: npm >= 8 %endif %description From a6d411aa8f59abf8e16c6623ca21dd070667b655 Mon Sep 17 00:00:00 2001 From: "Brian J. Murrell" Date: Tue, 1 Nov 2022 16:24:40 -0400 Subject: [PATCH 2/2] Add some comments about nodejs module --- fedora/jellyfin-web.spec | 2 ++ 1 file changed, 2 insertions(+) diff --git a/fedora/jellyfin-web.spec b/fedora/jellyfin-web.spec index 69d04e7709..28407b8bcd 100644 --- a/fedora/jellyfin-web.spec +++ b/fedora/jellyfin-web.spec @@ -15,6 +15,8 @@ BuildRequires: nodejs %else BuildRequires: git # Nodejs 16 is required and npm >= 8 should bring in NodeJS 16 +# This requires the build environment to use the nodejs:16 module stream: +# dnf module {install|switch-to}:web nodejs:16 BuildRequires: npm >= 8 %endif