Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.gentoo.dev > #70404
| From | Sam James <sam@gentoo.org> |
|---|---|
| Newsgroups | linux.gentoo.dev |
| Subject | Re: [gentoo-dev] [PATCH v3 2/5] selinux-policy-2.eclass: respect the sanctity of -rN for EAPI=8 |
| Date | 2026-04-08 07:10 +0200 |
| Message-ID | <MHtkR-dwNx-5@gated-at.bofh.it> (permalink) |
| References | <MH9YR-djx9-3@gated-at.bofh.it> <MH9YS-djx9-7@gated-at.bofh.it> <MHarT-djHB-3@gated-at.bofh.it> |
| Organization | Gentoo |
[Multipart message — attachments visible in raw view] - view raw
Ulrich Müller <ulm@gentoo.org> writes:
>>>>>> On Tue, 07 Apr 2026, Rahul Sandhu wrote:
>
>> +if [[ ${EAPI} = 7 ]]; then
>> + # @ECLASS_VARIABLE: BASEPOL
>> + # @DESCRIPTION:
>> + # This variable contains the version string of the selinux-base-policy package
>> + # that this module build depends on. It is used to patch with the appropriate
>> + # patch bundle(s) that are part of selinux-base-policy.
>> + : "${BASEPOL:="${PVR}"}"
>> +else
>> + # @ECLASS_VARIABLE: BASEPOL
>> + # @DESCRIPTION:
>> + # This variable contains the version string of the selinux-base-policy package
>> + # that this module build depends on. It is used to patch with the appropriate
>> + # patch bundle(s) that are part of selinux-base-policy.
>> + # _pN corresponds to -rN in the patch bundle(s).
>> + : "${BASEPOL:="${PV/_p/-r}"}"
>> +fi
>
> This won't work. eclass-to-manpage expects eclass documentation to be
> left-aligned. Anything that is indented will be ignored.
>
> You also cannot have a conditional there, i.e. if both comments were
> without indentation then eclass-to-manpage would see a duplicate
> definition and would terminate with an error.
Yes. Just change it so that it's one description where the text mentions
different behaviour for EAPI 7 vs other values.
That's better anyway even if you ignore alignment or processing
problems -- what are the semantics supposed to be for a duplicate
eclassdoc? :)
Back to linux.gentoo.dev | Previous | Next — Previous in thread | Next in thread | Find similar
[gentoo-dev] [PATCH v3 2/5] selinux-policy-2.eclass: respect the sanctity of -rN for EAPI=8 Rahul Sandhu <nvraxn@posteo.uk> - 2026-04-07 10:30 +0200
Re: [gentoo-dev] [PATCH v3 2/5] selinux-policy-2.eclass: respect the sanctity of -rN for EAPI=8 Ulrich Müller <ulm@gentoo.org> - 2026-04-07 11:00 +0200
Re: [gentoo-dev] [PATCH v3 2/5] selinux-policy-2.eclass: respect the sanctity of -rN for EAPI=8 Sam James <sam@gentoo.org> - 2026-04-08 07:10 +0200
Re: [gentoo-dev] [PATCH v3 2/5] selinux-policy-2.eclass: respect the sanctity of -rN for EAPI=8 "Rahul Sandhu" <nvraxn@posteo.uk> - 2026-04-08 11:00 +0200
[gentoo-dev] Re: [PATCH v3 2/5] selinux-policy-2.eclass: respect the sanctity of -rN for EAPI=8 Kenton Groombridge <concord@gentoo.org> - 2026-04-08 01:40 +0200
[gentoo-dev] Re: [PATCH v3 2/5] selinux-policy-2.eclass: respect the sanctity of -rN for EAPI=8 Sam James <sam@gentoo.org> - 2026-04-08 07:10 +0200
[gentoo-dev] Re: [PATCH v3 2/5] selinux-policy-2.eclass: respect the sanctity of -rN for EAPI=8 "Rahul Sandhu" <nvraxn@posteo.uk> - 2026-04-08 11:00 +0200
csiph-web