Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-27 Thread Stefan Herbrechtsmeier
Am 27.12.21 um 15:54 schrieb Eero Aaltonen: On Mon, 2021-12-27 at 14:38 +0100, Stefan Herbrechtsmeier via lists.openembedded.org wrote: Hi Alex, Am 25.12.21 um 20:41 schrieb Alexander Kanavin: On Sat, 25 Dec 2021 at 20:32, Stefan Herbrechtsmeier mailto:ste...@herbrechtsmeier.net>> wrote:

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-27 Thread Eero Aaltonen
On Mon, 2021-12-27 at 14:38 +0100, Stefan Herbrechtsmeier via lists.openembedded.org wrote: > Hi Alex, > > Am 25.12.21 um 20:41 schrieb Alexander Kanavin: > > On Sat, 25 Dec 2021 at 20:32, Stefan Herbrechtsmeier > > mailto:ste...@herbrechtsmeier.net>> > > wrote: > > > > > I'm not sure how t

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-27 Thread Alexander Kanavin
On Mon, 27 Dec 2021 at 14:38, Stefan Herbrechtsmeier < ste...@herbrechtsmeier.net> wrote: > > I tend to think that the best (and the hardest) option is to improve > > these tools so that they're usable inside do_fetch (e.g. fulfil the > > caching/reproducibility criteria for a bitbake fetcher), an

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-27 Thread Stefan Herbrechtsmeier
Hi Alex, Am 25.12.21 um 20:41 schrieb Alexander Kanavin: On Sat, 25 Dec 2021 at 20:32, Stefan Herbrechtsmeier mailto:ste...@herbrechtsmeier.net>> wrote: > I'm not sure how to deal with that, so there aren't that many options here. This is a common problem for all language specifi

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-27 Thread Konrad Weihmann
On 27.12.21 13:54, Stefan Herbrechtsmeier wrote: Hi Konrad, Am 25.12.21 um 21:43 schrieb Konrad Weihmann: What I so far don't really get is why increase in parsing time is such a big deal. I admit when we're talking about npm it's some kind of a drastic increase in recipes one would have to

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-27 Thread Stefan Herbrechtsmeier
Am 25.12.21 um 21:58 schrieb Konrad Weihmann: On 25.12.21 20:32, Stefan Herbrechtsmeier wrote: Isn't the circular dependencies problem mainly a problem of the native packages? In many cases the dependency only exists at compile time of the main project and not at compile time of the depende

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-27 Thread Stefan Herbrechtsmeier
Hi Konrad, Am 25.12.21 um 21:43 schrieb Konrad Weihmann: What I so far don't really get is why increase in parsing time is such a big deal. I admit when we're talking about npm it's some kind of a drastic increase in recipes one would have to maintain, just because some random project decides

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-25 Thread Konrad Weihmann
One more thing that came to my mind is that the project is missing demo recipes/real life examples in core that actually would show that one can achieve reproducibility in the sense of the project's definition for both npm and go. So far there is the fetcher code but real life users are living

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-25 Thread Konrad Weihmann
On 25.12.21 20:32, Stefan Herbrechtsmeier wrote: Isn't the circular dependencies problem mainly a problem of the native packages? In many cases the dependency only exists at compile time of the main project and not at compile time of the dependency itself. In case of a native package all de

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-25 Thread Konrad Weihmann
What I so far don't really get is why increase in parsing time is such a big deal. I admit when we're talking about npm it's some kind of a drastic increase in recipes one would have to maintain, just because some random project decides to use a trillion dependencies instead of writing two or t

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-25 Thread Alexander Kanavin
On Sat, 25 Dec 2021 at 20:32, Stefan Herbrechtsmeier < ste...@herbrechtsmeier.net> wrote: > > I'm not sure how to deal with that, so there aren't that many options > here. > > This is a common problem for all language specific package managers > (python / pip, Node.js / npm, Rust / Carge, go) and

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-25 Thread Khem Raj
On Fri, Dec 24, 2021 at 2:36 AM Konrad Weihmann wrote: > > I had a look at the failures and most of them are fairly easy to fix - > all but the go recipes like influxdb. > > I'm not sure how to deal with that, so there aren't that many options here. > > Either > - inject all the needed with a fixe

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-25 Thread Stefan Herbrechtsmeier
Hi Konrad, Am 24.12.21 um 11:36 schrieb Konrad Weihmann: I had a look at the failures and most of them are fairly easy to fix - all but the go recipes like influxdb. The go class doesn't work without network by default if the project doesn't provide a vendor folder. I'm not sure how to dea

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-25 Thread Stefan Herbrechtsmeier
Hi Richard, Am 24.12.21 um 09:30 schrieb Richard Purdie: On Thu, 2021-12-23 at 22:00 -0800, Khem Raj wrote: is this list of failures due to this patch ? https://errors.yoctoproject.org/Errors/Build/137579/ Looks likely, yes. Recipes shouldn't be attempting downloads in do_compile :( In this

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-24 Thread Konrad Weihmann
I had a look at the failures and most of them are fairly easy to fix - all but the go recipes like influxdb. I'm not sure how to deal with that, so there aren't that many options here. Either - inject all the needed with a fixed revision, but that would prevent devtool from catching updates to

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-24 Thread Richard Purdie
On Thu, 2021-12-23 at 22:00 -0800, Khem Raj wrote: > is this list of failures due to this patch ? > https://errors.yoctoproject.org/Errors/Build/137579/ Looks likely, yes. Recipes shouldn't be attempting downloads in do_compile :( Cheers, Richard -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-23 Thread Khem Raj
is this list of failures due to this patch ? https://errors.yoctoproject.org/Errors/Build/137579/ On Wed, Dec 22, 2021 at 3:20 PM Richard Purdie wrote: > > Use the newly added nonetwork task flag to disable network access where > possible in unpack/patch/configure/compile/install tasks. > > We ca

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-23 Thread Alexander Kanavin
On Thu, 23 Dec 2021 at 14:11, Richard Purdie < richard.pur...@linuxfoundation.org> wrote: > > This is actually a brilliant idea, which would also make it easier to > > control this behavior from a user's perspective > > Part of me wonders if we really do want to make this "easy" for the user :/ >

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-23 Thread Jose Quaresma
1 > > To: openembedded-core@lists.openembedded.org > > Subject: [OE-core] [PATCH] base/patch: Disable network for > unpack/patch/configure/compile/install > > > > Use the newly added nonetwork task flag to disable network access where > > possible in unpack/patch/configure/c

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-23 Thread Konrad Weihmann
-core@lists.openembedded.org On Behalf Of Richard Purdie Sent: den 23 december 2021 00:21 To: openembedded-core@lists.openembedded.org Subject: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install Use the newly added nonetwork task flag to disable network access

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-23 Thread Andreas Müller
riginal Message- > >>>> From: openembedded-core@lists.openembedded.org > >>>> On Behalf Of Richard Purdie > >>>> Sent: den 23 december 2021 00:21 > >>>> To: openembedded-core@lists.openembedded.org > >>>> Subject: [OE-core] [PAT

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-23 Thread Konrad Weihmann
: openembedded-core@lists.openembedded.org Subject: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install Use the newly added nonetwork task flag to disable network access where possible in unpack/patch/configure/compile/install tasks. We can't disable networki

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-23 Thread Richard Purdie
On Thu, 2021-12-23 at 08:28 +0300, Alexander Kanavin wrote: > Should there be tests for this? Would be good to check that the network is > indeed disabled in these tasks. I think Ross was looking at that. The challenge will be the list needed to know which distros this works on and which it does n

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-23 Thread Richard Purdie
ber 2021 00:21 > > > To: openembedded-core@lists.openembedded.org > > > Subject: [OE-core] [PATCH] base/patch: Disable network for > > > unpack/patch/configure/compile/install > > > > > > Use the newly added nonetwork task flag to disable network acces

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-23 Thread Konrad Weihmann
On 23.12.21 11:49, Peter Kjellerstedt wrote: -Original Message- From: openembedded-core@lists.openembedded.org On Behalf Of Richard Purdie Sent: den 23 december 2021 00:21 To: openembedded-core@lists.openembedded.org Subject: [OE-core] [PATCH] base/patch: Disable network for unpack

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-23 Thread Peter Kjellerstedt
> -Original Message- > From: openembedded-core@lists.openembedded.org > On Behalf Of Richard Purdie > Sent: den 23 december 2021 00:21 > To: openembedded-core@lists.openembedded.org > Subject: [OE-core] [PATCH] base/patch: Disable network for > unpack/patch/conf

Re: [OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-22 Thread Alexander Kanavin
Should there be tests for this? Would be good to check that the network is indeed disabled in these tasks. Alex On Thu 23. Dec 2021 at 2.20, Richard Purdie < richard.pur...@linuxfoundation.org> wrote: > Use the newly added nonetwork task flag to disable network access where > possible in unpack/

[OE-core] [PATCH] base/patch: Disable network for unpack/patch/configure/compile/install

2021-12-22 Thread Richard Purdie
Use the newly added nonetwork task flag to disable network access where possible in unpack/patch/configure/compile/install tasks. We can't disable networking in sstate tasks due to sstate downloads and also so we can report hash equivalence to the server. Signed-off-by: Richard Purdie --- meta/