All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick Williams <patrick@stwcx.xyz>
To: Deng Tyler <tyler.sabdon@gmail.com>
Cc: OpenBMC Maillist <openbmc@lists.ozlabs.org>
Subject: Re: Run unit-test during development
Date: Thu, 16 Dec 2021 11:12:18 -0600	[thread overview]
Message-ID: <YbtzcvbSwTjGzfcH@heinlein> (raw)
In-Reply-To: <CAO9PYRJD0MJsuY8M_80yJci-nT8idKL7VhSXJzvX_JS1wreJzA@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1597 bytes --]

On Thu, Dec 16, 2021 at 01:53:41PM +0800, Deng Tyler wrote:
> Hi Patrick:
>     Thank for your kind reply and that's really helpful. I am trying to
> figure out the meson subprojects system but still encounter lot of barrier.
> I run unt test using command "meson buiildir && ninja -C builddir test" in
> my development environment but never work. Could you help to cite an
> example in current openbmc repository? Do openbmc contributor run unit test
> by meson subprojects or run-unit-test-docker.sh when they are developing?

I can't really comment on what other people do, but I can tell you what I do.

1. If the repository supports Meson, I always attempt to build / test using
   standalone support (ie. meson subprojects).

2. If 'meson subprojects' doesn't seem to work, I fix it to make it work.

3. If the repository is anything else and it isn't too much effort, I try to
   port it to meson.

4. When all else fails I use 'run-unit-test-docker.sh'.  I often execute it with
   an alias I have: `NO_FORMAT_CODE=1 lf-ut`[1].

---

Do you have examples of meson+ninja failing?  I'm fairly certain it works fine
for almost every repository I'm listed as a maintainer on.  I've also used it
(or fixed it) fairly recently for: bmcweb, phosphor-logging, and
phosphor-certificate-manager.

Maybe you can point to what failure you're seeing in that case as well (you
can upload a log to gist.github.com )?



1. https://github.com/williamspatrick/dotfiles/blob/fb542abf99c944051544990f69a6d20c82d2a904/env/30_linux/lfopenbmc.zsh#L76

-- 
Patrick Williams

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2021-12-16 17:13 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-13  8:46 Run unit-test during development Deng Tyler
2021-12-13 16:00 ` Patrick Williams
2021-12-16  5:53   ` Deng Tyler
2021-12-16 17:12     ` Patrick Williams [this message]
2021-12-17  2:36       ` Deng Tyler
2021-12-17  2:39         ` Andrew Jeffery

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=YbtzcvbSwTjGzfcH@heinlein \
    --to=patrick@stwcx.xyz \
    --cc=openbmc@lists.ozlabs.org \
    --cc=tyler.sabdon@gmail.com \
    /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.