From patchwork Thu Sep 14 11:17:03 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Masahiro Yamada X-Patchwork-Id: 112538 Delivered-To: patch@linaro.org Received: by 10.140.106.117 with SMTP id d108csp621135qgf; Thu, 14 Sep 2017 04:19:05 -0700 (PDT) X-Google-Smtp-Source: ADKCNb4piqkDboazggEGevA2Q2iky+oceqVLVmONEBuyBH07Eb4n0RoMBQwU2l7SGO/GjXw9zusG X-Received: by 10.98.66.220 with SMTP id h89mr20776931pfd.270.1505387945377; Thu, 14 Sep 2017 04:19:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1505387945; cv=none; d=google.com; s=arc-20160816; b=r9SICDX29Nk3taHN1o+7NuUPTfK5/GYZT+kdks1jOMdiFAkAQVa+uFzApuiBmq2l5/ KbCZOzHQp8v9FWNJvA/Crrr3+ldlWeOi8/aWLiJZusa8RVgK3lPIDDozAgPGaD67ZIGa 20T5F9g21IHVKkkGpxzjijs4RC+eC1Sat9kYD6PfouAxPCwWt64ynnLNN7HkL9f1Fvdp jAvsVXc3l9ucd6U0tsxfLfQkLEqnGNUDswa2nBaV8015h8mAvhZ+XLaV9PFAFVr7XsYK DAb+ADTxhsw5bgA8AYoY20dSw6IgMGN2+X4ZCabfnXL8HLIokXd13AJTwZy6tunuFklE 7pGA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :dkim-signature:dkim-filter:arc-authentication-results; bh=nz34UXgaoMR1JE6TD5zqrFX7oAy6GGNAH+vCrja1Pmo=; b=XuKeNElu03n8d8hLuPk+w3+Y0z0LXQjdF8dOi7zxpXtlgYK9JHCbHnSfjgQC5U72ZF OWfp08ORle2UPo9537XA/KJt+HSH8aNTKvVSTiSSiSIZMS1XP2mkvNxZkrOEwD1R7Xe9 c9INGNhMj08U4U2US4O8Z4+370s6khHTsOEvjfz2YGRA2kPTWH6ZaNe093rB9aYiEZL7 kIT3VqZYzCGL/tPvfkFAH2MezZ1HaAZ55fOkMFr6sjllC68xQgMdiGkZzMkB8AFkPIYF HNUB1as8f28SdpgVJDv90l+y6fy6+tP411n8V9RlkjR9FdmRaYlODL+N+nsPeIaxnPVG 1msg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@nifty.com header.s=dec2015msa header.b=xXFG9Tm7; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f3si4302867pgn.150.2017.09.14.04.19.05; Thu, 14 Sep 2017 04:19:05 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@nifty.com header.s=dec2015msa header.b=xXFG9Tm7; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751841AbdINLSy (ORCPT + 26 others); Thu, 14 Sep 2017 07:18:54 -0400 Received: from conuserg-08.nifty.com ([210.131.2.75]:48087 "EHLO conuserg-08.nifty.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751494AbdINLSw (ORCPT ); Thu, 14 Sep 2017 07:18:52 -0400 Received: from pug.e01.socionext.com (p14092-ipngnfx01kyoto.kyoto.ocn.ne.jp [153.142.97.92]) (authenticated) by conuserg-08.nifty.com with ESMTP id v8EBHDmv002914; Thu, 14 Sep 2017 20:17:14 +0900 DKIM-Filter: OpenDKIM Filter v2.10.3 conuserg-08.nifty.com v8EBHDmv002914 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nifty.com; s=dec2015msa; t=1505387834; bh=nz34UXgaoMR1JE6TD5zqrFX7oAy6GGNAH+vCrja1Pmo=; h=From:To:Cc:Subject:Date:From; b=xXFG9Tm7F2wedEeg+paBqV7nWtoq+g2+mVhpEB5Fja4nqj9Z0kAE9ljLe8pTs4Mz2 SkQ00A4cKnAkYyOZx4S6VARwNH8A6Y+2IMEkBk8zpdl5/SYN2/xQT678k5WoxPvj5H CWCM2ZT5gjexnttgjOiSXhuY0KInMOmq6ISaJKxFeY2Lc8DM/+XwijyQYZuSTPyQGY ZUYogOLro+vp9G53qlLJYcpttErweYNMus116ZLurdzbp6Mr7AHk+7hEVQP5eonKXu fK7OxkQqEqdoc/Egs9ftDTjIYV0e1EDOyHU6Sa/DfTxMJfGd2OFfiCzhziLNCWRk8A ic1Q1as1BnZbw== X-Nifty-SrcIP: [153.142.97.92] From: Masahiro Yamada To: linux-kbuild@vger.kernel.org Cc: Sam Ravnborg , Riku Voipio , Masahiro Yamada , Michal Marek , linux-kernel@vger.kernel.org Subject: [PATCH] kbuild: (bin)rpm-pkg: fix .version handling Date: Thu, 14 Sep 2017 20:17:03 +0900 Message-Id: <1505387823-2033-1-git-send-email-yamada.masahiro@socionext.com> X-Mailer: git-send-email 2.7.4 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The "Release:" field of the spec file is determined based on the .version file. However, the .version file is not copied to the source tar file. So, when we build the kernel from the source package, the UTS_VERSION always indicates #1. This does not match with "rpm -q". The kernel UTS_VERSION and "rpm -q" do not agree for binrpm-pkg, either. Please note the kernel has already been built before the spec file is created. Currently, mkspec invokes mkversion. This script returns an incremented version. So, the "Release:" field of the spec file is greater than the version in the kernel by one. For the source package build (where .version file is missing), we can give KBUILD_BUILD_VERSION=%{release} to the build command. For the binary package build, we can simply read out the .version file because it contains the version number that was used for building the kernel image. We can remove scripts/mkversion because scripts/package/Makefile need not touch the .version file. Signed-off-by: Masahiro Yamada --- scripts/mkversion | 6 ------ scripts/package/Makefile | 5 ----- scripts/package/mkspec | 4 ++-- 3 files changed, 2 insertions(+), 13 deletions(-) delete mode 100644 scripts/mkversion -- 2.7.4 diff --git a/scripts/mkversion b/scripts/mkversion deleted file mode 100644 index c12addc..0000000 --- a/scripts/mkversion +++ /dev/null @@ -1,6 +0,0 @@ -if [ ! -f .version ] -then - echo 1 -else - expr 0`cat .version` + 1 -fi diff --git a/scripts/package/Makefile b/scripts/package/Makefile index 71b4a8a..73f9f31 100644 --- a/scripts/package/Makefile +++ b/scripts/package/Makefile @@ -50,8 +50,6 @@ rpm-pkg rpm: FORCE $(MAKE) clean $(CONFIG_SHELL) $(MKSPEC) >$(objtree)/kernel.spec $(call cmd,src_tar,$(KERNELPATH),kernel.spec) - $(CONFIG_SHELL) $(srctree)/scripts/mkversion > $(objtree)/.tmp_version - mv -f $(objtree)/.tmp_version $(objtree)/.version rpmbuild $(RPMOPTS) --target $(UTS_MACHINE) -ta $(KERNELPATH).tar.gz rm $(KERNELPATH).tar.gz kernel.spec @@ -60,9 +58,6 @@ rpm-pkg rpm: FORCE binrpm-pkg: FORCE $(MAKE) KBUILD_SRC= $(CONFIG_SHELL) $(MKSPEC) prebuilt > $(objtree)/binkernel.spec - $(CONFIG_SHELL) $(srctree)/scripts/mkversion > $(objtree)/.tmp_version - mv -f $(objtree)/.tmp_version $(objtree)/.version - rpmbuild $(RPMOPTS) --define "_builddir $(objtree)" --target \ $(UTS_MACHINE) -bb $(objtree)/binkernel.spec rm binkernel.spec diff --git a/scripts/package/mkspec b/scripts/package/mkspec index bb43f15..c82ef1b 100755 --- a/scripts/package/mkspec +++ b/scripts/package/mkspec @@ -29,7 +29,7 @@ echo "Summary: The Linux Kernel" echo "Version: $__KERNELRELEASE" # we need to determine the NEXT version number so that uname and # rpm -q will agree -echo "Release: `. $srctree/scripts/mkversion`" +echo "Release: $(cat .version)" echo "License: GPL" echo "Group: System Environment/Kernel" echo "Vendor: The Linux Community" @@ -77,7 +77,7 @@ fi echo "%build" if ! $PREBUILT; then -echo "make clean && make %{?_smp_mflags}" +echo "make clean && make %{?_smp_mflags} KBUILD_BUILD_VERSION=%{release}" echo "" fi