[OE-core] [PATCH v2 23/28] Revert "classes/populate_sdk_ext: Add OE_SDK_EXT_SILENT env variable"

[Richard Purdie]

On Fri, 2016-02-05 at 09:40 +1300, Paul Eggleton wrote:
> On Thu, 04 Feb 2016 14:38:56 Aníbal Limón wrote:
> > No i don't mean that way because i don't agree with this change and
> > i
> > consider this as *CENSURE* because if i someone don't agree a
> > change the
> > why? information need to be placed in the git repository for future
> > references.
> 
> We don't put the entire history of decisions on a change in the git
> history 
> (certainly not in the form of commit and revert, anyway) or it would
> be really 
> hard to follow when looking back in future. If you want to record
> that we 
> discussed this alternative solution, the place to do it would be in
> the commit 
> message where you make the redirection change, which was the
> alternative to 
> this.

I'd also add that this is clearly being discussed on the mailing list,
so there *is* a very clear public record of it.

There are patches I've disagreed with which I have merged and there are
patches I've written, then dropped as after discussion they weren't
right (sstate using xz is a recent obvious one). Censorship would be
not even allowing those discussions to happen, we don't do that, very
much the opposite.

As Paul says, we don't put it all into git history though.

Cheers,

Richard