[OE-core] version upgrade to python-click - patch

[Felix Snow]

Hello, 

Thanks for the reply, I have tried several times over the last few days to subscribe to the devel list so I can send the patch there but it doesn't seem to work, so sending it here instead. Patch is attached.

The changes worked as I applied them in our firmware, and I cloned your repo locally, carefully put the changes in and created the patch. Am not sure exactly how to test it from within your repo. If you can fire it up and put the following into the terminal:

python3 -c "import click; print(click.__version__)"

and it returns “7.0” then it is probably working. Would not hurt to do the same with python2. 

Regards,
Felix Snow

-----Ursprüngliche Nachricht-----
Von: Randy MacLeod [mailto:randy.macleod at windriver.com] 
Gesendet: Dienstag, 12. Februar 2019 19:06
An: Felix Snow <fsnow at phoenixcontact.com>; openembedded-core at lists.openembedded.org; openembedded-devel at lists.openembedded.org
Betreff: Re: [OE-core] version upgrade to python-click

On 2/12/19 5:02 AM, Felix Snow wrote:
> Hello,
> 
> it seems that the latest version of python-click in open-embedded is 6.7 :
> 
> https://layers.openembedded.org/layerindex/recipe/52136/
> 
> This was released in January 2017, but 7.0 came out in September 2018:
> 
> https://pypi.org/project/click/#history
> 
> I have had a considerable struggle with click on our embedded linux 
> around locale.
> 
> It was a hard requirement and there are were a lot of complaints prior 
> to version 7.0
> 
> Here are a few examples:
> 
> https://github.com/NixOS/nixpkgs/pull/41843
> 
> https://github.com/NixOS/nixpkgs/issues/40146
> 
> https://github.com/NixOS/nixpkgs/issues/38991
> 
> I believe they removed the hard dependency in this pull request -
> https://github.com/pallets/click/pull/880
> 
> which went into version 7.0
> 
> If you look at the code of that request there is very little which 
> fixes this, but it would probably still be cleaner at least long term 
> to have the newer version of click than to patch that in our system.  
> The locale issue is one of a long list of changes in 7.0 
> http://click.palletsprojects.com/en/7.x/changelog/

Thanks for pointing that out.

Can you update the recipe, test it and send a patch to:
    openembedded-devel at lists.openembedded.org
as per:
    https://www.openembedded.org/wiki/How_to_submit_a_patch_to_OpenEmbedded

../Randy

> 
> Actually, click is not explicitly mentioned in the bb file, but 
> python-flask is, and that somehow installs click with it, I think as 
> there is another recipe for python-flask which includes click.
> 
> Cheers,
> 
> Felix
> 
> .......................................................................................
> 
> PHOENIX CONTACT Cyber Security GmbH
> 
> Richard-Willstätter-Straße 6
> 
> D-12489 Berlin
> 
> 　
> 
> Register Court: AG Charlottenburg, HR B 202908
> 
> Geschäftsführer/General Manager: Kilian Golm
> 
> //
> 


--
# Randy MacLeod
# Wind River Linux

.......................................................................................
PHOENIX CONTACT Cyber Security GmbH 
Richard-Willstätter-Straße 6  
D-12489 Berlin  
 
Register Court: AG Charlottenburg, HR B 202908 
Geschäftsführer/General Manager: Kilian Golm
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-upgrade-python-click-from-6.7-to-7.0.patch
Type: application/octet-stream
Size: 2324 bytes
Desc: 0001-upgrade-python-click-from-6.7-to-7.0.patch
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20190221/7f001e16/attachment-0001.obj>