From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] package/ltp-testsuite: simplify code to exclude test-cases
Date: Tue, 26 Jan 2021 19:43:48 +0100 [thread overview]
Message-ID: <20210126184348.GY2325@scaer> (raw)
In-Reply-To: <YBBYSkMrryKEr3yx@pevik>
Petr, All,
On 2021-01-26 18:58 +0100, Petr Vorel spake thusly:
> > We have three conditions under which some tests may get removed:
> > uClibc, musl, static libraries. All three use the same mechanism
> > to exclude those test-cases: remove the files.
>
> > The first two use a common variable to list the affected files,
> > and share the same hook of their own to iterate over that list,
> > while the third has its own hook.
>
> > This is not very clean, so switch to using a single variable and
> > a single hook for all three conditions.
> Nice cleanup, thank you!
:-)
> Reviewed-by: Petr Vorel <petr.vorel@gmail.com>
Applied to master, thanks for the review.
Regards,
Yann E. MORIN.
> Kind regards,
> Petr
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
prev parent reply other threads:[~2021-01-26 18:43 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-26 17:47 [Buildroot] [PATCH] package/ltp-testsuite: simplify code to exclude test-cases Yann E. MORIN
2021-01-26 17:58 ` Petr Vorel
2021-01-26 18:43 ` Yann E. MORIN [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=20210126184348.GY2325@scaer \
--to=yann.morin.1998@free.fr \
--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.