Re: [PATCH 0/2] tools: ynl: two patches to ease building with rpmbuild

From: Jakub Kicinski
Date: Mon Nov 11 2024 - 18:58:44 EST


On Mon, 11 Nov 2024 14:04:43 +0100 Jan Stancek wrote:
> I'm looking to build and package ynl for Fedora and Centos Stream users.

Great to hear!

> Default rpmbuild has couple hardening options enabled by default [1][2],
> which currently prevent ynl from building.

Could you rebase on:
https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git/
and resend? I see some fuzz:

Applying: tools: ynl: add script dir to sys.path
Using index info to reconstruct a base tree...
M tools/net/ynl/cli.py
M tools/net/ynl/ynl-gen-c.py
Falling back to patching base and 3-way merge...
Auto-merging tools/net/ynl/ynl-gen-c.py
Auto-merging tools/net/ynl/cli.py
Applying: tools: ynl: extend CFLAGS to keep options from environment


With that fixed feel free to add to the patches:

Acked-by: Jakub Kicinski <kuba@xxxxxxxxxx>


One thing I keep thinking about, maybe you already read this, is to
add some sort of spec search path and install the specs under /usr.
So the user can simply say --family X on the CLI without specifying
the fs full path to the YAML file. Would you be willing to send a patch
for this?