diff mbox

[resend] source-check: change behavior to check only selected packages / add source-check-all

Message ID cceaa854b98654b39b1f.1337107383@beantl019720
State Superseded
Headers show

Commit Message

Thomas De Schampheleire May 15, 2012, 6:43 p.m. UTC
The original 'source-check' target first selects all packages, and then checks
whether the package tarballs can be found. This is useful for Buildroot
maintainers, but less useful for developers working on a specific project. The
latter only care about the packages used in that project.

This patch removes the allyesconfig dependency to source-check so that only
selected packages are checked. The original behavior is moved to a new target
'source-check-all'.

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

---
 Makefile |  8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

Comments

Arnout Vandecappelle May 15, 2012, 8:45 p.m. UTC | #1
On 05/15/12 20:43, Thomas De Schampheleire wrote:
> The original 'source-check' target first selects all packages, and then checks
> whether the package tarballs can be found. This is useful for Buildroot
> maintainers, but less useful for developers working on a specific project. The
> latter only care about the packages used in that project.
>
> This patch removes the allyesconfig dependency to source-check so that only
> selected packages are checked. The original behavior is moved to a new target
> 'source-check-all'.
>
> Signed-off-by: Thomas De Schampheleire<thomas.de.schampheleire@gmail.com>

Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Luca Ceresoli May 16, 2012, 11:02 a.m. UTC | #2
Thomas De Schampheleire wrote:
> The original 'source-check' target first selects all packages, and then checks
> whether the package tarballs can be found. This is useful for Buildroot
> maintainers, but less useful for developers working on a specific project. The
> latter only care about the packages used in that project.
>
> This patch removes the allyesconfig dependency to source-check so that only
> selected packages are checked. The original behavior is moved to a new target
> 'source-check-all'.
>
> Signed-off-by: Thomas De Schampheleire<thomas.de.schampheleire@gmail.com>
>

Acked-by: Luca Ceresoli <luca@lucaceresoli.net>

Luca
Thomas Petazzoni May 16, 2012, 11:41 a.m. UTC | #3
Le Tue, 15 May 2012 20:43:03 +0200,
Thomas De Schampheleire <patrickdepinguin+buildroot@gmail.com> a écrit :

> The original 'source-check' target first selects all packages, and then checks
> whether the package tarballs can be found. This is useful for Buildroot
> maintainers, but less useful for developers working on a specific project. The
> latter only care about the packages used in that project.
> 
> This patch removes the allyesconfig dependency to source-check so that only
> selected packages are checked. The original behavior is moved to a new target
> 'source-check-all'.
> 
> Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>

Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
diff mbox

Patch

diff --git a/Makefile b/Makefile
--- a/Makefile
+++ b/Makefile
@@ -48,7 +48,7 @@  export BR2_VERSION_FULL:=$(BR2_VERSION)$
 noconfig_targets:=menuconfig nconfig gconfig xconfig config oldconfig randconfig \
 	defconfig %_defconfig savedefconfig allyesconfig allnoconfig silentoldconfig release \
 	randpackageconfig allyespackageconfig allnopackageconfig \
-	source-check print-version
+	source-check source-check-all print-version
 
 # Strip quotes and then whitespaces
 qstrip=$(strip $(subst ",,$(1)))
@@ -584,7 +584,8 @@  savedefconfig: $(BUILD_DIR)/buildroot-co
 	@$(COMMON_CONFIG_ENV) $< --savedefconfig=$(CONFIG_DIR)/defconfig $(CONFIG_CONFIG_IN)
 
 # check if download URLs are outdated
-source-check: allyesconfig
+source-check-all: allyesconfig source-check
+source-check:
 	$(MAKE) DL_MODE=SOURCE_CHECK $(EXTRAMAKEARGS) source
 
 endif # ifeq ($(BR2_HAVE_DOT_CONFIG),y)
@@ -671,7 +672,8 @@  endif
 	@echo
 	@echo 'Miscellaneous:'
 	@echo '  source                 - download all sources needed for offline-build'
-	@echo '  source-check           - check all packages for valid download URLs'
+	@echo '  source-check           - check selected packages for valid download URLs'
+	@echo '  source-check-all       - check all packages for valid download URLs'
 	@echo '  external-deps          - list external packages used'
 	@echo
 	@echo '  make V=0|1             - 0 => quiet build (default), 1 => verbose build'