PR#89: Two simple policy updates that were previously approved - fesco/fesco-docs

fesco / fesco-docs

#89 Two simple policy updates that were previously approved

Merged 10 months ago by zbyszek. Opened 10 months ago by zbyszek.

fesco/ zbyszek/fesco-docs various-document-its into main

sigs: ownership of ghc packages by Haskell SIG

Zbigniew Jędrzejewski-Szmek • 10 months ago

439d6d8

updates: add exception for ruff

Zbigniew Jędrzejewski-Szmek • 10 months ago

4fc383b

updates: order subsections alphabetically

Zbigniew Jędrzejewski-Szmek • 10 months ago

df5aef9

fesco/modules/ROOT/pages/SIG_policy.adoc

file modified

		`@@ -18,6 +18,11 @@`
		Note that `+%gometa+` automatically inserts `+BuildRequires: go-rpm-macros+`,
		`so packages that use that macro are also included in this policy.`

		`+ == Haskell SIG`
		`+`
		+ The Haskell SIG group (`+@haskell-lang-sig+`) MUST be granted the `+commit+` or `+admin+` ACLs
		+ on all Haskell library packages, i.e. all packages named `+ghc-*+`.
		`+`
		`== R SIG`

		The R SIG group (`+@r-maint-sig+`) MUST be granted `+commit+` or `+admin+` ACLs

fesco/modules/ROOT/pages/Updates_Policy.adoc

file modified

+11 -9

		`@@ -374,11 +374,10 @@`

		`The following packages have been granted exceptions for the following reasons:`

		`- [[rust]]`
		`- ===== Rust -devel packages`
		`- This exception covers all Rust -devel packages, i.e. rpms with Rust sources.`
		`- Those packages are used to build packaged Rust binaries`
		`- and are not directly usable by users.`
		`+ [[kde]]`
		`+ ===== KDE`
		`+`
		`+ Refer to the https://fedoraproject.org/wiki/SIGs/KDE/Update_policy[KDE update policy] for more details.`

		`[[kernel-package]]`
		`===== kernel package`
		`@@ -390,10 +389,11 @@`
		`allowing users to boot older kernels in the event newer ones fail to work,`
		`and allowing time for kernel maintainers to fix any critical bugs in new kernels on older stable releases.`

		`- [[kde]]`
		`- ===== KDE`
		`-`
		`- Refer to the https://fedoraproject.org/wiki/SIGs/KDE/Update_policy[KDE update policy] for more details`
		`+ [[rust]]`
		`+ ===== Rust -devel packages`
		`+ This exception covers all Rust -devel packages, i.e. rpms with Rust sources.`
		`+ Those packages are used to build packaged Rust binaries`
		`+ and are not directly usable by users.`

		`===== Other packages`

		`@@ -404,6 +404,8 @@`
		`(https://pagure.io/fesco/issue/2652[FESCo #2652]),`
		* Python code formatter `python-black`
		`(https://pagure.io/fesco/issue/2652[FESCo #2652]),`
		+ * Python code linter `ruff`
		`+ (https://pagure.io/fesco/issue/3197[FESCo #3197]),`
		* Python test executor `python-tox`
		`(https://pagure.io/fesco/issue/2652[FESCo #2652]),`
		* command-line HTTP client `httpie`

no initial comment

decathorpe commented 10 months ago

The Haskell SIG group (+@haskell-lang-sig+) MUST be granted the +commit+ or +admin+ ACLs on all Haskell packages, in particular all +ghc-*+ libraries.

This is not an actually actionable policy. What defines a "Haskell package"? Which one of commit or admin should be the automatically applied default? This needs to be determined in the FESCo ticket.

decathorpe commented 10 months ago

FTR, what was in the initial request for this policy (see FESCO ticket) was "starting with all ghc-* libraries", and not all Haskell packages.

zbyszek commented 10 months ago

Which one of commit or admin should be the automatically applied default?

The policy is clear: there are two alternatives; either one satisfies the policy. And this is what we want: the owner can add commit (the weaker form), but if they add admin, that is fine too. If we said that it "MUST be commit", then packages using admin would not satisfy the policy as phrased. The same formulation is used for other packages.