Bug 10411

Summary: Provide a working python-pylint recipe
Product: [Yocto Project Subprojects] Patchwork/Patchtest Reporter: Leonardo Sandoval Gonzalez <leonardo.sandoval.gonzalez>
Component: Patchtest TestsuiteAssignee: Daniela Plascencia <daniela.plascencia>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: Medium+ CC: alejandro.hernandez, mhalstead
Version: 2.2   
Target Milestone: 2.3 M1   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)
Bug Depends on:    
Bug Blocks: 10383    

Description Leonardo Sandoval Gonzalez 2016-10-11 15:32:49 UTC 
There is a recipe on meta-oe [1] defining python-pylint module. However, when using it on target, some run dependencies are missing [2]. As [3], we need to provide a working recipe, tested with the patchtest environment.

This recipe is not part of oe-core, but patchtest-oe uses it, so I find it convenient to track this.

[1] http://cgit.openembedded.org/meta-openembedded/commit/?id=ee23e9d9792a0a923294144a5f5b54a67b74eb84
[2] http://lists.openembedded.org/pipermail/openembedded-devel/2016-September/109016.html
[3] https://bugzilla.yoctoproject.org/show_bug.cgi?id=10410
Comment 1 Alejandro Hernandez 2016-10-13 22:42:22 UTC 
If this is used by patchtest-oe perhaps we should consider putting it on oe-core.
Comment 2 Leonardo Sandoval Gonzalez 2016-10-14 17:35:04 UTC 
(In reply to comment #1)
> If this is used by patchtest-oe perhaps we should consider putting it on
> oe-core.

Actually, the module is already on meta-openembedded but after I sent it I realized that it did not work, then I replied indicating that a v2 will be sent asap. At the end, the v1 was merged so the step here is to fix it. This is in fact a bug, not a enhancement.
Comment 3 Leonardo Sandoval Gonzalez 2016-10-19 14:05:09 UTC 
We have realized that pylint has several dependencies and the corresponding recipes does not exist (at any tracked openembedded layer). pylint is used on a single test case so it does not affect much of the patchtest-oe test suite. This work is currenlty being done but it may not be ready for 2.2 M4. It can be moved to 2.3M1 and should be finishied early M1.
Comment 4 Daniela Plascencia 2016-11-28 16:55:07 UTC 
(In reply to comment #3)
> We have realized that pylint has several dependencies and the corresponding
> recipes does not exist (at any tracked openembedded layer). pylint is used
> on a single test case so it does not affect much of the patchtest-oe test
> suite. This work is currenlty being done but it may not be ready for 2.2 M4.
> It can be moved to 2.3M1 and should be finishied early M1.

At the moment, missing runtime dependencies are:
- python-astroid
- python-wrapt
- python-backports.functools-lru-cache
- python-lazy-proxy-object
Comment 6 Leonardo Sandoval Gonzalez 2016-12-12 14:44:37 UTC 
(In reply to comment #5)
> The series solving pylint's build and runtime dependencies is already merged
> in meta-openembedded [1-7], but the package has to be tested within the
> patchtest environment before we can close this bug.
> 
> [1]
> http://cgit.openembedded.org/meta-openembedded/commit/
> ?id=d9b5870edaddee69612dada2aa4bc2d8c3f70f44
> [2]http://cgit.openembedded.org/meta-openembedded/commit/
> ?id=d83612a1e750b98b1c537fada777d615d9d6628a
> [3]http://cgit.openembedded.org/meta-openembedded/commit/
> ?id=3aac48c722fa836e04e95a495d758b8ee0d0cdda
> [4]http://cgit.openembedded.org/meta-openembedded/commit/
> ?id=8ea8fff79c91c985884859eca49769379e06c313
> [5]http://cgit.openembedded.org/meta-openembedded/commit/
> ?id=f5a5a84d5c551a3b1daa2c8092279fbed9abe54f
> [6]http://cgit.openembedded.org/meta-openembedded/commit/
> ?id=1b73d2c713938f10931749ece3a703ad9004376a
> [7]http://cgit.openembedded.org/meta-openembedded/commit/
> ?id=d356e03ced0873a0fe1a7a69cb05ee803e363888

Daniela. lets close this one related to the recipe itself. I will file another one related to patchtest & pylint test case.