Page MenuHomeSolus

Initial commit of python-distro
ClosedPublic

Authored by kyrios123 on Fri, Sep 6, 9:44 PM.

Details

Summary

Inclusion of distro in the repository

Test Plan

Tested with certbot (new dependency)

Diff Detail

Repository
R4812 python-distro
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ma-renaud created this revision.Fri, Sep 6, 9:44 PM
ma-renaud requested review of this revision.Fri, Sep 6, 9:44 PM
kyrios123 requested changes to this revision.Sat, Sep 7, 5:56 AM
kyrios123 added a subscriber: kyrios123.

Python 2 macros should be dropped, it's a dependency for conan which only requires python 3 (and python 2 support ends in 3 months anyway).

package.yml
15

I am pretty sure the package would build without this.

This revision now requires changes to proceed.Sat, Sep 7, 5:56 AM
kyrios123 commandeered this revision.Sun, Sep 8, 1:25 PM
kyrios123 edited reviewers, added: ma-renaud; removed: kyrios123.

This is a new dependency for certbot and since I already reworked this patch, I'll submit it so you don't have to redo the work.
Thanks for the submission anyway, it's appreciated ! 😃

kyrios123 updated this revision to Diff 17065.Sun, Sep 8, 1:28 PM
kyrios123 edited the summary of this revision. (Show Details)
kyrios123 edited the test plan for this revision. (Show Details)
  • Fix package dependencies
  • Drop uneeded python2 stuffs
  • Enable unit tests
JoshStrobl accepted this revision.Sun, Sep 8, 6:13 PM
JoshStrobl added a subscriber: JoshStrobl.

LGTM, thanks!

This revision is now accepted and ready to land.Sun, Sep 8, 6:13 PM
This revision was automatically updated to reflect the committed changes.