All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnout Vandecappelle <arnout@mind.be>
To: Norbert Lange <nolange79@gmail.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: "Joseph Kogut" <joseph.kogut@gmail.com>,
	"Adam Duskett" <aduskett@gmail.com>,
	"Michał Łyszczek" <michal.lyszczek@bofc.pl>,
	buildroot <buildroot@buildroot.org>
Subject: Re: [Buildroot] [PATCH] skeleton + init: prepare /run/lock and adjust compat symlinks
Date: Wed, 12 Jan 2022 19:39:43 +0100	[thread overview]
Message-ID: <077dd60c-f482-da24-1718-7d75688a5f8a@mind.be> (raw)
In-Reply-To: <CADYdroO91V3OkkyZfsKTW9jPfi0rxw6+f+ei=SUUSW7V5+ZkRQ@mail.gmail.com>



On 12/01/2022 09:53, Norbert Lange wrote:
> 
> 
> 
> Thomas Petazzoni <thomas.petazzoni@bootlin.com 
> <mailto:thomas.petazzoni@bootlin.com>> schrieb am Mi., 12. Jan. 2022, 09:14:
> 
>     Hello Norbert,
> 
>     On Mon,  5 Oct 2020 00:41:36 +0200
>     Norbert Lange <nolange79@gmail.com <mailto:nolange79@gmail.com>> wrote:
> 
>      > Linux distros changed /var/run and /var/lock to reside on an tmpfs in
>      > /run a long time ago, and buildroot seems to agree by providing this
>      > tmpfs on all supported init systems.
> 
>     I am not 100% sure but it seems like this patch has potentially broken
>     the build of vtun, which now fails with:
> 
>        /usr/bin/install: cannot create directory
>     '/home/giuliobenetti/autobuild/run/instance-0/output-1/target/var/lock':
>     File exists
> 
>     If you look at http://autobuild.buildroot.net/?reason=vtun%
>     <http://autobuild.buildroot.net/?reason=vtun%>, vtun
>     version 3.0.4 restarted failing to build on January 10, with this error.
> 
>     Could you have a look ?
> 
>     Best regards,
> 
>     Thomas
>     -- 
>     Thomas Petazzoni, CTO, Bootlin
>     Embedded Linux and Kernel engineering
>     https://bootlin.com <https://bootlin.com>
> 
> 
> Seems like the issue is, that run/lock is not existing when installing.
> Anything installed there will be gone (already before this commit), so it's 
> kinda pointless to do so.

  Ah, yes, there are many packages that require a directory in a tmpfs and that 
create it at build time (because they dont "know" that it's going to be a tmpfs) 
and at runtime (in the init script, through tmpfiles, or by the application itself).

> I don't know how many packages have this issue, imho it would be better to fix 
> them instead of making a run/lock directory for broken packages (that will get 
> removed later)

  We're not going to patch packages just for that, because unavoidably some 
upstreams are not going to be receptive. Creating the directory is a much 
simpler solution.

  Regards,
  Arnout

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

      reply	other threads:[~2022-01-12 18:39 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-04 22:41 [Buildroot] [PATCH] skeleton + init: prepare /run/lock and adjust compat symlinks Norbert Lange
2022-01-09 10:29 ` Arnout Vandecappelle
2022-01-12  8:14 ` Thomas Petazzoni
2022-01-12  8:53   ` Norbert Lange
2022-01-12 18:39     ` Arnout Vandecappelle [this message]

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=077dd60c-f482-da24-1718-7d75688a5f8a@mind.be \
    --to=arnout@mind.be \
    --cc=aduskett@gmail.com \
    --cc=buildroot@buildroot.org \
    --cc=joseph.kogut@gmail.com \
    --cc=michal.lyszczek@bofc.pl \
    --cc=nolange79@gmail.com \
    --cc=thomas.petazzoni@bootlin.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 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.