Re: [PATCH net-next v3 09/13] doc/netlink: Regenerate netlink .rst files if ynl-gen-rst changes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Donald Hunter <donald.hunter@xxxxxxxxx>
Subject: Re: [PATCH net-next v3 09/13] doc/netlink: Regenerate netlink .rst files if ynl-gen-rst changes
From: Breno Leitao <leitao@xxxxxxxxxx>
Date: Tue, 12 Dec 2023 15:34:41 -0800
Cc: netdev@xxxxxxxxxxxxxxx, Jakub Kicinski <kuba@xxxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>, Eric Dumazet <edumazet@xxxxxxxxxx>, Paolo Abeni <pabeni@xxxxxxxxxx>, Jonathan Corbet <corbet@xxxxxxx>, linux-doc@xxxxxxxxxxxxxxx, Jacob Keller <jacob.e.keller@xxxxxxxxx>, donald.hunter@xxxxxxxxxx
In-reply-to: <20231212221552.3622-10-donald.hunter@gmail.com>
References: <20231212221552.3622-1-donald.hunter@gmail.com> <20231212221552.3622-10-donald.hunter@gmail.com>

> +$(YNL_INDEX): $(YNL_RST_FILES) $(YNL_TOOL)
> +	$(Q)$(YNL_TOOL) -o $@ -x

Isn't $(YNL_INDEX) depending to $(YNL_TOOL) indirectly since it depends
on $(YNL_RST_FILES) ?

I mean, do you really need the line above?

> +$(YNL_RST_DIR)/%.rst: $(YNL_YAML_DIR)/%.yaml $(YNL_TOOL)
> +	$(Q)$(YNL_TOOL) -i $< -o $@

Follow-Ups:
- Re: [PATCH net-next v3 09/13] doc/netlink: Regenerate netlink .rst files if ynl-gen-rst changes
  - From: Donald Hunter

References:
- [PATCH net-next v3 00/13] tools/net/ynl: Add 'sub-message' support to ynl
  - From: Donald Hunter
- [PATCH net-next v3 09/13] doc/netlink: Regenerate netlink .rst files if ynl-gen-rst changes
  - From: Donald Hunter

Prev by Date: Re: [PATCH] Documentation, mm/unaccepted: document accept_memory kernel parameter
Next by Date: Re: [PATCH net-next v3 13/13] tools/net/ynl-gen-rst: Remove extra indentation from generated docs
Previous by thread: [PATCH net-next v3 09/13] doc/netlink: Regenerate netlink .rst files if ynl-gen-rst changes
Next by thread: Re: [PATCH net-next v3 09/13] doc/netlink: Regenerate netlink .rst files if ynl-gen-rst changes
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Newbies] [Security] [Netfilter] [Bugtraq] [Linux FS] [Yosemite Forum] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Samba] [Video 4 Linux] [Device Mapper] [Linux Resources]