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

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.