Merge pull request #3571 from mihawk90/fedora-spec-rework

Cleanup and standardise Fedora build (web)

deployment/Dockerfile.fedora

@@ -1,4 +1,4 @@
-FROM fedora:33
+FROM fedora:36
 
 # Docker build arguments
 ARG SOURCE_DIR=/jellyfin
@@ -11,7 +11,7 @@ ENV IS_DOCKER=YES
 
 # Prepare Fedora environment
 RUN dnf update -y \
-  && dnf install -y @buildsys-build rpmdevtools git dnf-plugins-core nodejs autoconf automake glibc-devel
+  && dnf install -y @buildsys-build rpmdevtools git dnf-plugins-core nodejs autoconf automake glibc-devel make
 
 # Link to build script
 RUN ln -sf ${SOURCE_DIR}/deployment/build.fedora /build.sh

fedora/Makefile

@@ -9,8 +9,12 @@ TARBALL :=$(NAME)-$(subst -,~,$(VERSION)).tar.gz
 
 epel-7-x86_64_repos := https://rpm.nodesource.com/pub_16.x/el/\$$releasever/\$$basearch/
 
+fed_ver := $(shell rpm -E %fedora)
+# fallback when not running on Fedora
+fed_ver ?= 36
+TARGET  ?= fedora-$(fed_ver)-x86_64
+
 outdir  ?= $(PWD)/$(DIR)/
-TARGET  ?= fedora-35-x86_64
 
 srpm: $(DIR)/$(SRPM)
 tarball: $(DIR)/$(TARBALL)

fedora/jellyfin-web.spec

@@ -4,7 +4,7 @@ Name:           jellyfin-web
 Version:        10.8.0
 Release:        1%{?dist}
 Summary:        The Free Software Media System web client
-License:        GPLv3
+License:        GPLv2
 URL:            https://jellyfin.org
 # Jellyfin Server tarball created by `make -f .copr/Makefile srpm`, real URL ends with `v%%{version}.tar.gz`
 Source0:        jellyfin-web-%{version}.tar.gz
@@ -17,9 +17,6 @@ BuildRequires:	git
 BuildRequires:	npm
 %endif
 
-# Disable Automatic Dependency Processing
-AutoReqProv:    no
-
 %description
 Jellyfin is a free software media system that puts you in control of managing and streaming your media.
 
@@ -27,22 +24,26 @@ Jellyfin is a free software media system that puts you in control of managing an
 %prep
 %autosetup -n jellyfin-web-%{version} -b 0
 
-%build
-
-%install
 %if 0%{?rhel} > 0 && 0%{?rhel} < 8
 # Required for CentOS build
 chown root:root -R .
 %endif
+
+
+%build
 npm ci --no-audit --unsafe-perm
-%{__mkdir} -p %{buildroot}%{_datadir}
-mv dist %{buildroot}%{_datadir}/jellyfin-web
-%{__install} -D -m 0644 LICENSE %{buildroot}%{_datadir}/licenses/jellyfin/LICENSE
+
+
+%install
+%{__mkdir} -p %{buildroot}%{_libdir}/jellyfin/jellyfin-web
+%{__cp} -r dist/* %{buildroot}%{_libdir}/jellyfin/jellyfin-web
+
 
 %files
 %defattr(644,root,root,755)
-%{_datadir}/jellyfin-web
-%{_datadir}/licenses/jellyfin/LICENSE
+%{_libdir}/jellyfin/jellyfin-web
+%license LICENSE
+
 
 %changelog
 * Fri Jun 10 2022 Jellyfin Packaging Team <packaging@jellyfin.org>