package/Makefile: move source tar creation to a function

Commit Message

Riku Voipio Sept. 1, 2015, 1:48 p.m. UTC

From: Riku Voipio <[e-mail]>

Split source tarball creation from rpm-pkg target
so it can be used from deb-pkg target as well. Other
improvements:

- Replace symlink to self with tar --transform
- Print the name of tarball created in quiet mode

This patch prepares the groundwork for deb-pkg source
package adding bit.

Signed-off-by: Riku Voipio <[e-mail]>
---
 scripts/package/Makefile | 28 +++++++++++++++-------------
 1 file changed, 15 insertions(+), 13 deletions(-)

Comments

Jim Davis Sept. 1, 2015, 2:31 p.m. UTC | #1

On Tue, Sep 1, 2015 at 6:48 AM,  <[e-mail]> wrote:
> From: Riku Voipio <[e-mail]>
>
> Split source tarball creation from rpm-pkg target
> so it can be used from deb-pkg target as well. Other
> improvements:
>
> - Replace symlink to self with tar --transform

While probably everyone -- or almost everyone -- uses gnu tar, that
particular option isn't available in older versions of gnu tar
(https://groups.google.com/forum/#!topic/uk.comp.os.linux/-7LDdMSRo5w).
Perhaps something could be added to Documentation/Changes about the
gnu tar version required?

Riku Voipio Sept. 2, 2015, 4:58 a.m. UTC | #2

On 1 September 2015 at 17:31, Jim Davis <[e-mail]> wrote:
>> - Replace symlink to self with tar --transform

> While probably everyone -- or almost everyone -- uses gnu tar, that
> particular option isn't available in older versions of gnu tar
> (https://groups.google.com/forum/#!topic/uk.comp.os.linux/-7LDdMSRo5w).
> Perhaps something could be added to Documentation/Changes about the
> gnu tar version required?

Looking at the changelog, this was added to tar 1.16 which was
released in 2006. Nine years sounds like a reasonable timeframe to
expect a feature being available?

Riku
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to [e-mail]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Michal Marek Sept. 2, 2015, 6:09 a.m. UTC | #3

Dne 2.9.2015 v 06:58 Riku Voipio napsal(a):
> On 1 September 2015 at 17:31, Jim Davis <[e-mail]> wrote:
>>> - Replace symlink to self with tar --transform
> 
>> While probably everyone -- or almost everyone -- uses gnu tar, that
>> particular option isn't available in older versions of gnu tar
>> (https://groups.google.com/forum/#!topic/uk.comp.os.linux/-7LDdMSRo5w).
>> Perhaps something could be added to Documentation/Changes about the
>> gnu tar version required?
> 
> Looking at the changelog, this was added to tar 1.16 which was
> released in 2006. Nine years sounds like a reasonable timeframe to
> expect a feature being available?

But it costs us only a this ln and rm invocation to continue supporting
older GNU tar versions and non-GNU implementation. So I'd leave this
part as is. The rest of the patch is OK.

Michal

--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to [e-mail]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/scripts/package/Makefile b/scripts/package/Makefile
index 8b11d5a..65cf12d 100644
--- a/scripts/package/Makefile
+++ b/scripts/package/Makefile
@@ -24,27 +24,29 @@ 
 # Remove hyphens since they have special meaning in RPM filenames
 KERNELPATH := kernel-$(subst -,_,$(KERNELRELEASE))
 # Include only those top-level files that are needed by make, plus the GPL copy
-TAR_CONTENT := $(KBUILD_ALLDIRS) kernel.spec .config .scmversion Makefile \
+TAR_CONTENT := $(KBUILD_ALLDIRS) .config .scmversion Makefile \
                Kbuild Kconfig COPYING $(wildcard localversion*)
-TAR_CONTENT := $(addprefix $(KERNELPATH)/,$(TAR_CONTENT))
 MKSPEC     := $(srctree)/scripts/package/mkspec
 
+quiet_cmd_src_tar = TAR     $(2).tar.gz
+      cmd_src_tar = \
+if test "$(objtree)" != "$(srctree)"; then \
+	echo "Building source tarball is not possible outside the"; \
+	echo "kernel source tree. Don't set KBUILD_OUTPUT, or use the"; \
+	echo "binrpm-pkg or bindeb-pkg target instead."; \
+	false; \
+fi ; \
+$(srctree)/scripts/setlocalversion --save-scmversion; \
+tar -cz $(RCS_TAR_IGNORE) --transform 's,^,$(2)/,' \
+    -f $(2).tar.gz $(TAR_CONTENT) $(3);\
+rm -f $(objtree)/.scmversion
+
 # rpm-pkg
 # ---------------------------------------------------------------------------
 rpm-pkg rpm: FORCE
-	@if test "$(objtree)" != "$(srctree)"; then \
-		echo "Building source + binary RPM is not possible outside the"; \
-		echo "kernel source tree. Don't set KBUILD_OUTPUT, or use the"; \
-		echo "binrpm-pkg target instead."; \
-		false; \
-	fi
 	$(MAKE) clean
-	ln -sf $(srctree) $(KERNELPATH)
 	$(CONFIG_SHELL) $(MKSPEC) >$(objtree)/kernel.spec
-	$(CONFIG_SHELL) $(srctree)/scripts/setlocalversion --save-scmversion
-	tar -cz $(RCS_TAR_IGNORE) -f $(KERNELPATH).tar.gz $(TAR_CONTENT)
-	rm $(KERNELPATH)
-	rm -f $(objtree)/.scmversion
+	$(call cmd,src_tar,$(KERNELPATH),kernel.spec)
 	$(CONFIG_SHELL) $(srctree)/scripts/mkversion > $(objtree)/.tmp_version
 	mv -f $(objtree)/.tmp_version $(objtree)/.version
 	rpmbuild --target $(UTS_MACHINE) -ta $(KERNELPATH).tar.gz