All of lore.kernel.org
 help / color / mirror / Atom feed
From: Masahiro Yamada <yamada.masahiro@socionext.com>
To: Dmitry Goldin <dgoldin@protonmail.ch>
Cc: Andreas Schwab <schwab@linux-m68k.org>,
	Greg KH <gregkh@linuxfoundation.org>,
	"linux-kernel\\@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"joel\\@joelfernandes.org" <joel@joelfernandes.org>,
	Ben Hutchings <ben@decadent.org.uk>
Subject: Re: [PATCH v2] kheaders: making headers archive reproducible
Date: Tue, 8 Oct 2019 17:14:51 +0900	[thread overview]
Message-ID: <CAK7LNASwrKohUUY22Ru06DcG5nUpqRJW3ZjZR+2BZYsX8hfvJw@mail.gmail.com> (raw)
In-Reply-To: <BbFL6w_pvJJ1heDKuGhto7sFNt-6M-GQSqysyQ75Lgd_MOwqEGzkFdhqvmcDhS27MbsEZ239tZ-1BMjC_ObkRB16jR8vS2Ri8HGJWul6wsw=@protonmail.ch>

On Tue, Oct 8, 2019 at 5:07 PM Dmitry Goldin <dgoldin@protonmail.ch> wrote:
>
> Hi,
>
> ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
> On Monday, October 7, 2019 2:26 PM, Andreas Schwab <schwab@linux-m68k.org> wrote:
>
> > On Okt 07 2019, Greg KH gregkh@linuxfoundation.org wrote:
> >
> > > On Mon, Oct 07, 2019 at 01:49:47PM +0200, Andreas Schwab wrote:
> > >
> > > > GEN kernel/kheaders_data.tar.xz
> > > > tar: unrecognized option '--sort=name'
> > > > Try `tar --help' or`tar --usage' for more information.
> > > > make[2]: *** [kernel/kheaders_data.tar.xz] Error 64
> > > > make[1]: *** [kernel] Error 2
> > > > make: *** [sub-make] Error 2
> > > > $ tar --version
> > > > tar (GNU tar) 1.26
> > > > Copyright (C) 2011 Free Software Foundation, Inc.
> > >
> > > Wow that's an old version of tar. 2011? What happens if you use a more
> > > modern one?
> >
> > That's the most modern I have available on that machine.
>
> Hmm. --sort was introduced in 1.28 in 2014. Do you think it would warrant some sort of version check and fallback or is this something we can expect the user to handle if their distribution happens to not ship anything more recent? A few sensible workarounds come to mind.

I think the former.

The release in 2014 is quite new, so
we can not always expect it on the users' system.




>
> In any case, likely it would make sense to at least update to https://github.com/torvalds/linux/blob/master/Documentation/process/changes.rst with the minimal version we decide on.
>
>
> Dmitry



-- 
Best Regards
Masahiro Yamada

  reply	other threads:[~2019-10-08  8:15 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-04 10:40 [PATCH v2] kheaders: making headers archive reproducible Dmitry Goldin
2019-10-04 15:18 ` gregkh
2019-10-04 18:14 ` Joel Fernandes
2019-10-05  3:31 ` Masahiro Yamada
2019-10-07 11:49 ` Andreas Schwab
2019-10-07 11:52   ` Greg KH
2019-10-07 12:26     ` Andreas Schwab
2019-10-08  8:07       ` Dmitry Goldin
2019-10-08  8:14         ` Masahiro Yamada [this message]
2019-10-08  9:54           ` Dmitry Goldin
2019-10-08 10:27             ` Masahiro Yamada

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=CAK7LNASwrKohUUY22Ru06DcG5nUpqRJW3ZjZR+2BZYsX8hfvJw@mail.gmail.com \
    --to=yamada.masahiro@socionext.com \
    --cc=ben@decadent.org.uk \
    --cc=dgoldin@protonmail.ch \
    --cc=gregkh@linuxfoundation.org \
    --cc=joel@joelfernandes.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=schwab@linux-m68k.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.