public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: Claudius Heine <claudius.heine.ext@siemens.com>
To: "Maxim Yu. Osipov" <mosipov@ilbers.de>, isar-users@googlegroups.com
Cc: Claudius Heine <ch@denx.de>
Subject: Re: [PATCH 0/1] Disable daemon activation
Date: Tue, 5 Jun 2018 14:45:57 +0200	[thread overview]
Message-ID: <efa93b81-6049-db29-0d3c-663e0a48f380@siemens.com> (raw)
In-Reply-To: <e2b9e413-7240-f3e5-feba-48f5001393e7@ilbers.de>

Hi Maxim,

On 2018-06-05 14:38, Maxim Yu. Osipov wrote:
> Hi Claudius,
> 
> On 06/05/2018 01:52 PM, Claudius Heine wrote:
>> Hi Maxim,
>>
>> On 2018-06-05 10:42, Maxim Yu. Osipov wrote:
>>> Hi Claudius,
>>>
>>> On 06/04/2018 01:21 PM, claudius.heine.ext@siemens.com wrote:
>>>> From: Claudius Heine <ch@denx.de>
>>>>
>>>> Hi,
>>>>
>>>> this patch disables the daemon activation in the chroot environment.
>>>>
>>>> Since I could not reproduce the problem this patch handles, please test
>>>
>>> When Alex reported this problem he mentioned that to reproduce the 
>>> problem run './scripts/ci_build.sh -q':
>>
>> I was sort of interested in a minimal test setup with just one target. 
>> Running a build that takes hours on my system is not something I like 
>> to do just to investigate it. I have to run it twice just to see if 
>> that problem occurs here and if this patch fixes it.
>>
>> I tried using just one target with my current setup, but I could not 
>> reproduce it, so now its up to reporter to see if that fix resolves 
>> this issue.
>>
>> But what I did, to prevent back and forth sending of patches and 
>> reports, was just to copy most of the code from the debian installer 
>> project and see if that code runs correctly in isar. And as far as I 
>> see it does.
> 
> I've tested your patch (first version) by running ci_build.sh script (it 
> takes one hour to run on my laptop) -
> I don't see anymore running daemons after the build is complete.
> 
> Now I'm testing the second version (it applied smoothly except one 
> whitespace warning) and, hopefully, your patch will appear soon in 'next'.

This patch does not change any code, just adds copyright notices, so you 
could skip that and instead build without this patch applied to see if 
you can reproduce this issue.

Cheers,
Claudius

> 
> 
> Thanks,
> Maxim.
> 
>> Claudius
>>
>>>
>>> <---->
>>>
>>> -------- Forwarded Message --------
>>> Subject: Daemons and deboostrap
>>> Date: Sun, 27 May 2018 19:00:28 +0300
>>> From: Alexander Smirnov <asmirnov@ilbers.de>
>>> To: isar-users <isar-users@googlegroups.com>
>>>
>>> Hi all,
>>>
>>> testing patches from mailing list I've found the problem with the 
>>> current next: there are lots of daemons remain running after build is 
>>> complete (./scripts/ci_build.sh -q).
>>>
>>> <---->
>>>
>>>
>>> Kind regards,
>>> Maxim.
>>>
>>>
>>>> if this fixes it, if the code looks fine.
>>>>
>>>> Cheers,
>>>> Claudius
>>>>
>>>> Claudius Heine (1):
>>>>    meta/isar-bootstrap: deactivate daemon activation in chroot
>>>>      environment
>>>>
>>>>   meta/classes/isar-bootstrap-helper.bbclass    |   2 +
>>>>   .../isar-bootstrap/files/chroot-setup.sh      | 133 
>>>> ++++++++++++++++++
>>>>   .../isar-bootstrap/isar-bootstrap.bb          |  11 +-
>>>>   3 files changed, 144 insertions(+), 2 deletions(-)
>>>>   create mode 100644 
>>>> meta/recipes-core/isar-bootstrap/files/chroot-setup.sh
>>>>
>>>
>>>
>>
> 
> 

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de

      reply	other threads:[~2018-06-05 12:45 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-04 11:21 claudius.heine.ext
2018-06-04 11:21 ` [PATCH 1/1] meta/isar-bootstrap: deactivate daemon activation in chroot environment claudius.heine.ext
2018-06-05 11:05   ` Jan Kiszka
2018-06-04 17:36 ` [PATCH 0/1] Disable daemon activation Henning Schild
2018-06-04 17:48   ` Claudius Heine
2018-06-04 18:02     ` Henning Schild
2018-06-05  8:42 ` Maxim Yu. Osipov
2018-06-05 11:52   ` Claudius Heine
2018-06-05 12:38     ` Maxim Yu. Osipov
2018-06-05 12:45       ` Claudius Heine [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=efa93b81-6049-db29-0d3c-663e0a48f380@siemens.com \
    --to=claudius.heine.ext@siemens.com \
    --cc=ch@denx.de \
    --cc=isar-users@googlegroups.com \
    --cc=mosipov@ilbers.de \
    /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