summaryrefslogtreecommitdiffstats
path: root/python/makevars.py
diff options
context:
space:
mode:
authorwhitespace <nobody@nobody>2020-10-07 23:22:26 +0200
committerQuentin Young <qlyoung@nvidia.com>2020-10-07 23:22:26 +0200
commit701a01920eee5431d2052aad92aefbdf50ac2139 (patch)
tree2bf2339327241f59593b9583b060ebb347db1cea /python/makevars.py
parentMerge pull request #7058 from Niral-Networks/niral_dev_vrf_ospf6 (diff)
downloadfrr-701a01920eee5431d2052aad92aefbdf50ac2139.tar.xz
frr-701a01920eee5431d2052aad92aefbdf50ac2139.zip
*: reformat python files
We are now using black. Signed-off-by: Quentin Young <qlyoung@nvidia.com>
Diffstat (limited to 'python/makevars.py')
-rw-r--r--python/makevars.py48
1 files changed, 29 insertions, 19 deletions
diff --git a/python/makevars.py b/python/makevars.py
index 63bf8c5ee..951cd3438 100644
--- a/python/makevars.py
+++ b/python/makevars.py
@@ -6,10 +6,12 @@ import os
import subprocess
import re
+
class MakeVarsBase(object):
- '''
+ """
common code between MakeVars and MakeReVars
- '''
+ """
+
def __init__(self):
self._data = dict()
@@ -18,31 +20,35 @@ class MakeVarsBase(object):
self.getvars([k])
return self._data[k]
- def get(self, k, defval = None):
+ def get(self, k, defval=None):
if k not in self._data:
self.getvars([k])
return self._data.get(k) or defval
+
class MakeVars(MakeVarsBase):
- '''
+ """
makevars['FOO_CFLAGS'] gets you "FOO_CFLAGS" from Makefile
This variant works by invoking make as a subprocess, i.e. Makefile must
be valid and working. (This is sometimes a problem if depfiles have not
been generated.)
- '''
+ """
+
def getvars(self, varlist):
- '''
+ """
get a batch list of variables from make. faster than individual calls.
- '''
+ """
rdfd, wrfd = os.pipe()
- shvars = ['shvar-%s' % s for s in varlist]
- make = subprocess.Popen(['make', '-s', 'VARFD=%d' % wrfd] + shvars, pass_fds = [wrfd])
+ shvars = ["shvar-%s" % s for s in varlist]
+ make = subprocess.Popen(
+ ["make", "-s", "VARFD=%d" % wrfd] + shvars, pass_fds=[wrfd]
+ )
os.close(wrfd)
- data = b''
+ data = b""
- rdf = os.fdopen(rdfd, 'rb')
+ rdf = os.fdopen(rdfd, "rb")
while True:
rdata = rdf.read()
if len(rdata) == 0:
@@ -52,30 +58,34 @@ class MakeVars(MakeVarsBase):
del rdf
make.wait()
- data = data.decode('US-ASCII').strip().split('\n')
+ data = data.decode("US-ASCII").strip().split("\n")
for row in data:
- k, v = row.split('=', 1)
+ k, v = row.split("=", 1)
v = v[1:-1]
self._data[k] = v
+
class MakeReVars(MakeVarsBase):
- '''
+ """
makevars['FOO_CFLAGS'] gets you "FOO_CFLAGS" from Makefile
This variant works by regexing through Makefile. This means the Makefile
does not need to be fully working, but on the other hand it doesn't support
fancy complicated make expressions.
- '''
- var_re = re.compile(r'^([^=#\n\s]+)[ \t]*=[ \t]*([^#\n]*)(?:#.*)?$', flags=re.MULTILINE)
- repl_re = re.compile(r'\$(?:([A-Za-z])|\(([^\)]+)\))')
+ """
+
+ var_re = re.compile(
+ r"^([^=#\n\s]+)[ \t]*=[ \t]*([^#\n]*)(?:#.*)?$", flags=re.MULTILINE
+ )
+ repl_re = re.compile(r"\$(?:([A-Za-z])|\(([^\)]+)\))")
def __init__(self, maketext):
super(MakeReVars, self).__init__()
- self._vars = dict(self.var_re.findall(maketext.replace('\\\n', '')))
+ self._vars = dict(self.var_re.findall(maketext.replace("\\\n", "")))
def replacevar(self, match):
varname = match.group(1) or match.group(2)
- return self._vars.get(varname, '')
+ return self._vars.get(varname, "")
def getvars(self, varlist):
for varname in varlist: