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 <decathorpe@xxxxxxxxx> wrote: > > > > On Mon, Mar 27, 2023 at 2:10 PM Neal Gompa <ngompa13@xxxxxxxxx> wrote: > > > > > > On Sun, Mar 26, 2023 at 12:27 PM Kalev Lember <kalevlember@xxxxxxxxx> wrote: > > > > > > > > On Sun, Mar 26, 2023 at 5:06 PM Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> 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@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx 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@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue