[Buildroot] [PATCH] linux: Build and install kernel selftests

[Yann E. MORIN <yann.morin.1998@free.fr>]

Cyril, All,

On 2015-12-21 16:44 +1100, Cyril Bur spake thusly:
> On Thu, 17 Dec 2015 19:07:35 +0100
> "Yann E. MORIN" <yann.morin.1998@free.fr> wrote:
> Totally agreed will address everything you mention. I should have mentioned a
> few in more detail, my bad for not going over it in more detail before posting,
> you're correct that a few things in there are actually non obvious.

No problem! Things just happen to be obvious when you actually do the
job, because everything is fresh in your memory. Patch reviews are there
to eventually highlight that fact. So, really, that's not a problem.

[--SNIP--]
> > > > > +SELFTESTS_INSTALL_STAGING = YES    
> > > > Why install it in staging?  
> > > I actually hadn't initially but I did get to a point in my work where I had
> > > to objdump a test binary. So, since I needed it, it stayed.  
> > But can't you objdump the binaries that are installed in target/ instead?
> 
> I think I tried that initially but IIRC they get stripped when put into target/
> which proved annoying when objdumping.

But then, the 'original' binaries are still present in the build
directory, where they are definitely not stripped (at least, not by
Builroot). Isn't that enough?

Staging is really for libraries and headers that are to be used by
another package. Kernel selftests are not headers, they are not
libraries, so nothing will include/link them. So, I still believe
there's no reason to put them in staging.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'