All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] %_defconfig: save the path to the defconfig
@ 2015-02-02 15:34 Arnout Vandecappelle
  2015-02-02 16:22 ` Thomas De Schampheleire
  2015-02-02 17:04 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Arnout Vandecappelle @ 2015-02-02 15:34 UTC (permalink / raw)
  To: buildroot

We have the BR2_DEFCONFIG feature that saves the path to the defconfig
file that was specified on the command line, so that a later
savedefconfig would immediately save to the right location. This wasn't
done for the defconfigs in the configs/ directory, however, to avoid
accidentally overwriting them.

Now we decided that it would be more useful to overwrite the defconfigs
in the configs/ directory after all. To do this, we pass the path to
that defconfig in the environment.

Cc: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 Makefile | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/Makefile b/Makefile
index dd2b7b6..2192efd 100644
--- a/Makefile
+++ b/Makefile
@@ -781,13 +781,16 @@ defconfig: $(BUILD_DIR)/buildroot-config/conf outputmakefile
 	@mkdir -p $(BUILD_DIR)/buildroot-config
 	@$(COMMON_CONFIG_ENV) $< --defconfig$(if $(DEFCONFIG),=$(DEFCONFIG)) $(CONFIG_CONFIG_IN)
 
+# Override the BR2_DEFCONFIG from COMMON_CONFIG_ENV with the new defconfig
 %_defconfig: $(BUILD_DIR)/buildroot-config/conf $(TOPDIR)/configs/%_defconfig outputmakefile
 	@mkdir -p $(BUILD_DIR)/buildroot-config
-	@$(COMMON_CONFIG_ENV) $< --defconfig=$(TOPDIR)/configs/$@ $(CONFIG_CONFIG_IN)
+	@$(COMMON_CONFIG_ENV) BR2_DEFCONFIG=$(TOPDIR)/configs/$@ \
+		$< --defconfig=$(TOPDIR)/configs/$@ $(CONFIG_CONFIG_IN)
 
 %_defconfig: $(BUILD_DIR)/buildroot-config/conf $(BR2_EXTERNAL)/configs/%_defconfig outputmakefile
 	@mkdir -p $(BUILD_DIR)/buildroot-config
-	@$(COMMON_CONFIG_ENV) $< --defconfig=$(BR2_EXTERNAL)/configs/$@ $(CONFIG_CONFIG_IN)
+	@$(COMMON_CONFIG_ENV) BR2_DEFCONFIG=$(BR2_EXTERNAL)/configs/$@ \
+		$< --defconfig=$(BR2_EXTERNAL)/configs/$@ $(CONFIG_CONFIG_IN)
 
 savedefconfig: $(BUILD_DIR)/buildroot-config/conf outputmakefile
 	@mkdir -p $(BUILD_DIR)/buildroot-config
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH] %_defconfig: save the path to the defconfig
  2015-02-02 15:34 [Buildroot] [PATCH] %_defconfig: save the path to the defconfig Arnout Vandecappelle
@ 2015-02-02 16:22 ` Thomas De Schampheleire
  2015-02-02 17:04 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas De Schampheleire @ 2015-02-02 16:22 UTC (permalink / raw)
  To: buildroot

On Mon, Feb 2, 2015 at 4:34 PM, Arnout Vandecappelle (Essensium/Mind)
<arnout@mind.be> wrote:
> We have the BR2_DEFCONFIG feature that saves the path to the defconfig
> file that was specified on the command line, so that a later
> savedefconfig would immediately save to the right location. This wasn't
> done for the defconfigs in the configs/ directory, however, to avoid
> accidentally overwriting them.
>
> Now we decided that it would be more useful to overwrite the defconfigs
> in the configs/ directory after all. To do this, we pass the path to
> that defconfig in the environment.
>
> Cc: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

Tested-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH] %_defconfig: save the path to the defconfig
  2015-02-02 15:34 [Buildroot] [PATCH] %_defconfig: save the path to the defconfig Arnout Vandecappelle
  2015-02-02 16:22 ` Thomas De Schampheleire
@ 2015-02-02 17:04 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2015-02-02 17:04 UTC (permalink / raw)
  To: buildroot

>>>>> "Arnout" == Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> writes:

 > We have the BR2_DEFCONFIG feature that saves the path to the defconfig
 > file that was specified on the command line, so that a later
 > savedefconfig would immediately save to the right location. This wasn't
 > done for the defconfigs in the configs/ directory, however, to avoid
 > accidentally overwriting them.

 > Now we decided that it would be more useful to overwrite the defconfigs
 > in the configs/ directory after all. To do this, we pass the path to
 > that defconfig in the environment.

 > Cc: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
 > Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

Committed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-02-02 17:04 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-02 15:34 [Buildroot] [PATCH] %_defconfig: save the path to the defconfig Arnout Vandecappelle
2015-02-02 16:22 ` Thomas De Schampheleire
2015-02-02 17:04 ` Peter Korsgaard

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.