All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Barker <paul@paulbarker.me.uk>
To: tom@ewsting.org
Cc: brendan.le.foll@intel.com, yocto@yoctoproject.org,
	opkg-devel@googlegroups.com,
	Thomas Ingleby <thomas.c.ingleby@intel.com>,
	paul.eggleton@linux.intel.com
Subject: Re: [PATCH V2] opkg-build: add detection if using GNU tar.
Date: Thu, 5 Jun 2014 19:56:02 +0000	[thread overview]
Message-ID: <20140605195602.GN7521@gmail.com> (raw)
In-Reply-To: <1401791135-11672-1-git-send-email-tom@ewsting.org>

[-- Attachment #1: Type: text/plain, Size: 1243 bytes --]

On Tue, Jun 03, 2014 at 11:25:35AM +0100, tom@ewsting.org wrote:
> From: Thomas Ingleby <thomas.c.ingleby@intel.com>
> 
> * Some options of GNU tar do not exist on other implementations
> 
> Signed-off-by: Thomas Ingleby <thomas.c.ingleby@intel.com>

I've made a couple of slight further changes, hope you don't mind.

I've changed the subject to:

    opkg-build: Detect if tar supports '--format' option

I've also added a comment before the block of code which sets tarformat:

+# Determine if tar supports the --format argument by checking the help output.
+#
+# This is needed because:
+#    - Busybox tar doesn't support '--format'
+#    - On some Linux distros, tar now defaults to posix format if '--format'
+#      isn't explictly specified
+#    - Opkg doesn't currently support posix format archives
+#
+# It's easier to check for mention of the '--format' option than to detect the
+# tar implementation and maintain a list of which support '--format'.

I've merged this to master with those changes. Sadly I made an error writing the
comment and pushed too soon so I've pushed another commit to fix that.

Many thanks,

-- 
Paul Barker

Email: paul@paulbarker.me.uk
http://www.paulbarker.me.uk

[-- Attachment #2: Type: application/pgp-signature, Size: 501 bytes --]

  reply	other threads:[~2014-06-05 19:56 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-03 10:25 [PATCH V2] opkg-build: add detection if using GNU tar tom
2014-06-05 19:56 ` Paul Barker [this message]
2014-06-05 20:09   ` Tom Ingleby

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140605195602.GN7521@gmail.com \
    --to=paul@paulbarker.me.uk \
    --cc=brendan.le.foll@intel.com \
    --cc=opkg-devel@googlegroups.com \
    --cc=paul.eggleton@linux.intel.com \
    --cc=thomas.c.ingleby@intel.com \
    --cc=tom@ewsting.org \
    --cc=yocto@yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.