[Buildroot] [PATCH v4 1/3] package/radlib: kconfig and makefile

Kinsella, Ray ray.kinsella at intel.com
Thu Apr 7 11:04:02 UTC 2016


Hi Arnout,

On Thu, 2016-04-07 at 00:20 +0200, Arnout Vandecappelle wrote:
>   Very quick feedback, not a detailed review.
> 
>   First of all, the three patches should be squashed into one,
> because all three
> are needed to have a functional package. Otherwise, the build would
> be broken
> between the patches.
> 

Confused, just doing what I was asked in the previous review. 
Can you offer a little more guidance on how I should structure. 

Agree that you need all the pieces for a functional package. 

On Tue, 2016-03-29 at 20:07 +0200, Thomas Petazzoni wrote:
> Hello,
> So you're doing three different things. Which calls for three
> different
> patches.
> 
> I'll do a full review once you send it with git send-email, because
> the
> line wrapping makes it really hard to follow.




More information about the buildroot mailing list