All of lore.kernel.org
 help / color / mirror / Atom feed
From: AKASHI Takahiro <takahiro.akashi@linaro.org>
To: u-boot@lists.denx.de
Subject: [PATCH 08/13] test/py: efi_secboot: remove all "re.search"
Date: Tue, 2 Jun 2020 14:58:11 +0900	[thread overview]
Message-ID: <20200602055811.GE20446@laputa> (raw)
In-Reply-To: <333ee33f-ead5-1c91-a4a9-689153d349ef@gmx.de>

On Sat, May 30, 2020 at 09:04:11AM +0200, Heinrich Schuchardt wrote:
> On 5/29/20 8:41 AM, AKASHI Takahiro wrote:
> > Currently, we don't use any regular expression in matching outputs from
> > U-Boot. Since its use is just redundant, we can remove all.
> >
> > Signed-off-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
> 
> Applying: test/py: efi_secboot: remove all "re.search"
> error: patch failed: test/py/tests/test_efi_secboot/test_signed.py:32
> error: test/py/tests/test_efi_secboot/test_signed.py: patch does not apply
> error: patch failed: test/py/tests/test_efi_secboot/test_unsigned.py:33
> error: test/py/tests/test_efi_secboot/test_unsigned.py: patch does not apply
> Patch failed at 0001 test/py: efi_secboot: remove all "re.search"
> 
> Could you, please, rebase your patches.

Probably due to the commit,
===
commit 5827c2545849
Author: Heinrich Schuchardt <xypron.glpk@gmx.de>
Date:   Mon May 4 12:21:51 2020 +0200

    test: stabilize test_efi_secboot
===

I have already proposed to revert this patch in
https://lists.denx.de/pipermail/u-boot/2020-May/412859.html

Anyway,
the essential issue is not my test, but efi_console which
tries to unconditionally send control code which a terminal may not
understand.

-Takahiro Akashi


> Best regards
> 
> Heinrich

  reply	other threads:[~2020-06-02  5:58 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-29  6:41 [PATCH 00/13] efi_loader: rework/improve UEFI secure boot code AKASHI Takahiro
2020-05-29  6:41 ` [PATCH 01/13] efi_loader: signature: move efi_guid_cert_type_pkcs7 to efi_signature.c AKASHI Takahiro
2020-05-29 10:27   ` Heinrich Schuchardt
2020-05-29  6:41 ` [PATCH 02/13] efi_loader: image_loader: add a check against certificate type of authenticode AKASHI Takahiro
2020-05-29 10:37   ` Heinrich Schuchardt
2020-06-02  2:22     ` AKASHI Takahiro
2020-05-29  6:41 ` [PATCH 03/13] efi_loader: image_loader: retrieve authenticode only if it exists AKASHI Takahiro
2020-05-30  6:02   ` Heinrich Schuchardt
2020-05-29  6:41 ` [PATCH 04/13] efi_loader: signature: fix a size check against revocation list AKASHI Takahiro
2020-05-30  6:42   ` Heinrich Schuchardt
2020-05-29  6:41 ` [PATCH 05/13] efi_loader: signature: make efi_hash_regions more generic AKASHI Takahiro
2020-05-30  6:58   ` Heinrich Schuchardt
2020-06-02  5:05     ` AKASHI Takahiro
2020-05-29  6:41 ` [PATCH 06/13] efi_loader: image_loader: verification for all signatures should pass AKASHI Takahiro
2020-05-30  7:01   ` Heinrich Schuchardt
2020-06-02  5:22     ` AKASHI Takahiro
2020-05-29  6:41 ` [PATCH 07/13] efi_loader: image_loader: add digest-based verification for signed image AKASHI Takahiro
2020-05-30  7:09   ` Heinrich Schuchardt
2020-06-02  5:31     ` AKASHI Takahiro
2020-05-29  6:41 ` [PATCH 08/13] test/py: efi_secboot: remove all "re.search" AKASHI Takahiro
2020-05-30  7:04   ` Heinrich Schuchardt
2020-06-02  5:58     ` AKASHI Takahiro [this message]
2020-06-02  8:27       ` Heinrich Schuchardt
2020-07-02 16:21   ` Heinrich Schuchardt
2020-05-29  6:41 ` [PATCH 09/13] test/py: efi_secboot: fix test case 1g of test_authvar AKASHI Takahiro
2020-07-02 16:28   ` Heinrich Schuchardt
2020-05-29  6:41 ` [PATCH 10/13] test/py: efi_secboot: split "signed image" test case-1 into two cases AKASHI Takahiro
2020-05-29  6:41 ` [PATCH 11/13] test/py: efi_secboot: add a test against certificate revocation AKASHI Takahiro
2020-05-29  6:41 ` [PATCH 12/13] test/py: efi_secboot: add a test for multiple signatures AKASHI Takahiro
2020-05-29  6:41 ` [PATCH 13/13] test/py: efi_secboot: add a test for verifying with digest of signed image AKASHI Takahiro

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=20200602055811.GE20446@laputa \
    --to=takahiro.akashi@linaro.org \
    --cc=u-boot@lists.denx.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 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.