V Mon, Mar 27, 2023 at 11:46:49AM -0400, Neal Gompa napsal(a):
> On Mon, Mar 27, 2023 at 11:24 AM Fabio Valentini <decatho...@gmail.com> wrote:
> >
> > On Mon, Mar 27, 2023 at 2:10 PM Neal Gompa <ngomp...@gmail.com> wrote:
> > >
> > > On Sun, Mar 26, 2023 at 12:27 PM Kalev Lember <kalevlem...@gmail.com> 
> > > wrote:
> > > >
> > > > On Sun, Mar 26, 2023 at 5:06 PM Zbigniew Jędrzejewski-Szmek 
> > > > <zbys...@in.waw.pl> wrote:
> > > >>
> > > >> We should probably make it ^(/etc/|/bin/|/sbin/|/usr/bin/|/usr/sbin/). 
> > > >> Split-usr
> > > >> distros are a blast from the past, but adding the extra paths wouldn't 
> > > >> change
> > > >> anything for us, since we don't use them.
> > > >
> > > >
> > > > Please add /app/bin/ and /app/sbin/ as well to that list for flatpak 
> > > > rpm builds.
> > > >
> > >
> > > This shouldn't be needed because the buildsystem can happily pull
> > > filelists.xml. That said, this is part of the reason why we don't
> > > mandate a prefix in createrepo_c. We can't know what the prefix is
> > > going to be for RPMs for projects.
> >
> > If this is a difficult topic because different distros have different
> > expectations here, would it be possible to make this configurable
> > (either at runtime or at build time)?
> > That way createrepo_c wouldn't need to be opinionated ...
> >
> 
> Just don't anchor the regex with "^" the beginning. Then it works for 
> anything.
> 
> That is: "/bin/|/lib(|32|64|exec)/|/etc/"
> 
I don't think it's wise to match /usr/libexec which is defined as
<https://refspecs.linuxfoundation.org/FHS_3.0/fhs/ch04s07.html>:

    /usr/libexec includes internal binaries that are not intended to be
    executed directly by users or shell scripts.

-- Petr

Attachment: signature.asc
Description: PGP signature

_______________________________________________
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to