From 9af3351a76dd3bef2178c89968f082d306b43fa7 Mon Sep 17 00:00:00 2001
From: Marek Kulik <mkulik@redhat.com>
Date: Aug 17 2021 10:45:45 +0000
Subject:  Changes and notes for build requirements


* BuildRequires in proxysql .spec file has been divided and commented to
increase readibility. Dependencies specificly required/used by
proxysql code are now marked.

* Few special dependencies like perl, that are only used by additional
scripts has been also separated.

* systemd requirement has been replaced by systemd-rpm-macros according
to fedoraproject documentation.
https://docs.fedoraproject.org/en-US/packaging-guidelines/Scriptlets/#_systemd
This is ony compatible with RHEL >= 8 or the equivalent version of a fedora.

---

diff --git a/proxysql.spec b/proxysql.spec
index 5b40a31..a0a952e 100644
--- a/proxysql.spec
+++ b/proxysql.spec
@@ -8,17 +8,26 @@ URL:           http://www.proxysql.com/
 # and prometheus-cpp which is MIT
 License:       GPLv3+ and LGPLv2+ and BSD and MIT
 
-BuildRequires: make
-BuildRequires: openssl-devel, libev-devel, perl
-BuildRequires: cmake, gcc-c++, libcurl-devel, pcre-devel
-BuildRequires: systemd
-BuildRequires: libconfig-devel, lz4-devel
-BuildRequires: libdaemon-devel
-BuildRequires: sqlite-devel
+BuildRequires: make, automake
+BuildRequires: cmake, gcc-c++
+BuildRequires: systemd-rpm-macros
+
+# Required by proxysql code
+BuildRequires: libtool
+BuildRequires: pcre-devel
 BuildRequires: zlib-devel
+BuildRequires: libev-devel
+BuildRequires: sqlite-devel
+BuildRequires: openssl-devel
+BuildRequires: libcurl-devel
+BuildRequires: libdaemon-devel
+BuildRequires: libconfig-devel, lz4-devel
+
+# Used by provided (scripts) tools
+BuildRequires: perl
+
+# Specific dependency for Fedora/RHEL/Centos
 BuildRequires: gnutls-devel
-BuildRequires: libtool
-BuildRequires: automake
 
 Suggests: mariadb, community-mysql