Bug 9636

Summary: wic script should use argparse instead of optparse library
Product: [Build System, Metadata & Runtime] OE-Core Reporter: Humberto Ibarra <humberto.ibarra.lopez>
Component: Scripts and ToolsAssignee: Daniela Plascencia <daniela.plascencia>
Status: RESOLVED FIXED QA Contact: David Rodriguez <david.israelx.rodriguez.castellanos>
Severity: enhancement    
Priority: Medium CC: eduard.bartosh, jose.perez.carranza, leonardo.sandoval.gonzalez, ross.burton
Version: 2.2   
Target Milestone: 2.4 M3   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)

Description Humberto Ibarra 2016-05-14 19:41:45 UTC
Similar issue to the one described in #8321, only this applies to wic instead of yocto-bsp script.

optparse library is deprecated since 2.7, scripts should use argparse instead.
Comment 1 Leonardo Sandoval Gonzalez 2016-06-16 16:53:23 UTC
Humberto, optparse in python3 is still available so priority is a bit lower on this.
Comment 2 Leonardo Sandoval Gonzalez 2016-06-21 20:35:58 UTC
some related info on this transition https://docs.python.org/2/library/argparse.html#upgrading-optparse-code
Comment 3 Leonardo Sandoval Gonzalez 2016-06-23 19:55:08 UTC
My fault, moving it to M1 again. CCB should agree on any milestone movement.
Comment 4 Leonardo Sandoval Gonzalez 2017-01-25 20:29:53 UTC
This is not a priority right now so moving it to next release.
Comment 5 Ed Bartosh 2017-05-02 14:56:03 UTC
The patch has been sent to the mailiing list for review: http://lists.openembedded.org/pipermail/openembedded-core/2017-May/136235.html
Comment 6 Leonardo Sandoval Gonzalez 2017-05-02 15:15:20 UTC
(In reply to comment #5)
> The patch has been sent to the mailiing list for review:
> http://lists.openembedded.org/pipermail/openembedded-core/2017-May/136235.
> html

Thanks Ed for updating with this info.

Daniela, can you please do a quick check of the patchset?
Comment 7 Ed Bartosh 2017-05-24 08:23:09 UTC
The patchset has been merged to master. commit id in poky: 05aa049e2c314731385e1d18b5fd6e1f1e63ea8f