From: Prarit Bhargava <pra...@redhat.com>
redhat/Makefile: Add some descriptions
Add some descriptive text about some of the Makefile sections.
Signed-off-by: Prarit Bhargava <pra...@redhat.com>
diff --git a/redhat/Makefile b/redhat/Makefile
index blahblah..blahblah 100644
--- a/redhat/Makefile
+++ b/redhat/Makefile
@@ -1,4 +1,8 @@
-# deprecated variables
+#
+# Deprecated variables
+#
+# This section is intentionally left at the top of the Makefile so it won't be
ignored.
+#
ifdef BREW_FLAGS
# deprecated in 5.17.0
$(warning WARNING: BREW_FLAGS will be deprecated in a later release, use
BUILD_FLAGS instead.)
@@ -139,6 +143,10 @@ PKGRELEASE:=$(PREBUILD)$(BUILD)$(BUILDID)
RPMVERSION:=$(RPMKVERSION).$(RPMKPATCHLEVEL)-$(PKGRELEASE)
SPECRELEASE:=$(PREBUILD)$(BUILD)%{?buildid}%{?dist}
+#
+# This conditional statement is where fedora, centos, and other (aka RHEL)
+# specific values should be set.
+#
ifeq ("$(DISTRO)", "fedora")
# A non-unified tarball means that the tarball is only upstream sources
# and the patches get applied as a diff in the spec file
@@ -250,6 +258,14 @@ ifndef BUILD_FLAGS
BUILD_FLAGS += $(TEST_FLAGS)
endif
+#
+# Targets
+#
+# Do not place variable declarations below this line. Variables are exported
+# to scripts called in the targets below and do not need to be passed on the
+# command-line.
+#
+
default: dist-help
dist-python-check:
diff --git a/redhat/Makefile.variables b/redhat/Makefile.variables
index blahblah..blahblah 100644
--- a/redhat/Makefile.variables
+++ b/redhat/Makefile.variables
@@ -1,3 +1,10 @@
+# Only the variables listed in this file, ../Makefile.rhelver, and
+# Makefile.rhpkg are available for command line modification. There is no
+# guarantee on the stability of variables found in this file, or any other
+# redhat/ Makefile.
+#
+# Merge requests to include additional variables in _this_ file must include
+# details of why the variable must be exported.
# Variables below should be changed as needed for building on a different
# branch, or with a different upstream branch. This should be the only
--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1641
_______________________________________________
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it:
https://pagure.io/fedora-infrastructure
