Re: [PATCH 2/2] python/sepolicy: add a pyproject.toml file

Cathy Hu <[email protected]> Fri, 24 Jul 2026 17:10:18 +0200
Newsgroups	org.kernel.vger.selinux
Message-ID	<[email protected]>
On 7/24/26 4:44 PM, Stephen Smalley wrote:
> On Fri, Jul 24, 2026 at 10:26 AM Stephen Smalley
> <[email protected]> wrote:
>>
>> On Wed, Jul 22, 2026 at 6:20 AM Cathy Hu <[email protected]> wrote:
>>>
>>> On 7/21/26 8:53 PM, Stephen Smalley wrote:
>>>> Add a pyproject.toml file for the sepolicy python module.  This
>>>> required also updating setup.py to avoid defining author outside of
>>>> pyproject.toml, and I further updated the authors to refer to the list
>>>> like libselinux since Dan is enjoying retirement.
>>>>
>>>> Fixes: https://github.com/SELinuxProject/selinux/issues/505
>>>> Signed-off-by: Stephen Smalley <[email protected]>
>>>> ---
>>>>    python/sepolicy/pyproject.toml | 19 +++++++++++++++++++
>>>>    python/sepolicy/setup.py       |  2 --
>>>>    2 files changed, 19 insertions(+), 2 deletions(-)
>>>>    create mode 100644 python/sepolicy/pyproject.toml
>>>>
>>>> diff --git a/python/sepolicy/pyproject.toml b/python/sepolicy/pyproject.toml
>>>> new file mode 100644
>>>> index 00000000..472faf7a
>>>> --- /dev/null
>>>> +++ b/python/sepolicy/pyproject.toml
>>>> @@ -0,0 +1,19 @@
>>>> +[build-system]
>>>> +requires = ["setuptools", "wheel"]
>>>> +build-backend = "setuptools.build_meta"
>>>> +
>>>> +[project]
>>>> +name = "sepolicy"
>>>> +version = "3.11"
>>>> +description = "Python tools for SELinux policy analysis (sepolicy, sepolgen)"
>>>> +license = "GPL-2.0-or-later"
>>>> +authors = [
>>>> +    { name = "SELinux Project", email = "[email protected]" }
>>>> +]
>>>> +dependencies = [
>>>> +    "libselinux-python",
>>>> +    "libsemanage-python",
>>>> +]
>>>> +
>>>> +[project.scripts]
>>>> +sepolicy = "sepolicy.__main__:main"
>>>> diff --git a/python/sepolicy/setup.py b/python/sepolicy/setup.py
>>>> index ca0396b5..5c4f2e5d 100644
>>>> --- a/python/sepolicy/setup.py
>>>> +++ b/python/sepolicy/setup.py
>>>> @@ -8,8 +8,6 @@ setup(
>>>>        name="sepolicy",
>>>>        version="3.11",
>>>>        description="Python SELinux Policy Analyses bindings",
>>>> -    author="Daniel Walsh",
>>>> -    author_email="[email protected]",
>>>>        packages=[
>>>>            "sepolicy",
>>>>            "sepolicy.templates",
>>>
>>> lgtm, or at least i don't see the warning anymore in the build system
>>
>> Thanks, both merged.
> 
> And both reverted since they broke the testfarm build.
> Will defer to someone better versed in this area to fix this.

that's odd, not sure why the patch successfully built in our system, i can have a look next week
OpenPGP_signature.asc (application/pgp-signature, 870 B)