annet 1.1.1__tar.gz → 1.1.2__tar.gz

{annet-1.1.1/annet.egg-info → annet-1.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: annet
-Version: 1.1.1
+Version: 1.1.2
 Summary: annet
 Home-page: https://github.com/annetutil/annet
 License: MIT
@@ -14,7 +14,6 @@ Requires-Dist: jsonpointer>=2.4
 Requires-Dist: PyYAML>=6.0.1
 Requires-Dist: Pygments>=2.14.0
 Requires-Dist: Mako>=1.2.4
-Requires-Dist: Jinja2>=3.1.2
 Requires-Dist: packaging>=23.2
 Requires-Dist: contextlog>=1.1
 Requires-Dist: valkit>=0.1.4

{annet-1.1.1 → annet-1.1.2}/annet/annlib/lib.py

@@ -18,7 +18,6 @@ from functools import lru_cache
 from typing import List, NamedTuple, Optional, Tuple, Union
 
 import contextlog
-import jinja2
 import mako.template
 
 _logger = contextlog.get_logger()
@@ -292,19 +291,6 @@ def mako_render(template, dedent=False, **kwargs):
     return ret
 
 
-def jinja_render(template, dedent=False, **kwargs):
-    @lru_cache(None)
-    def _compile_jinja(template, dedent):
-        if dedent:
-            template = textwrap.dedent(template).strip()
-        return jinja2.Template(template)
-
-    ret = _compile_jinja(template, dedent).render(**kwargs)
-    if dedent:
-        ret = ret.strip()
-    return ret
-
-
 # =====
 def find_exc_in_stack(
     container_exc: Exception,

{annet-1.1.1 → annet-1.1.2}/annet/annlib/rbparser/platform.py

@@ -25,7 +25,7 @@ VENDOR_DIFF = {
     "routeros": "common.default_diff",
     "aruba": "aruba.default_diff",
     "pc": "common.default_diff",
-    "ribbon": "common.default_diff",
+    "ribbon": "juniper.default_diff",
     "b4com": "common.default_diff",
 }
 
@@ -40,7 +40,7 @@ VENDOR_DIFF_ORDERED = {
     "routeros": "common.ordered_diff",
     "aruba": "common.ordered_diff",
     "pc": "common.ordered_diff",
-    "ribbon": "common.ordered_diff",
+    "ribbon": "juniper.ordered_diff",
     "b4com": "common.ordered_diff",
 }
 

{annet-1.1.1 → annet-1.1.2}/annet/annlib/tabparser.py

@@ -387,6 +387,24 @@ class AsrFormatter(BlockExitFormatter):
             yield from super().block_exit(context)
 
 
+class JuniperPatch:
+    def __init__(self):
+        """In the case of comments, odict is not suitable: there may be several identical edit and exit"""
+        self._items = []
+
+    def __setitem__(self, key, value):
+        self._items.append((key, value))
+
+    def keys(self):
+        return list(self)
+
+    def items(self):
+        return self._items
+
+    def __iter__(self):
+        return iter(item[0] for item in self._items)
+
+
 class JuniperFormatter(CommonFormatter):
     patch_set_prefix = "set"
 
@@ -399,7 +417,7 @@ class JuniperFormatter(CommonFormatter):
         comment: str
 
         def __post_init__(self):
-            self.row = self.row.strip()
+            self.row = " ".join(map(lambda x: x.strip("\"'"), self.row.strip().split(" ")))
             self.comment = self.comment.strip()
 
         @classmethod
@@ -485,7 +503,8 @@ class JuniperFormatter(CommonFormatter):
             yield line + self._statement_end
 
     def cmd_paths(self, patch, _prev=tuple()):
-        commands = odict()
+        commands = JuniperPatch()
+
         for item in patch.itms:
             key, childs, context = item.row, item.child, item.context
 
@@ -497,33 +516,39 @@ class JuniperFormatter(CommonFormatter):
                     value = (
                         ""
                         if key.startswith("delete")
-                        else context["comment"]
+                        else key.removeprefix(self.Comment.begin).removesuffix(self.Comment.end).strip()
                     )
-
-                    cmd = "\n".join(
-                        (
-                            "edit " + " ".join(_prev),
-                            " ".join(("annotate", context["row"].split(" ")[0], f'"{value}"')),
-                            "exit"
-                        )
+                    cmds = (
+                        f"edit {' '.join(_prev)}",
+                        " ".join(("annotate", context["row"].split(" ")[0], f'"{value}"')),
+                        "exit"
                     )
                 elif key.startswith("delete"):
-                    cmd = " ".join(("delete", *_prev, key.replace("delete", "", 1).strip()))
+                    cmds = (
+                        " ".join(("delete", *_prev, key.replace("delete", "", 1).strip())),
+                    )
                 elif key.startswith("activate"):
-                    cmd = " ".join(("activate", *_prev, key.replace("activate", "", 1).strip()))
+                    cmds = (
+                        " ".join(("activate", *_prev, key.replace("activate", "", 1).strip())),
+                    )
                 elif key.startswith("deactivate"):
-                    cmd = " ".join(("deactivate", *_prev, key.replace("deactivate", "", 1).strip()))
+                    cmds = (
+                        " ".join(("deactivate", *_prev, key.replace("deactivate", "", 1).strip())),
+                    )
                 else:
-                    cmd = " ".join((self.patch_set_prefix, *_prev, key.strip()))
+                    cmds = (
+                        " ".join((self.patch_set_prefix, *_prev, key.strip())),
+                    )
 
                 # Expanding [ a b c ] junipers list of arguments
-                if matches := re.search(r"^(.*)\s+\[(.+)\]$", cmd):
-                    for c in matches.group(2).split(" "):
-                        if c.strip():
-                            cmd = " ".join([matches.group(1), c])
-                            commands[(cmd,)] = context
-                else:
-                    commands[(cmd,)] = context
+                for cmd in cmds:
+                    if matches := re.search(r"^(.*)\s+\[(.+)\]$", cmd):
+                        for c in matches.group(2).split(" "):
+                            if c.strip():
+                                items = " ".join([matches.group(1), c])
+                                commands[(items,)] = context
+                    else:
+                        commands[(cmd,)] = context
 
         return commands
 

{annet-1.1.1 → annet-1.1.2}/annet/api/__init__.py

@@ -700,7 +700,7 @@ async def adeploy(
         ans = deployer.ask_deploy()
         if ans != "y":
             return 2 ** 2
-        progress_bar = None
+
         if sys.stdout.isatty() and not args.no_progress:
             progress_bar = annet.deploy_ui.ProgressBars(odict([(device.fqdn, {}) for device in deploy_cmds]))
             progress_bar.init()

{annet-1.1.1 → annet-1.1.2}/annet/generators/entire.py

@@ -14,7 +14,6 @@ from typing import (
 
 from annet.lib import (
     flatten,
-    jinja_render,
     mako_render,
 )
 from .base import BaseGenerator, _filter_str
@@ -70,9 +69,6 @@ class Entire(BaseGenerator):
     def mako(self, text, **kwargs) -> str:
         return mako_render(text, dedent=True, device=self.__device, **kwargs)
 
-    def jinja(self, text, **kwargs) -> str:
-        return jinja_render(text, dedent=True, device=self.__device, **kwargs)
-
     # =====
 
     @classmethod

{annet-1.1.1 → annet-1.1.2}/annet/lib.py

@@ -24,7 +24,6 @@ from annet.annlib.lib import (  # pylint: disable=unused-import
     huawei_expand_vlandb,
     huawei_iface_ranges,
     is_relative,
-    jinja_render,
     jun_activate,
     jun_is_inactive,
     juniper_fmt_prefix_lists_acl,
@@ -151,10 +150,15 @@ def do_async(coro: Awaitable[ReturnType], new_thread=False) -> ReturnType:
 
         def wrapper(main):
             nonlocal res
-            res = asyncio.run(main)
+            try:
+                res = asyncio.run(main)
+            except BaseException as e:
+                res = e
         thread = threading.Thread(target=wrapper, args=(coro,))
         thread.start()
         thread.join()
+        if isinstance(res, BaseException):
+            raise res
         return res
     else:
         return asyncio.run(coro)

{annet-1.1.1 → annet-1.1.2}/annet/rulebook/juniper/__init__.py

@@ -12,8 +12,8 @@ def comment_processor(item: common.DiffItem):
     if item.op in (Op.REMOVED, Op.ADDED) and item.row.startswith(JuniperFormatter.Comment.begin):
         comment = JuniperFormatter.Comment.loads(item.row)
 
+        item.diff_pre["attrs"]["context"]["comment"] = True
         item.diff_pre["attrs"]["context"]["row"] = comment.row
-        item.diff_pre["attrs"]["context"]["comment"] = comment.comment
         item.diff_pre["key"] = item.diff_pre["raw_rule"] = (
             f"{JuniperFormatter.Comment.begin} {comment.row} {JuniperFormatter.Comment.end}"
         )

{annet-1.1.1 → annet-1.1.2/annet.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: annet
-Version: 1.1.1
+Version: 1.1.2
 Summary: annet
 Home-page: https://github.com/annetutil/annet
 License: MIT
@@ -14,7 +14,6 @@ Requires-Dist: jsonpointer>=2.4
 Requires-Dist: PyYAML>=6.0.1
 Requires-Dist: Pygments>=2.14.0
 Requires-Dist: Mako>=1.2.4
-Requires-Dist: Jinja2>=3.1.2
 Requires-Dist: packaging>=23.2
 Requires-Dist: contextlog>=1.1
 Requires-Dist: valkit>=0.1.4

{annet-1.1.1 → annet-1.1.2}/annet.egg-info/requires.txt

@@ -5,7 +5,6 @@ jsonpointer>=2.4
 PyYAML>=6.0.1
 Pygments>=2.14.0
 Mako>=1.2.4
-Jinja2>=3.1.2
 packaging>=23.2
 contextlog>=1.1
 valkit>=0.1.4

{annet-1.1.1 → annet-1.1.2}/requirements.txt

@@ -5,7 +5,6 @@ jsonpointer>=2.4
 PyYAML>=6.0.1
 Pygments>=2.14.0
 Mako>=1.2.4
-Jinja2>=3.1.2
 packaging>=23.2
 contextlog>=1.1
 valkit>=0.1.4