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

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.