linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jim Davis <jim.epost@gmail.com>
To: Paolo Pisati <paolo.pisati@canonical.com>
Cc: "Masahiro Yamada" <yamada.masahiro@socionext.com>,
	"Michal Marek" <michal.lkml@markovi.net>,
	"Behan Webster" <behanw@converseincode.com>,
	"Matthias Kaehlcke" <mka@chromium.org>,
	"Vinícius Tinti" <viniciustinti@gmail.com>,
	linux-kbuild <linux-kbuild@vger.kernel.org>,
	linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3] scripts/package: snap-pkg target
Date: Tue, 5 Dec 2017 14:23:46 -0700	[thread overview]
Message-ID: <CA+r1ZhjW+OeTnCkffjMfoORx63h8PTNW91HVyvbqVWCpud0UiQ@mail.gmail.com> (raw)
In-Reply-To: <CAMsH0TSaQ02dZOWSPj88tkC5G4bWe7hOOeOgUVOCasz4GRyRbQ@mail.gmail.com>

On Tue, Dec 5, 2017 at 4:43 AM, Paolo Pisati <paolo.pisati@canonical.com> wrote:

>> I noticed that if I tried to make snap-pkg with the O=/some/dir option
>> the tar step failed but the rest of the build continued, and seems to
>> have finished successfully.  Should the snap-pkg target stop after a
>> tar failure?
>
> That is weird, how did you do that?

make O=/some/dir defconfig (or your-config-of-choice)
make O=/some/dir snap-pkg

The snap-pkg target is calling the src_tar command, which bails when
$(objtree) isn't $(srctree).

Is $(objtree) = $(srctree) a necessary restriction with snap builds?
It's a restriction for rpm builds for some reason or another, but I
thought that was just a peculiarity of the rpm build process.  If the
snap's tarball could be built without using src_tar, like the buildtar
script does, then Bob's your uncle.

Though if the source tree is an essential part of the snap build then
perhaps the build should just exit if the tarball can't be generated
for whatever reason, including users doing weird things :)

-- 
Jim

  reply	other threads:[~2017-12-05 21:23 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20171114123834.xsjiy2ynvott4gae@brain>
2017-11-27 11:07 ` [PATCH v3] scripts/package: snap-pkg target Paolo Pisati
2017-11-27 18:33   ` Jim Davis
2017-11-28 17:14     ` Paolo Pisati
2017-11-28 19:35       ` Jim Davis
2017-11-29  7:33         ` Masahiro Yamada
2017-11-29 17:05           ` Paolo Pisati
2017-11-30 23:15             ` Jim Davis
2017-12-05 11:43               ` Paolo Pisati
2017-12-05 21:23                 ` Jim Davis [this message]
2017-12-06 10:36                   ` Masahiro Yamada
2017-12-06 18:14                     ` Paolo Pisati
2017-12-07 15:31                       ` Masahiro Yamada
2017-12-12 15:08                         ` Masahiro Yamada
2017-12-01 12:03             ` Riku Voipio
2017-12-05 11:31               ` Paolo Pisati
2017-11-29 17:03         ` Paolo Pisati

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=CA+r1ZhjW+OeTnCkffjMfoORx63h8PTNW91HVyvbqVWCpud0UiQ@mail.gmail.com \
    --to=jim.epost@gmail.com \
    --cc=behanw@converseincode.com \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=michal.lkml@markovi.net \
    --cc=mka@chromium.org \
    --cc=paolo.pisati@canonical.com \
    --cc=viniciustinti@gmail.com \
    --cc=yamada.masahiro@socionext.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).