From: Caspar Zhang <caspar@casparzhang.com>
To: LTP list <ltp-list@lists.sourceforge.net>
Subject: Re: [LTP] [RFC] make mce-test a submodule
Date: Fri, 06 Apr 2012 22:37:22 +0800 [thread overview]
Message-ID: <4F7EFFA2.6060105@casparzhang.com> (raw)
In-Reply-To: <4F58E36E.5000306@casparzhang.com>
On 03/09/2012 12:50 AM, Caspar Zhang wrote:
> Hi Garrett, Andi, and all:
>
> When I looked into current LTP tree, I found it a little messy due to
> some mce-test stuff, such as
>
> - mce-test docs goes to $TOPDIR/README, and
> - some mce-test files still lying under testcase/mce-test/.
>
> I think we need a cleanup for it.
>
> Also an idea comes to my mind, that we can try to make mce-test a
> submodule of LTP.
>
> I am also thinking about the pros and cons:
>
> pro(s):
> - easier for code maintenance, no need to sync periodically from
> maintainer side
>
> con(s):
> - two extra steps added for LTP users, they have to do:
> $ git submodule init
> $ git submodule update
> to enable using mce-test.
>
> mce-test could no longer be found at kernel.org, so I cloned it from
> Andi's github repo, not sure if it is the right place:
>
> git://github.com/andikleen/mce-test.git
>
> I have made this change to my personal LTP fork, you can review it here:
>
> https://github.com/casparant/ltp/commit/5cca17f633
>
> Any comments are welcomed.
>
> Thanks,
> Caspar
Thanks all for reviewing this change. Patch merged.
Caspar
------------------------------------------------------------------------------
For Developers, A Lot Can Happen In A Second.
Boundary is the first to Know...and Tell You.
Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
http://p.sf.net/sfu/Boundary-d2dvs2
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
prev parent reply other threads:[~2012-04-06 14:37 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <4F58E36E.5000306@casparzhang.com>
2012-03-09 0:25 ` [LTP] [RFC] make mce-test a submodule Wanlong Gao
2012-03-16 7:27 ` Caspar Zhang
2012-03-16 9:53 ` Garrett Cooper
2012-03-16 9:58 ` Wanlong Gao
2012-03-16 11:17 ` Caspar Zhang
2012-03-16 14:35 ` Andi Kleen
2012-03-21 9:18 ` Chen Gong
2012-04-06 14:33 ` Caspar Zhang
2012-04-09 3:10 ` Chen Gong
2012-04-09 4:03 ` Caspar Zhang
2012-04-09 6:02 ` Chen Gong
2012-04-09 6:05 ` Wanlong Gao
2012-04-09 6:07 ` Caspar Zhang
2012-04-17 2:03 ` Chen Gong
2012-04-17 2:38 ` Caspar Zhang
2012-05-23 1:52 ` Chen Gong
2012-05-30 9:37 ` Chen Gong
2012-05-30 10:25 ` Caspar Zhang
2012-05-31 2:06 ` Chen Gong
2012-04-06 14:37 ` Caspar Zhang [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=4F7EFFA2.6060105@casparzhang.com \
--to=caspar@casparzhang.com \
--cc=ltp-list@lists.sourceforge.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.