[libcamera-devel,v5,03/23] utils: ipc: add parser script
diff mbox series

Message ID 20201205103106.242080-4-paul.elder@ideasonboard.com
State Superseded
Delegated to: Paul Elder
Headers show
Series
  • IPA isolation implementation
Related show

Commit Message

Paul Elder Dec. 5, 2020, 10:30 a.m. UTC
On some systems, python2 might still be the default python. Enforce
python3 by wrapping the mojo parser script in a python3 script. This
also has the benefit of not modifying mojo.

Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

---
No change in v5

No change in v4

New in v3
---
 utils/ipc/parser.py | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)
 create mode 100755 utils/ipc/parser.py

Comments

Niklas Söderlund Dec. 18, 2020, 3:57 p.m. UTC | #1
Hi Paul,

Thanks for your patch.

On 2020-12-05 19:30:46 +0900, Paul Elder wrote:
> On some systems, python2 might still be the default python. Enforce
> python3 by wrapping the mojo parser script in a python3 script. This
> also has the benefit of not modifying mojo.
> 
> Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>

> 
> ---
> No change in v5
> 
> No change in v4
> 
> New in v3
> ---
>  utils/ipc/parser.py | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
>  create mode 100755 utils/ipc/parser.py
> 
> diff --git a/utils/ipc/parser.py b/utils/ipc/parser.py
> new file mode 100755
> index 00000000..f46820fa
> --- /dev/null
> +++ b/utils/ipc/parser.py
> @@ -0,0 +1,20 @@
> +#!/usr/bin/env python3
> +# SPDX-License-Identifier: BSD-3-Clause
> +# Copyright (C) 2020, Google Inc.
> +#
> +# Author: Paul Elder <paul.elder@ideasonboard.com>
> +#
> +# parser.py - Run mojo parser with python3
> +
> +import os
> +import sys
> +
> +# TODO set sys.pycache_prefix for >= python3.8
> +sys.dont_write_bytecode = True
> +
> +# Make sure that mojom_parser.py can import mojom
> +sys.path.append(f'{os.path.dirname(__file__)}/mojo/public/tools/mojom')
> +
> +import mojo.public.tools.mojom.mojom_parser as parser
> +
> +parser.Run(sys.argv[1:])
> -- 
> 2.27.0
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel@lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

Patch
diff mbox series

diff --git a/utils/ipc/parser.py b/utils/ipc/parser.py
new file mode 100755
index 00000000..f46820fa
--- /dev/null
+++ b/utils/ipc/parser.py
@@ -0,0 +1,20 @@ 
+#!/usr/bin/env python3
+# SPDX-License-Identifier: BSD-3-Clause
+# Copyright (C) 2020, Google Inc.
+#
+# Author: Paul Elder <paul.elder@ideasonboard.com>
+#
+# parser.py - Run mojo parser with python3
+
+import os
+import sys
+
+# TODO set sys.pycache_prefix for >= python3.8
+sys.dont_write_bytecode = True
+
+# Make sure that mojom_parser.py can import mojom
+sys.path.append(f'{os.path.dirname(__file__)}/mojo/public/tools/mojom')
+
+import mojo.public.tools.mojom.mojom_parser as parser
+
+parser.Run(sys.argv[1:])