From: SeongJae Park <sj38.park@gmail.com>
To: SeongJae Park <sj38.park@gmail.com>
Cc: shuah <shuah@kernel.org>,
brendanhiggins@google.com, sjpark@amazon.com, corbet@lwn.net,
kunit-dev@googlegroups.com, linux-doc@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org,
sjpark@amazon.de
Subject: Re: Re: Re: [PATCH v6 0/6] Fix nits in the kunit
Date: Fri, 20 Dec 2019 06:18:31 +0100 [thread overview]
Message-ID: <20191220051831.25296-1-sj38.park@gmail.com> (raw)
In-Reply-To: <20191220050121.24334-1-sj38.park@gmail.com> (raw)
On Fri, 20 Dec 2019 06:01:21 +0100 SeongJae Park <sj38.park@gmail.com> wrote:
> On Thu, 19 Dec 2019 16:37:28 -0700 shuah <shuah@kernel.org> wrote:
>
> > On 12/11/19 7:27 PM, SeongJae Park wrote:
> > > This patchset contains trivial fixes for the kunit documentations and
> > > the wrapper python scripts.
> > >
> > >
> > > Baseline
> > > --------
> > >
> > > This patchset is based on 'kselftest/fixes' branch of
> > > linux-kselftest[1]. A complete tree is available at my repo:
> > > https://github.com/sjp38/linux/tree/kunit_fix/20191205_v6
> > >
> > >
> > > Version History
> > > ---------------
> > >
> > > Changes from v5
> > > (https://lore.kernel.org/linux-kselftest/20191205093440.21824-1-sjpark@amazon.com):
> > > - Rebased on kselftest/fixes
> > > - Add 'Reviewed-by' and 'Tested-by' from Brendan Higgins
> > >
> > > Changes from v4
> > > (https://lore.kernel.org/linux-doc/1575490683-13015-1-git-send-email-sj38.park@gmail.com/):
> > > - Rebased on Heidi Fahim's patch[2]
> > > - Fix failing kunit_tool_test test
> > > - Add 'build_dir' option test in 'kunit_tool_test.py'
> > >
> > > Changes from v3
> > > (https://lore.kernel.org/linux-kselftest/20191204192141.GA247851@google.com):
> > > - Fix the 4th patch, "kunit: Place 'test.log' under the 'build_dir'" to
> > > set default value of 'build_dir' as '' instead of NULL so that kunit
> > > can run even though '--build_dir' option is not given.
> > >
> > > Changes from v2
> > > (https://lore.kernel.org/linux-kselftest/1575361141-6806-1-git-send-email-sj38.park@gmail.com):
> > > - Make 'build_dir' if not exists (missed from v3 by mistake)
> > >
> > > Changes from v1
> > > (https://lore.kernel.org/linux-doc/1575242724-4937-1-git-send-email-sj38.park@gmail.com):
> > > - Remove "docs/kunit/start: Skip wrapper run command" (A similar
> > > approach is ongoing)
> > > - Make 'build_dir' if not exists
> > >
> > > SeongJae Park (6):
> > > docs/kunit/start: Use in-tree 'kunit_defconfig'
> > > kunit: Remove duplicated defconfig creation
> > > kunit: Create default config in '--build_dir'
> > > kunit: Place 'test.log' under the 'build_dir'
> > > kunit: Rename 'kunitconfig' to '.kunitconfig'
> > > kunit/kunit_tool_test: Test '--build_dir' option run
> > >
> > > Documentation/dev-tools/kunit/start.rst | 13 +++++--------
> > > tools/testing/kunit/kunit.py | 18 +++++++++++-------
> > > tools/testing/kunit/kunit_kernel.py | 10 +++++-----
> > > tools/testing/kunit/kunit_tool_test.py | 10 +++++++++-
> > > 4 files changed, 30 insertions(+), 21 deletions(-)
> > >
> >
> > Hi SeongJae Park,
> >
> > Please make sure your From and Signed-off-by addresses match. I tried
> > applying these patches and I am seeing warnings.
> >
> > WARNING: Missing Signed-off-by: line by nominal patch author 'SeongJae
> > Park <sj38.park@gmail.com>'
>
> I tried to reproduce the warning on my side using 'git am' and 'checkpatch.pl',
> but those doesn't show me such warnings. Could you please let me know what
> command you use? Or, you could simply ignore the warning and use 'SeongJae
> Park <sjpark@amazon.de>'.
Seems like 'send-email' has ignored 'From:' in my original patche files. I
rebased those on latest kselftest/fixes and sent v7. If anything wrong, please
let me know.
Thanks,
SeongJae Park
>
>
> Thanks,
> SeongJae Park
>
> >
> > thanks,
> > -- Shuah
>
prev parent reply other threads:[~2019-12-20 5:18 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-12 2:27 [PATCH v6 0/6] Fix nits in the kunit SeongJae Park
2019-12-12 2:27 ` [PATCH v6 1/6] docs/kunit/start: Use in-tree 'kunit_defconfig' SeongJae Park
2019-12-12 2:27 ` [PATCH v6 2/6] kunit: Remove duplicated defconfig creation SeongJae Park
2019-12-12 2:27 ` [PATCH v6 3/6] kunit: Create default config in '--build_dir' SeongJae Park
2019-12-12 2:27 ` [PATCH v6 4/6] kunit: Place 'test.log' under the 'build_dir' SeongJae Park
2019-12-12 2:27 ` [PATCH v6 5/6] kunit: Rename 'kunitconfig' to '.kunitconfig' SeongJae Park
2019-12-12 2:27 ` [PATCH v6 6/6] kunit/kunit_tool_test: Test '--build_dir' option run SeongJae Park
2019-12-12 21:31 ` [PATCH v6 0/6] Fix nits in the kunit Brendan Higgins
2019-12-12 21:44 ` shuah
2019-12-19 23:37 ` shuah
2019-12-20 5:01 ` SeongJae Park
2019-12-20 5:18 ` SeongJae Park [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=20191220051831.25296-1-sj38.park@gmail.com \
--to=sj38.park@gmail.com \
--cc=brendanhiggins@google.com \
--cc=corbet@lwn.net \
--cc=kunit-dev@googlegroups.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=shuah@kernel.org \
--cc=sjpark@amazon.com \
--cc=sjpark@amazon.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;
as well as URLs for NNTP newsgroup(s).