Hi,

what is the actual benefit of this? The image building recipes are hard enough
to understand as-is, is it really worth it to replace a plain "kernel-bin |
append-dtb | lzma" command sequence with yet another variable indirection just
to safe a few bytes in the Makefile?

I could understand the appeal of introducing a macro (or variable) if there's
a lot of repeated complex expressions in each KERNEL := ... recipe but that
isn't the case here.

If you really want to go down this route, then introducing an actual new build
step which aggregates the three steps into one would be cleaner imho.

Something similar to the following;

define Build/kernel-dtb-lzma
        $(call Build/kernel-bin)
        $(call Build/append-dtb)
        $(call Build/lzma)
endef

And then

KERNEL := kernel-dtb-lzma | uImage lzma


~ Jo

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to