All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v5 0/5] Introduce bmap-tools as target + host package
Date: Wed, 9 Jun 2021 09:50:22 +0200	[thread overview]
Message-ID: <20210609095022.4f59aa2f@windsurf> (raw)
In-Reply-To: <6e9afcd08150926a669d4df1198461e0acc7f063.camel@orolia.com>

Hello Nicolas,

On Wed, 09 Jun 2021 09:35:38 +0200
Nicolas Carrier <nicolas.carrier@orolia.com> wrote:

> * the target package works fine
> * so does the host package
> * the tests TestPy3BmapTools and TestPy2BmapTools are failing at run
> time.
> 
> The first error in the run log is:
> 
> qemu-system-arm: Invalid SD card size: 60 MiB
> SD card size has to be a power of 2, e.g. 64 MiB.
> 
> So I set a fixed image size of 64M.
> With this modification, the python 2 test passes, but with that, the
> python 3 one, fails with a timeout issue:
> 
> ...
>   File "/usr/lib/python3/dist-packages/pexpect/expect.py", line 181, in
> expect_loop
>     return self.timeout(e)
>   File "/usr/lib/python3/dist-packages/pexpect/expect.py", line 144, in
> timeout
>     raise exc
> pexpect.exceptions.TIMEOUT: Timeout exceeded.
> ...
> 
> By increasing the timeout to 60 seconds, I could have all the tests
> passing.
> 
> I attached my diff for reference.

Thanks a lot for the testing! I'll double check, but I believe the
tests were working here. But of course, I've done quite a bit of back
and forth, so let's double check :)

> And I have a question, is keeping python 2 compatibility still
> required? Shouldn't we get rid of this test? It'd save quite an amount
> of CPU time :)

python2 is already marked deprecated in BR, and we're more and more
thinking of getting rid of it. I think this may happen in the near
future.

Best regards,

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2021-06-09  7:50 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-07 20:42 [Buildroot] [PATCH v5 0/5] Introduce bmap-tools as target + host package Thomas Petazzoni
2021-06-07 20:42 ` [Buildroot] [PATCH v5 1/5] package/bmap-tools: new package Thomas Petazzoni
2021-06-07 20:42 ` [Buildroot] [PATCH v5 2/5] package/bmap-tools: enable host package Thomas Petazzoni
2021-06-07 20:42 ` [Buildroot] [PATCH v5 3/5] support/testing/infra/emulator.py: update pre-built kernels Thomas Petazzoni
2021-06-07 20:42 ` [Buildroot] [PATCH v5 4/5] support/testing/tests/package/test_bmap_tools: new test Thomas Petazzoni
2021-06-07 20:42 ` [Buildroot] [PATCH v5 5/5] support/testing/tests/package/test_bmap_tools: add test for host bmap-tools Thomas Petazzoni
2021-06-09  7:35 ` [Buildroot] [PATCH v5 0/5] Introduce bmap-tools as target + host package Nicolas Carrier
2021-06-09  7:50   ` Thomas Petazzoni [this message]
2021-06-09  8:31     ` Nicolas Carrier
2021-06-09 18:52   ` Thomas Petazzoni
2021-06-09 19:16     ` Romain Naour
2021-06-10  7:39       ` Nicolas Carrier
2021-06-10 12:31         ` Thomas Petazzoni
2021-06-10 13:15           ` Nicolas Carrier

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=20210609095022.4f59aa2f@windsurf \
    --to=thomas.petazzoni@bootlin.com \
    --cc=buildroot@busybox.net \
    /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.