From patchwork Mon Apr 7 09:56:47 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Fabio Porcedda X-Patchwork-Id: 337436 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from silver.osuosl.org (silver.osuosl.org [140.211.166.136]) by ozlabs.org (Postfix) with ESMTP id 6000D1400E1 for ; Mon, 7 Apr 2014 19:57:06 +1000 (EST) Received: from localhost (localhost [127.0.0.1]) by silver.osuosl.org (Postfix) with ESMTP id 7265230921; Mon, 7 Apr 2014 09:57:05 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from silver.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ZAX15u5onNdi; Mon, 7 Apr 2014 09:56:59 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by silver.osuosl.org (Postfix) with ESMTP id 2A14430981; Mon, 7 Apr 2014 09:56:59 +0000 (UTC) X-Original-To: buildroot@lists.busybox.net Delivered-To: buildroot@osuosl.org Received: from whitealder.osuosl.org (whitealder.osuosl.org [140.211.166.138]) by ash.osuosl.org (Postfix) with ESMTP id 0BC881BF9E7 for ; Mon, 7 Apr 2014 09:56:58 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by whitealder.osuosl.org (Postfix) with ESMTP id 074968B2AB for ; Mon, 7 Apr 2014 09:56:58 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from whitealder.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ke8vQF2YbJoc for ; Mon, 7 Apr 2014 09:56:56 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.7.6 Received: from mail-pd0-f182.google.com (mail-pd0-f182.google.com [209.85.192.182]) by whitealder.osuosl.org (Postfix) with ESMTPS id 602618B2A0 for ; Mon, 7 Apr 2014 09:56:56 +0000 (UTC) Received: by mail-pd0-f182.google.com with SMTP id y10so6349737pdj.27 for ; Mon, 07 Apr 2014 02:56:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:subject:date:message-id:in-reply-to:references; bh=OXHksam9MgFZVHYMyBCNv7fw3ecmhTgi3dbzBqccQA4=; b=CZL8OCtA7uKW8l9+UviiMXYejmetF9ajJsYIKgh+fVAZsV3/QjRpXOfnbv4QruVgsk FFnCBlqbwc0HqOAPpM12MFdudAQQ3xRNI5QfWQ2SyjpCgDfVY4S9nRXUEjH8sv0DsAOm X6W2jN3vNF2axmhyAy92jpvzPPE71Rh/muyOxVJkgtcEN/P9OUj+hAjuQyjkhdac4aF+ j3Dx69HOQyFdUCw+OP1GRArW+UL8FvzlJ/rH3o3PZ0zgReUbTnsBmraKkiCJlBWaKAE3 LDPoC/T6aChgvX8yo3NY2zLAZAL6LNj2HoHNSx8xSV3wesOvsoWcj9FaeB20nuwauc2r SXKg== X-Received: by 10.66.65.134 with SMTP id x6mr29952832pas.12.1396864616161; Mon, 07 Apr 2014 02:56:56 -0700 (PDT) Received: from ld2077.tmt.telital.com ([213.205.6.118]) by mx.google.com with ESMTPSA id bc4sm35673499pbb.2.2014.04.07.02.56.54 for (version=TLSv1.1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Mon, 07 Apr 2014 02:56:55 -0700 (PDT) From: Fabio Porcedda To: buildroot@uclibc.org Date: Mon, 7 Apr 2014 11:56:47 +0200 Message-Id: <1396864610-8426-2-git-send-email-fabio.porcedda@gmail.com> X-Mailer: git-send-email 1.8.3.2 In-Reply-To: <1396864610-8426-1-git-send-email-fabio.porcedda@gmail.com> References: <1396864610-8426-1-git-send-email-fabio.porcedda@gmail.com> Subject: [Buildroot] [PATCH v4 1/4] Makefile: fix coding style X-BeenThere: buildroot@busybox.net X-Mailman-Version: 2.1.14 Precedence: list List-Id: Discussion and development of buildroot List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: buildroot-bounces@busybox.net Sender: buildroot-bounces@busybox.net As stated in the buildroot user manual add just a single space before and after a '=' sign. Signed-off-by: Fabio Porcedda Acked-by: Arnout Vandecappelle (Essensium/Mind) --- Makefile | 184 +++++++++++++++++++++++++++++++-------------------------------- 1 file changed, 92 insertions(+), 92 deletions(-) diff --git a/Makefile b/Makefile index 6363ec3..82f2763 100644 --- a/Makefile +++ b/Makefile @@ -25,10 +25,10 @@ #-------------------------------------------------------------- # Set and export the version string -export BR2_VERSION:=2014.05-git +export BR2_VERSION := 2014.05-git # Check for minimal make version (note: this check will break at make 10.x) -MIN_MAKE_VERSION=3.81 +MIN_MAKE_VERSION = 3.81 ifneq ($(firstword $(sort $(MAKE_VERSION) $(MIN_MAKE_VERSION))),$(MIN_MAKE_VERSION)) $(error You have make '$(MAKE_VERSION)' installed. GNU make >= $(MIN_MAKE_VERSION) is required) endif @@ -59,33 +59,33 @@ export HOSTARCH := $(shell uname -m | \ # make BR2_JLEVEL= -j$((`getconf _NPROCESSORS_ONLN`+1)) # absolute path -TOPDIR:=$(shell pwd) -CONFIG_CONFIG_IN=Config.in -CONFIG=support/kconfig -DATE:=$(shell date +%Y%m%d) +TOPDIR := $(shell pwd) +CONFIG_CONFIG_IN = Config.in +CONFIG = support/kconfig +DATE := $(shell date +%Y%m%d) # Compute the full local version string so packages can use it as-is # Need to export it, so it can be got from environment in children (eg. mconf) -export BR2_VERSION_FULL:=$(BR2_VERSION)$(shell $(TOPDIR)/support/scripts/setlocalversion) +export BR2_VERSION_FULL := $(BR2_VERSION)$(shell $(TOPDIR)/support/scripts/setlocalversion) -noconfig_targets:=menuconfig nconfig gconfig xconfig config oldconfig randconfig \ +noconfig_targets := menuconfig nconfig gconfig xconfig config oldconfig randconfig \ defconfig %_defconfig allyesconfig allnoconfig silentoldconfig release \ randpackageconfig allyespackageconfig allnopackageconfig \ source-check print-version olddefconfig # Strip quotes and then whitespaces -qstrip=$(strip $(subst ",,$(1))) +qstrip = $(strip $(subst ",,$(1))) #")) # Variables for use in Make constructs -comma:=, -empty:= -space:=$(empty) $(empty) +comma := , +empty := +space := $(empty) $(empty) ifneq ("$(origin O)", "command line") -O:=output -CONFIG_DIR:=$(TOPDIR) -NEED_WRAPPER= +O := output +CONFIG_DIR := $(TOPDIR) +NEED_WRAPPER = else # other packages might also support Linux-style out of tree builds # with the O= syntax (E.G. Busybox does). As make automatically @@ -97,18 +97,18 @@ MAKEOVERRIDES = # of the origin function (command line -> environment). # Unfortunately some packages don't look at origin (E.G. uClibc 0.9.31+) # To really make O go away, we have to override it. -override O:=$(O) -CONFIG_DIR:=$(O) +override O := $(O) +CONFIG_DIR := $(O) # we need to pass O= everywhere we call back into the toplevel makefile EXTRAMAKEARGS = O=$(O) -NEED_WRAPPER=y +NEED_WRAPPER = y endif # bash prints the name of the directory on 'cd ' if CDPATH is # set, so unset it here to not cause problems. Notice that the export # line doesn't affect the environment of $(shell ..) calls, so # explictly throw away any output from 'cd' here. -export CDPATH:= +export CDPATH := BASE_DIR := $(shell mkdir -p $(O) && cd $(O) >/dev/null && pwd) $(if $(BASE_DIR),, $(error output directory "$(O)" does not exist)) @@ -150,26 +150,26 @@ endif # Avoids doing the $(or...) everytime _BR2_GRAPH_OUT := $(or $(BR2_GRAPH_OUT),pdf) -BUILD_DIR:=$(BASE_DIR)/build -BINARIES_DIR:=$(BASE_DIR)/images -TARGET_DIR:=$(BASE_DIR)/target +BUILD_DIR := $(BASE_DIR)/build +BINARIES_DIR := $(BASE_DIR)/images +TARGET_DIR := $(BASE_DIR)/target # initial definition so that 'make clean' works for most users, even without # .config. HOST_DIR will be overwritten later when .config is included. -HOST_DIR:=$(BASE_DIR)/host - -LEGAL_INFO_DIR=$(BASE_DIR)/legal-info -REDIST_SOURCES_DIR_TARGET=$(LEGAL_INFO_DIR)/sources -REDIST_SOURCES_DIR_HOST=$(LEGAL_INFO_DIR)/host-sources -LICENSE_FILES_DIR_TARGET=$(LEGAL_INFO_DIR)/licenses -LICENSE_FILES_DIR_HOST=$(LEGAL_INFO_DIR)/host-licenses -LEGAL_MANIFEST_CSV_TARGET=$(LEGAL_INFO_DIR)/manifest.csv -LEGAL_MANIFEST_CSV_HOST=$(LEGAL_INFO_DIR)/host-manifest.csv -LEGAL_LICENSES_TXT_TARGET=$(LEGAL_INFO_DIR)/licenses.txt -LEGAL_LICENSES_TXT_HOST=$(LEGAL_INFO_DIR)/host-licenses.txt -LEGAL_WARNINGS=$(LEGAL_INFO_DIR)/.warnings -LEGAL_REPORT=$(LEGAL_INFO_DIR)/README - -BR2_CONFIG=$(CONFIG_DIR)/.config +HOST_DIR := $(BASE_DIR)/host + +LEGAL_INFO_DIR = $(BASE_DIR)/legal-info +REDIST_SOURCES_DIR_TARGET = $(LEGAL_INFO_DIR)/sources +REDIST_SOURCES_DIR_HOST = $(LEGAL_INFO_DIR)/host-sources +LICENSE_FILES_DIR_TARGET = $(LEGAL_INFO_DIR)/licenses +LICENSE_FILES_DIR_HOST = $(LEGAL_INFO_DIR)/host-licenses +LEGAL_MANIFEST_CSV_TARGET = $(LEGAL_INFO_DIR)/manifest.csv +LEGAL_MANIFEST_CSV_HOST = $(LEGAL_INFO_DIR)/host-manifest.csv +LEGAL_LICENSES_TXT_TARGET = $(LEGAL_INFO_DIR)/licenses.txt +LEGAL_LICENSES_TXT_HOST = $(LEGAL_INFO_DIR)/host-licenses.txt +LEGAL_WARNINGS = $(LEGAL_INFO_DIR)/.warnings +LEGAL_REPORT = $(LEGAL_INFO_DIR)/README + +BR2_CONFIG = $(CONFIG_DIR)/.config # Pull in the user's configuration file ifeq ($(filter $(noconfig_targets),$(MAKECMDGOALS)),) @@ -180,22 +180,22 @@ endif # Use 'make V=1' to see the full commands ifdef V ifeq ("$(origin V)", "command line") - KBUILD_VERBOSE=$(V) + KBUILD_VERBOSE = $(V) endif endif ifndef KBUILD_VERBOSE - KBUILD_VERBOSE=0 + KBUILD_VERBOSE = 0 endif ifeq ($(KBUILD_VERBOSE),1) - quiet= - Q= + quiet = + Q = ifndef VERBOSE - VERBOSE=1 + VERBOSE = 1 endif else - quiet=quiet_ - Q=@ + quiet = quiet_ + Q = @ endif # we want bash as shell @@ -204,56 +204,56 @@ SHELL:=$(shell if [ -x "$$BASH" ]; then echo $$BASH; \ else echo sh; fi; fi) # kconfig uses CONFIG_SHELL -CONFIG_SHELL:=$(SHELL) +CONFIG_SHELL := $(SHELL) export SHELL CONFIG_SHELL quiet Q KBUILD_VERBOSE VERBOSE ifndef HOSTAR -HOSTAR:=ar +HOSTAR := ar endif ifndef HOSTAS HOSTAS:=as endif ifndef HOSTCC -HOSTCC:=gcc -HOSTCC:=$(shell which $(HOSTCC) || type -p $(HOSTCC) || echo gcc) +HOSTCC := gcc +HOSTCC := $(shell which $(HOSTCC) || type -p $(HOSTCC) || echo gcc) endif HOSTCC_NOCCACHE:=$(HOSTCC) ifndef HOSTCXX -HOSTCXX:=g++ -HOSTCXX:=$(shell which $(HOSTCXX) || type -p $(HOSTCXX) || echo g++) +HOSTCXX := g++ +HOSTCXX := $(shell which $(HOSTCXX) || type -p $(HOSTCXX) || echo g++) endif -HOSTCXX_NOCCACHE:=$(HOSTCXX) +HOSTCXX_NOCCACHE := $(HOSTCXX) ifndef HOSTFC -HOSTFC:=gfortran +HOSTFC := gfortran endif ifndef HOSTCPP HOSTCPP:=cpp endif ifndef HOSTLD -HOSTLD:=ld +HOSTLD := ld endif ifndef HOSTLN -HOSTLN:=ln +HOSTLN := ln endif ifndef HOSTNM -HOSTNM:=nm +HOSTNM := nm endif ifndef HOSTOBJCOPY -HOSTOBJCOPY:=objcopy +HOSTOBJCOPY := objcopy endif ifndef HOSTRANLIB -HOSTRANLIB:=ranlib -endif -HOSTAR:=$(shell which $(HOSTAR) || type -p $(HOSTAR) || echo ar) -HOSTAS:=$(shell which $(HOSTAS) || type -p $(HOSTAS) || echo as) -HOSTFC:=$(shell which $(HOSTLD) || type -p $(HOSTLD) || echo || which g77 || type -p g77 || echo gfortran) -HOSTCPP:=$(shell which $(HOSTCPP) || type -p $(HOSTCPP) || echo cpp) -HOSTLD:=$(shell which $(HOSTLD) || type -p $(HOSTLD) || echo ld) -HOSTLN:=$(shell which $(HOSTLN) || type -p $(HOSTLN) || echo ln) -HOSTNM:=$(shell which $(HOSTNM) || type -p $(HOSTNM) || echo nm) -HOSTOBJCOPY:=$(shell which $(HOSTOBJCOPY) || type -p $(HOSTOBJCOPY) || echo objcopy) -HOSTRANLIB:=$(shell which $(HOSTRANLIB) || type -p $(HOSTRANLIB) || echo ranlib) +HOSTRANLIB := ranlib +endif +HOSTAR := $(shell which $(HOSTAR) || type -p $(HOSTAR) || echo ar) +HOSTAS := $(shell which $(HOSTAS) || type -p $(HOSTAS) || echo as) +HOSTFC := $(shell which $(HOSTLD) || type -p $(HOSTLD) || echo || which g77 || type -p g77 || echo gfortran) +HOSTCPP := $(shell which $(HOSTCPP) || type -p $(HOSTCPP) || echo cpp) +HOSTLD := $(shell which $(HOSTLD) || type -p $(HOSTLD) || echo ld) +HOSTLN := $(shell which $(HOSTLN) || type -p $(HOSTLN) || echo ln) +HOSTNM := $(shell which $(HOSTNM) || type -p $(HOSTNM) || echo nm) +HOSTOBJCOPY := $(shell which $(HOSTOBJCOPY) || type -p $(HOSTOBJCOPY) || echo objcopy) +HOSTRANLIB := $(shell which $(HOSTRANLIB) || type -p $(HOSTRANLIB) || echo ranlib) export HOSTAR HOSTAS HOSTCC HOSTCXX HOSTFC HOSTLD export HOSTCC_NOCCACHE HOSTCXX_NOCCACHE @@ -290,7 +290,7 @@ unexport CONFIG_SITE unexport QMAKESPEC unexport TERMINFO -GNU_HOST_NAME:=$(shell support/gnuconfig/config.guess) +GNU_HOST_NAME := $(shell support/gnuconfig/config.guess) ################################################################################ # @@ -302,12 +302,12 @@ GNU_HOST_NAME:=$(shell support/gnuconfig/config.guess) TARGETS:= toolchain # silent mode requested? -QUIET:=$(if $(findstring s,$(MAKEFLAGS)),-q) +QUIET := $(if $(findstring s,$(MAKEFLAGS)),-q) # Strip off the annoying quoting -ARCH:=$(call qstrip,$(BR2_ARCH)) +ARCH := $(call qstrip,$(BR2_ARCH)) -KERNEL_ARCH:=$(shell echo "$(ARCH)" | sed -e "s/-.*//" \ +KERNEL_ARCH := $(shell echo "$(ARCH)" | sed -e "s/-.*//" \ -e s/i.86/i386/ -e s/sun4u/sparc64/ \ -e s/arcle/arc/ \ -e s/arceb/arc/ \ @@ -320,28 +320,28 @@ KERNEL_ARCH:=$(shell echo "$(ARCH)" | sed -e "s/-.*//" \ -e s/sh.*/sh/ \ -e s/microblazeel/microblaze/) -ZCAT:=$(call qstrip,$(BR2_ZCAT)) -BZCAT:=$(call qstrip,$(BR2_BZCAT)) -XZCAT:=$(call qstrip,$(BR2_XZCAT)) -TAR_OPTIONS=$(call qstrip,$(BR2_TAR_OPTIONS)) -xf +ZCAT := $(call qstrip,$(BR2_ZCAT)) +BZCAT := $(call qstrip,$(BR2_BZCAT)) +XZCAT := $(call qstrip,$(BR2_XZCAT)) +TAR_OPTIONS = $(call qstrip,$(BR2_TAR_OPTIONS)) -xf # packages compiled for the host go here -HOST_DIR:=$(call qstrip,$(BR2_HOST_DIR)) +HOST_DIR := $(call qstrip,$(BR2_HOST_DIR)) # locales to generate -GENERATE_LOCALE=$(call qstrip,$(BR2_GENERATE_LOCALE)) +GENERATE_LOCALE = $(call qstrip,$(BR2_GENERATE_LOCALE)) -TARGET_SKELETON=$(TOPDIR)/system/skeleton +TARGET_SKELETON = $(TOPDIR)/system/skeleton # Location of a file giving a big fat warning that output/target # should not be used as the root filesystem. -TARGET_DIR_WARNING_FILE=$(TARGET_DIR)/THIS_IS_NOT_YOUR_ROOT_FILESYSTEM +TARGET_DIR_WARNING_FILE = $(TARGET_DIR)/THIS_IS_NOT_YOUR_ROOT_FILESYSTEM ifeq ($(BR2_CCACHE),y) -CCACHE:=$(HOST_DIR)/usr/bin/ccache +CCACHE := $(HOST_DIR)/usr/bin/ccache BR_CACHE_DIR = $(call qstrip,$(BR2_CCACHE_DIR)) export BR_CACHE_DIR -HOSTCC := $(CCACHE) $(HOSTCC) +HOSTCC := $(CCACHE) $(HOSTCC) HOSTCXX := $(CCACHE) $(HOSTCXX) endif @@ -394,23 +394,23 @@ include system/system.mk include $(BR2_EXTERNAL)/external.mk ifeq ($(BR2_ENABLE_LOCALE_PURGE),y) -TARGETS+=target-purgelocales +TARGETS += target-purgelocales endif ifeq ($(BR2_TOOLCHAIN_USES_GLIBC),y) ifneq ($(GENERATE_LOCALE),) -TARGETS+=target-generatelocales +TARGETS += target-generatelocales endif endif ifeq ($(BR2_ECLIPSE_REGISTER),y) -TARGETS+=toolchain-eclipse-register +TARGETS += toolchain-eclipse-register endif include fs/common.mk -TARGETS_SOURCE:=$(patsubst %,%-source,$(TARGETS)) -TARGETS_DIRCLEAN:=$(patsubst %,%-dirclean,$(TARGETS)) +TARGETS_SOURCE := $(patsubst %,%-source,$(TARGETS)) +TARGETS_DIRCLEAN := $(patsubst %,%-dirclean,$(TARGETS)) # host-* dependencies have to be handled specially, as those aren't # visible in Kconfig and hence not added to a variable like TARGETS. @@ -431,7 +431,7 @@ HOST_DEPS = $(sort $(foreach dep,\ $($(dep)))) HOST_SOURCE += $(addsuffix -source,$(sort $(TARGETS_HOST_DEPS) $(HOST_DEPS))) -TARGETS_LEGAL_INFO:=$(patsubst %,%-legal-info,\ +TARGETS_LEGAL_INFO := $(patsubst %,%-legal-info,\ $(TARGETS) $(TARGETS_HOST_DEPS) $(HOST_DEPS)))) dirs: $(BUILD_DIR) $(STAGING_DIR) $(TARGET_DIR) \ @@ -484,7 +484,7 @@ $(STAGING_DIR): @ln -snf $(STAGING_DIR) $(BASE_DIR)/staging ifeq ($(BR2_ROOTFS_SKELETON_CUSTOM),y) -TARGET_SKELETON=$(BR2_ROOTFS_SKELETON_CUSTOM_PATH) +TARGET_SKELETON = $(BR2_ROOTFS_SKELETON_CUSTOM_PATH) endif RSYNC_VCS_EXCLUSIONS = \ @@ -578,8 +578,8 @@ endif $(USER_HOOKS_EXTRA_ENV) $(s) $(TARGET_DIR) $(call qstrip,$(BR2_ROOTFS_POST_SCRIPT_ARGS))$(sep)) ifeq ($(BR2_ENABLE_LOCALE_PURGE),y) -LOCALE_WHITELIST=$(BUILD_DIR)/locales.nopurge -LOCALE_NOPURGE=$(call qstrip,$(BR2_ENABLE_LOCALE_WHITELIST)) +LOCALE_WHITELIST = $(BUILD_DIR)/locales.nopurge +LOCALE_NOPURGE = $(call qstrip,$(BR2_ENABLE_LOCALE_WHITELIST)) target-purgelocales: rm -f $(LOCALE_WHITELIST) @@ -683,7 +683,7 @@ endif # ifeq ($(BR2_HAVE_DOT_CONFIG),y) # configuration # --------------------------------------------------------------------------- -HOSTCFLAGS=$(CFLAGS_FOR_BUILD) +HOSTCFLAGS = $(CFLAGS_FOR_BUILD) export HOSTCFLAGS $(BUILD_DIR)/buildroot-config/%onf: @@ -912,7 +912,7 @@ endif @echo 'See docs/README, or generate the Buildroot manual for further details' @echo -release: OUT=buildroot-$(BR2_VERSION) +release: OUT = buildroot-$(BR2_VERSION) # Create release tarballs. We need to fiddle a bit to add the generated # documentation to the git output