pkg-download: make sure git downloads fail for unknown versions

The current git download helper creates the tarball by doing:

    git archive <version> | gzip -c > <tarball>

Unfortunately, even if "git archive" fails and returns a non-zero
error code, gzip ignores that, compresses nothing, and returns success
(zero error code). The consequence of this behavior is that when the
git version provided in the package is incorrect, we are not failing
during the download step, but later on when trying to extract the
tarball (which was incorrectly created as a result of the failing git
archive).

To fix this, we change the tarball creation logic to:

   git archive -o <tarball>.tmp <version> &&
   gzip -c <tarball>.tmp > <tarball> &&
   rm -f <tarball>.tmp

If the build is interrupted during the "gzip" command, we may leave
the .tmp file behind us, but this also happens with wget downloads,
and is generally not considered a problem, since this temporary file
will be overwritten next time we attempt to do download this package.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>

diff --git a/package/pkg-download.mk b/package/pkg-download.mk
index c00689b89c4ed4398d3528ea8b72ee3182ca2556..2641d4e8b7c2cbfd142662a3a1009daf737d45c0 100644 (file)
--- a/package/pkg-download.mk
+++ b/package/pkg-download.mk
@@ -91,8 +91,9 @@ define DOWNLOAD_GIT
          (echo "Doing full clone" && \
           $(GIT) clone --bare $($(PKG)_SITE) $($(PKG)_BASE_NAME))) && \
        pushd $($(PKG)_BASE_NAME) > /dev/null && \
-       $(GIT) archive --format=tar --prefix=$($(PKG)_BASE_NAME)/ $($(PKG)_DL_VERSION) | \
-               gzip -c > $(DL_DIR)/$($(PKG)_SOURCE) && \
+       $(GIT) archive --format=tar --prefix=$($(PKG)_BASE_NAME)/ -o $(DL_DIR)/.$($(PKG)_SOURCE).tmp $($(PKG)_DL_VERSION) && \
+       gzip -c $(DL_DIR)/.$($(PKG)_SOURCE).tmp > $(DL_DIR)/$($(PKG)_SOURCE) && \
+       rm -f $(DL_DIR)/.$($(PKG)_SOURCE).tmp && \
        popd > /dev/null && \
        rm -rf $($(PKG)_DL_DIR) && \
        popd > /dev/null)