Page MenuHomeSolus

Update pyyaml to 5.1.2
ClosedPublic

Authored by kyrios123 on Aug 10 2019, 3:36 PM.

Details

Summary

News available on this page
Depends on D6917

Signed-off-by: Pierre-Yves <pyu@riseup.net>

Test Plan

rebuild packages with ypkg

Diff Detail

Repository
R2718 pyyaml
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

kyrios123 created this revision.Aug 10 2019, 3:36 PM
kyrios123 requested review of this revision.Aug 10 2019, 3:36 PM

I saw this but it didn't cause the script is working normally

.././common/Scripts/gettag.py:22: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
 y = yaml.load(f)
JoshStrobl accepted this revision.Aug 13 2019, 12:01 PM
JoshStrobl added a subscriber: JoshStrobl.

I saw this but it didn't cause the script is working normally

.././common/Scripts/gettag.py:22: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
 y = yaml.load(f)

Yea let's get this patch landed and I'll go ahead and update the script to use a specific loader choice per the new API.

This revision is now accepted and ready to land.Aug 13 2019, 12:01 PM
This revision was automatically updated to reflect the committed changes.