[Buildroot] [PATCH v2 00/12] default runtime test case for python packages v2

Ricardo Martincoski ricardo.martincoski at gmail.com
Sat Nov 10 02:15:21 UTC 2018


Hello Thomas,

On Tue, Nov 06, 2018 at 05:56 AM, Thomas Petazzoni wrote:

> On Mon, 05 Nov 2018 23:57:15 -0200, Ricardo Martincoski wrote:
> 
>> >  - Use an integer for the Python version variable, rather than a
>> >    string, there is really no need for this to be a string I believe ?  
>> 
>> You are correct. There is no need for a string. I will use an integer.
> 
> On the other hand, this python_version = 2/3 is only one line, and it
> saves only one line in the config fragment, BR2_PACKAGE_PYTHON=y or
> BR2_PACKAGE_PYTHON3=y.
> 
> So we replace one fairly explicit line:
> 
> 	BR2_PACKAGE_PYTHON=y
> 
> by just another line, which requires going to the base class to
> understand what is does:
> 
> 	python_version = 2
> 
> I.e, I am wondering if this refactoring is really useful in the end ?

Please let me send a v3, based from v1, doing the minimum to accomplish this
goal:
 - add common logic to test python packages with a script in a separate file
   instead of inline in the testcase
... and following these ideas:
 - use separate class
 - use no hacks/tricks
 - use explicit code

>From this v3 we can decide the direction to follow: whether to drop the series
or do some additional change, either as v4 or as followup patches.


Regards,
Ricardo


More information about the buildroot mailing list