| Message ID | 20200528125128.26915-2-cgzones@googlemail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v2,1/3] sepolgen: parse gen_tunable as bool | expand |
On Thu, May 28, 2020 at 8:52 AM Christian Göttsche <cgzones@googlemail.com> wrote: > > Signed-off-by: Christian Göttsche <cgzones@googlemail.com> This seems fine although I wonder if we should just do this inside of the o() definition and have it done for all callers. Acked-by: Stephen Smalley <stephen.smalley.work@gmail.com> > --- > python/sepolgen/src/sepolgen/refparser.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/python/sepolgen/src/sepolgen/refparser.py b/python/sepolgen/src/sepolgen/refparser.py > index f3e0ae87..9f850990 100644 > --- a/python/sepolgen/src/sepolgen/refparser.py > +++ b/python/sepolgen/src/sepolgen/refparser.py > @@ -1148,6 +1148,6 @@ def parse_headers(root, output=None, expand=True, debug=False): > status.step() > > if len(failures): > - o("failed to parse some headers: %s" % ", ".join(failures)) > + o("failed to parse some headers: %s\n" % ", ".join(failures)) > > return headers > -- > 2.27.0.rc2 >
diff --git a/python/sepolgen/src/sepolgen/refparser.py b/python/sepolgen/src/sepolgen/refparser.py index f3e0ae87..9f850990 100644 --- a/python/sepolgen/src/sepolgen/refparser.py +++ b/python/sepolgen/src/sepolgen/refparser.py @@ -1148,6 +1148,6 @@ def parse_headers(root, output=None, expand=True, debug=False): status.step() if len(failures): - o("failed to parse some headers: %s" % ", ".join(failures)) + o("failed to parse some headers: %s\n" % ", ".join(failures)) return headers
Signed-off-by: Christian Göttsche <cgzones@googlemail.com> --- python/sepolgen/src/sepolgen/refparser.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)