Date: Wed, 2 Apr 2014 11:47:08 -0700 From: Garrett Cooper <yaneurabeya@gmail.com> To: Craig Rodrigues <rodrigc@freebsd.org> Cc: "freebsd-testing@freebsd.org" <freebsd-testing@freebsd.org>, Peter Holm <peter@holm.cc> Subject: Re: ATF work Message-ID: <CAGHfRMBX7qeNZaWBUBYd0xejZvc8R=cfD64413tV5_3DfPh_zQ@mail.gmail.com> In-Reply-To: <CAG=rPVep6wrby4cej0fWSNtYrRhHLn5eOjDsNWWTYv0RpjDqUw@mail.gmail.com> References: <20140402085349.GA61802@x2.osted.lan> <CAOtMX2gGn5NMpvSR0FF=z70cUCDn=vTQvVWo2mXk8t9UrNKzuA@mail.gmail.com> <CAG=rPVep6wrby4cej0fWSNtYrRhHLn5eOjDsNWWTYv0RpjDqUw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Apr 2, 2014 at 10:21 AM, Craig Rodrigues <rodrigc@freebsd.org> wrote: > On Wed, Apr 2, 2014 at 9:31 AM, Alan Somers <asomers@freebsd.org> wrote: >> On Wed, Apr 2, 2014 at 2:53 AM, Peter Holm <peter@holm.cc> wrote: >>> I have uploaded a snapshot of the ATF work done for EMC Isilon, >>> based primarily on Garrett Cooper's work. >>> >>> http://people.freebsd.org/~pho/FreeBSD-HEAD-ATF-Isilon-20140401.diff.xz >>> + >>> chmod +x /usr/src/lib/libc/tests/net/gen_ether_subr >> >> Holy patch queue, Batman! There's fifty thousand lines of code in here! > > > When a patch is that big, it becomes very difficult to review. > > You might want to consider forking a copy on github of the > https://github.com/freebsd/freebsd repository and maintaining the patch there > until it is MFC'd: > > https://help.github.com/articles/fork-a-repo > https://help.github.com/articles/syncing-a-fork > > It's not perfect but some of the web interface tools around the github > ecosystem make reviewing patches (and contributing updates) a bit easier. I was going to move master aside and make all of the ATF work into its own separate branch again as my fork's master still contains little tidbits of other fixes and hacks that I made to make things work on my system(s). When I originally did this on my fork I was dealing with a lot more branches (nowadays I mostly maintain my fork instead of develop/port new code). It might be easiest to move forward if pho forks my repo once it's been cleaned up, and the Isilon patches to my code get integrated into my fork as a GitHub pull request. I'll start working on that today and will talk with pho@ offline about getting things all herded together properly. Thanks! -Garrett
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGHfRMBX7qeNZaWBUBYd0xejZvc8R=cfD64413tV5_3DfPh_zQ>