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

[Aníbal Limón]

On 02/04/2016 02:47 PM, Richard Purdie wrote:
> 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.

Ok, i'll add the comments and the link to the ML in the original commit
and drop the another two.

There is a place/wiki/etc where this kind of issues are explained?

	alimon


> 
> Cheers,
> 
> Richard
> 
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20160204/a9ac601b/attachment-0002.sig>