Re: [osv-dev] [PATCH] scripts: upgrade to python 3 - part 1

2020-02-19 Thread Waldek Kozaczuk
I like the idea and this by the was what we use in this shell script - https://github.com/cloudius-systems/osv/blob/69486729cde2a991335ad4d155d274b32a966231/scripts/build-capstan-mpm-packages#L93-L101. Now how standard is /etc/os-release? Can we rely on it be present on all linux

Re: [osv-dev] [PATCH] scripts: upgrade to python 3 - part 1

2020-02-19 Thread Nadav Har'El
On Thu, Feb 20, 2020 at 12:28 AM Nadav Har'El wrote: > On Wed, Feb 19, 2020 at 12:52 AM Waldek Kozaczuk > wrote: > >> So I think I have a small subset of this patch ready that encompassed >> everything from the original path but loader.py and trace related files. >> >> But I just realized a

Re: [osv-dev] [PATCH] scripts: upgrade to python 3 - part 1

2020-02-19 Thread Nadav Har'El
On Wed, Feb 19, 2020 at 12:52 AM Waldek Kozaczuk wrote: > So I think I have a small subset of this patch ready that encompassed > everything from the original path but loader.py and trace related files. > > But I just realized a problem with updates setup.py. It now depends on > distro package -

Re: [osv-dev] [PATCH] scripts: upgrade to python 3 - part 1

2020-02-19 Thread Fotis Xenakis
Since there is no easy and robust way to determine the distro with python >=3.8 (when platform.linux_distribution() was removed), I see mostly two ways forward: - Just require pip for running setup.py, which I personally find reasonable (python is an established requirement anyway). -

[osv-dev] [COMMIT osv master] scripts: upgrade to python 3 - part 1.1

2020-02-19 Thread Commit Bot
From: Waldemar Kozaczuk Committer: Waldemar Kozaczuk Branch: master scripts: upgrade to python 3 - part 1.1 This patch incorporates changes from original patch submitted by Matt Bass as well as extra changes made by Waldemar Kozaczuk to upgrade python scripts from version 2 to version 3. Some