Hi Bastian, Roland, thanks for the quick response and comments :)
On 20-06-16 10:34, Bastian Krause wrote: > > On 6/16/20 10:02 AM, Roland Hieber wrote: > > On Tue, Jun 16, 2020 at 12:56:18AM +0200, Marco Felsch wrote: > >> Allow users to specify other barebox image naming schemes. Now users can > >> specify a list of: > >> - unique image names and/or > >> - matching name patterns using wildcards. > >> > >> While on it I fixed the leaking image-results upon a 'ptxdist clean > >> barebox' command. Now all barebox artefacts are removed from the > >> $(IMAGEDIR). > >> > >> Signed-off-by: Marco Felsch <[email protected]> > >> --- > > > > LGTM on first glance, could you do the same for > > rules/templates/template-barebox-make too please? > > Is this really necessary? The idea of the barebox rule template is to > generate a BSP specific barebox rule, right? We already have.. > > BAREBOX_@PACKAGE@_IMAGES := @image@ > > ..in that template, so we don't really benefit from a config option here. My idea was to _hopefully_ remove the usage of this template and instead use ptxdist upstream mechanism: - layer -> different barebox configs - this patch -> different barebox images Regards, Marco > Regards, > Bastian > > > > > - Roland > > > >> platforms/barebox.in | 9 +++++++++ > >> rules/barebox.make | 19 +++++++++++++------ > >> 2 files changed, 22 insertions(+), 6 deletions(-) > >> > >> diff --git a/platforms/barebox.in b/platforms/barebox.in > >> index 526d7ede4..b92cf485e 100644 > >> --- a/platforms/barebox.in > >> +++ b/platforms/barebox.in > >> @@ -39,6 +39,15 @@ config BAREBOX_CONFIG > >> This entry specifies the .config file used to compile > >> barebox. > >> > >> +config BAREBOX_IMAGES > >> + prompt "barebox images to install" > >> + string > >> + default "barebox-*.img" > >> + help > >> + A comma seperated list of images which should be installed > >> + into the $(IMAGEDIR). The list can contain full image names > >> + and/or wildcard pattern image names. > >> + > >> config BAREBOX_EXTRA_ENV > >> prompt "extend the builtin barebox environment" > >> bool > >> diff --git a/rules/barebox.make b/rules/barebox.make > >> index d1b5fc598..65bb8251e 100644 > >> --- a/rules/barebox.make > >> +++ b/rules/barebox.make > >> @@ -148,6 +148,9 @@ $(STATEDIR)/barebox.install: > >> # Target-Install > >> # > >> ---------------------------------------------------------------------------- > >> > >> +BAREBOX_IMAGES_DIR := $(BAREBOX_BUILD_DIR)/images > >> +BAREBOX_IMAGES := $(strip $(subst > >> $(ptx/def/comma),$(ptx/def/space),$(call remove_quotes, > >> $(PTXCONF_BAREBOX_IMAGES)))) > >> + > >> $(STATEDIR)/barebox.targetinstall: > >> @$(call targetinfo) > >> > >> @@ -166,14 +169,15 @@ ifneq ($(strip $(BAREBOX_PROGS_TARGET_y)),) > >> endif > >> > >> @rm -f $(IMAGEDIR)/barebox-image > >> - @if [ -d $(BAREBOX_BUILD_DIR)/images ]; then \ > >> - find $(BAREBOX_BUILD_DIR)/images/ -name "barebox-*.img" | sort > >> | while read image; do \ > >> - install -D -m644 $$image $(IMAGEDIR)/`basename > >> $$image`; \ > >> + @$(foreach _image, $(BAREBOX_IMAGES), \ > >> + $(foreach image, $(wildcard $(BAREBOX_IMAGES_DIR)/$(_image)), \ > >> + install -D -m644 $(image) $(IMAGEDIR)/$(notdir > >> $(image)); \ > >> if [ ! -e $(IMAGEDIR)/barebox-image ]; then \ > >> - ln -sf `basename $$image` > >> $(IMAGEDIR)/barebox-image; \ > >> + ln -sf $(notdir $(image)) > >> $(IMAGEDIR)/barebox-image; \ > >> fi; \ > >> - done; \ > >> - fi > >> + ) \ > >> + ) > >> + > >> @if [ -e $(IMAGEDIR)/barebox-image ]; then \ > >> :; \ > >> elif [ -e $(BAREBOX_BUILD_DIR)/barebox-flash-image ]; then \ > >> @@ -199,6 +203,9 @@ $(STATEDIR)/barebox.clean: > >> @$(call clean_pkg, BAREBOX) > >> @$(foreach prog, $(BAREBOX_PROGS_HOST), \ > >> rm -vf $(PTXDIST_SYSROOT_HOST)/bin/$(notdir $(prog))$(ptx/nl)) > >> + @$(foreach _image, $(BAREBOX_IMAGES), \ > >> + $(foreach image, $(wildcard $(IMAGEDIR)/$(_image)), \ > >> + rm -vf $(image);)) > >> @rm -vf $(IMAGEDIR)/barebox-image > >> $(IMAGEDIR)/barebox-default-environment > >> > >> # > >> ---------------------------------------------------------------------------- > >> -- > >> 2.20.1 > >> > >> > >> _______________________________________________ > >> ptxdist mailing list > >> [email protected] > >> To unsubscribe, send a mail with subject "unsubscribe" to > >> [email protected] > >> > > > > > -- > Pengutronix e.K. | | > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | > > _______________________________________________ > ptxdist mailing list > [email protected] > To unsubscribe, send a mail with subject "unsubscribe" to > [email protected] > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list [email protected] To unsubscribe, send a mail with subject "unsubscribe" to [email protected]
