Date: Sat, 19 Oct 2013 22:51:44 -0400 From: Julio Merino <julio@meroh.net> To: Garrett Cooper <yaneurabeya@gmail.com> Cc: "freebsd-testing@freebsd.org" <freebsd-testing@freebsd.org>, Rui Paulo <rpaulo@fnop.net>, Simon Gerraty <sjg@juniper.net> Subject: Re: Populating /usr/tests Message-ID: <CADyfeQV2G3e3_mTts4CiQqZo-02ZNoid1%2BsHCW65YGQU_Yro4A@mail.gmail.com> In-Reply-To: <6DC6C1E8-7AEF-439E-9250-D223A56EF432@gmail.com> References: <CADyfeQU13ukMK1aMMXKof=Wigqf8hzEBOT91KGdPJcd5nQaWww@mail.gmail.com> <6DC6C1E8-7AEF-439E-9250-D223A56EF432@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Oct 19, 2013 at 4:49 PM, Garrett Cooper <yaneurabeya@gmail.com> wrote: > You're missing a lot of code in etc/, some in Makefile.inc1, and other pieces in share/mk as well with bsd.own.mk. Also, your approach with tests/Makefile is ok, but it doesn't partition well with prove.test.mk on my branch, etc. Hello Garrett, It's not obvious to me what you mean by "missing a lot of code" so I'm going to have to guess. Here are the two possibilities that come to mind: The first is: the patches are missing something because they won't behave in the intended way (where intended = what their description says). If that's the case, I really don't know what's missing and I'd like you to elaborate a bit more please. The patches seem to work and they do what their description says (no more and hopefully no less). Keep in mind that the scope of these patches is quite restricted and their only goal is "plug existing tests into /usr/tests". The second is: the patches are missing something because they don't do everything you already have on your branch. This is certainly the case. I'm not intending to cover all cases at once: it's easier to go step by step with small and easy to review proposals. Additional features can be built on top later and existing code can still be very easily tweaked. Now, because I think the second case is the most likely of the two... I'm still not really sure about what exactly you are referring to! I took a look at your branch and I can't see major differences in the basic code to bring /usr/tests up and running. One thing I found is that you have a tests-specific mtree file, which is a nice idea and I have brought that into the patchset. The other things that I could see... well, they may be necessary later on, but they don't seem to be required for this specific patchset. (In other words: they have to be looked at, but they can probably be looked at separately on their own.) Along these lines: do you have any suggestions on how to extract the set of changes in your branch that are not in head? I'm not talking about a diff: I'm talking about a collection of patches (or commits) describing the differences between HEAD and your branch in a semantical manner. Looking at a snapshot of the tree and comparing differences is really difficult and does not give any idea on how the changes were built. (Sorry, I don't know git enough nor your repository to do this easily; I believe it's possible, but it escapes me how.) Thanks, -- Julio Merino / @jmmv
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CADyfeQV2G3e3_mTts4CiQqZo-02ZNoid1%2BsHCW65YGQU_Yro4A>