From: Philip Prindeville Date: Thu, 25 Jan 2018 00:33:21 +0000 (-0700) Subject: treewide: combine VERSION_SED and VERSION_SED_SCRIPT X-Git-Url: http://git.cdn.openwrt.org/?a=commitdiff_plain;h=ff8e9a4ecbe70564c8649ea4ed69f28bec77dfa7;p=openwrt%2Fstaging%2Fblogic.git treewide: combine VERSION_SED and VERSION_SED_SCRIPT We don't need two versions of this. The escaping quotes is so that the sed commands aren't misinterpreted by shell; it has nothing to do with the contents of the file, thus one version is adequate. Signed-off-by: Philip Prindeville --- diff --git a/include/version.mk b/include/version.mk index 6c6eb75ff66d..ab427ecba897 100644 --- a/include/version.mk +++ b/include/version.mk @@ -85,7 +85,7 @@ $(subst &,\&,$(subst $(comma),\$(comma),$(subst ','\'',$(subst \,\\,$(1))))) endef #' -VERSION_SED:=$(SED) 's,%U,$(call sed_escape,$(VERSION_REPO)),g' \ +VERSION_SED_SCRIPT:=$(SED) 's,%U,$(call sed_escape,$(VERSION_REPO)),g' \ -e 's,%V,$(call sed_escape,$(VERSION_NUMBER)),g' \ -e 's,%v,\L$(call sed_escape,$(subst $(space),_,$(VERSION_NUMBER))),g' \ -e 's,%C,$(call sed_escape,$(VERSION_CODE)),g' \ @@ -104,4 +104,3 @@ VERSION_SED:=$(SED) 's,%U,$(call sed_escape,$(VERSION_REPO)),g' \ -e 's,%P,$(call sed_escape,$(VERSION_PRODUCT)),g' \ -e 's,%h,$(call sed_escape,$(VERSION_HWREV)),g' -VERSION_SED_SCRIPT:=$(VERSION_SED) diff --git a/package/base-files/Makefile b/package/base-files/Makefile index ed3448094875..ab21213b51f8 100644 --- a/package/base-files/Makefile +++ b/package/base-files/Makefile @@ -137,7 +137,7 @@ define Package/base-files/install fi; \ ) - $(VERSION_SED) \ + $(VERSION_SED_SCRIPT) \ $(1)/etc/banner \ $(1)/etc/openwrt_version \ $(1)/usr/lib/os-release @@ -194,7 +194,7 @@ define Package/base-files/install $(if $(CONFIG_CLEAN_IPKG),, \ mkdir -p $(1)/etc/opkg; \ $(call FeedSourcesAppend,$(1)/etc/opkg/distfeeds.conf); \ - $(VERSION_SED) $(1)/etc/opkg/distfeeds.conf) + $(VERSION_SED_SCRIPT) $(1)/etc/opkg/distfeeds.conf) endef ifneq ($(DUMP),1)