* [patch net-next] tools: ynl-gen: add "spec" arg to regen allowing to use only selected spec
@ 2023-08-22 11:50 Jiri Pirko
2023-08-22 17:09 ` Jakub Kicinski
0 siblings, 1 reply; 3+ messages in thread
From: Jiri Pirko @ 2023-08-22 11:50 UTC (permalink / raw)
To: netdev; +Cc: kuba, pabeni, davem, edumazet
From: Jiri Pirko <jiri@nvidia.com>
ynl-regen git greps and re-generates all generated files. Introduce a
command line argument "spec" allowing the user to specify only one yaml
spec file to use as a source of re-generation.
Example:
$ tools/net/ynl/ynl-regen.sh -s Documentation/netlink/specs/devlink.yaml -f
GEN kernel net/devlink/netlink_gen.c
GEN kernel net/devlink/netlink_gen.h
GEN user tools/net/ynl/generated/devlink-user.c
GEN user tools/net/ynl/generated/devlink-user.h
This is handy while working on a specific spec.
Signed-off-by: Jiri Pirko <jiri@nvidia.com>
---
tools/net/ynl/ynl-regen.sh | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/tools/net/ynl/ynl-regen.sh b/tools/net/ynl/ynl-regen.sh
index bdba24066cf1..ee77f6fcad60 100755
--- a/tools/net/ynl/ynl-regen.sh
+++ b/tools/net/ynl/ynl-regen.sh
@@ -5,11 +5,13 @@ TOOL=$(dirname $(realpath $0))/ynl-gen-c.py
force=
search=
+spec=
while [ ! -z "$1" ]; do
case "$1" in
-f ) force=yes; shift ;;
-p ) search=$2; shift 2 ;;
+ -s ) spec=$2; shift 2 ;;
* ) echo "Unrecognized option '$1'"; exit 1 ;;
esac
done
@@ -24,6 +26,10 @@ for f in $files; do
params=( $(git grep -B1 -h '/\* YNL-GEN' $f | sed 's@/\*\(.*\)\*/@\1@') )
args=$(sed -n 's@/\* YNL-ARG \(.*\) \*/@\1@p' $f)
+ if [ ! -z "$spec" -a "$spec" != "${params[0]}" ]; then
+ continue
+ fi
+
if [ $f -nt ${params[0]} -a -z "$force" ]; then
echo -e "\tSKIP $f"
continue
--
2.41.0
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [patch net-next] tools: ynl-gen: add "spec" arg to regen allowing to use only selected spec
2023-08-22 11:50 [patch net-next] tools: ynl-gen: add "spec" arg to regen allowing to use only selected spec Jiri Pirko
@ 2023-08-22 17:09 ` Jakub Kicinski
2023-08-22 17:41 ` Jiri Pirko
0 siblings, 1 reply; 3+ messages in thread
From: Jakub Kicinski @ 2023-08-22 17:09 UTC (permalink / raw)
To: Jiri Pirko; +Cc: netdev, pabeni, davem, edumazet
On Tue, 22 Aug 2023 13:50:00 +0200 Jiri Pirko wrote:
> $ tools/net/ynl/ynl-regen.sh -s Documentation/netlink/specs/devlink.yaml -f
touch Documentation/netlink/specs/devlink.yaml && ./tools/net/ynl/ynl-regen.sh
?
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [patch net-next] tools: ynl-gen: add "spec" arg to regen allowing to use only selected spec
2023-08-22 17:09 ` Jakub Kicinski
@ 2023-08-22 17:41 ` Jiri Pirko
0 siblings, 0 replies; 3+ messages in thread
From: Jiri Pirko @ 2023-08-22 17:41 UTC (permalink / raw)
To: Jakub Kicinski; +Cc: netdev, pabeni, davem, edumazet
Tue, Aug 22, 2023 at 07:09:40PM CEST, kuba@kernel.org wrote:
>On Tue, 22 Aug 2023 13:50:00 +0200 Jiri Pirko wrote:
>> $ tools/net/ynl/ynl-regen.sh -s Documentation/netlink/specs/devlink.yaml -f
>
>touch Documentation/netlink/specs/devlink.yaml && ./tools/net/ynl/ynl-regen.sh
Ah, leveraging the skip. Okay, makes sense.
>
>?
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-08-22 17:41 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-22 11:50 [patch net-next] tools: ynl-gen: add "spec" arg to regen allowing to use only selected spec Jiri Pirko
2023-08-22 17:09 ` Jakub Kicinski
2023-08-22 17:41 ` Jiri Pirko
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).