search

robshakir / pyangbind

A plugin for pyang that creates Python bindings for a YANG model.
Other
198 stars 120 forks source link

Failure on OpenConfig BFD Model #302

Closed wenovus closed 11 months ago

wenovus commented 11 months ago

There is a failure when I generate against the latest OpenConfig BFD model: https://gist.github.com/OpenConfigBot/f2158a52bc907ed4e690582d47f2af61

File "/workspace/bfd.openconfig-bfd.binding.py", line 2320
    async = __builtin__.property(_get_async)
          ^
SyntaxError: invalid syntax

You can reproduce with the following command:

OC_WORKSPACE=</path/to/oc/public/repo>

pyang --plugindir $PYANGBIND_PLUGINDIR -o bfd.openconfig-bfd.binding.py -f pybind -p $OC_WORKSPACE/release/models -p $OC_WORKSPACE/third_party/ietf $OC_WORKSPACE/release/models/bfd/openconfig-bfd.yang

Can you help take a look? grepping for async didn't give me anything in the repo.

JoseIgnacioTamayo commented 11 months ago

Hi Wen,

Yes, we know of this problem. See https://github.com/robshakir/pyangbind/pull/285.

Soon we will be publishing this and other fixes for pyangbind. Have just a little patience.

Feel free to comment on the GitHub issue to record the fact that the bug affects you.

Thanks, and all the best.

Jose Ignacio Tamayo Segarra LinkedIn http://br.linkedin.com/in/joseignaciotamayosegarra

On Fri, Jul 14, 2023 at 5:01 PM Wen Bo Li @.***> wrote:

There is a failure when I generate against the latest OpenConfig BFD model: https://gist.github.com/OpenConfigBot/f2158a52bc907ed4e690582d47f2af61

File "/workspace/bfd.openconfig-bfd.binding.py", line 2320 async = builtin.property(_get_async) ^ SyntaxError: invalid syntax

You can reproduce with the following command:

OC_WORKSPACE=</path/to/oc/public/repo>

pyang --plugindir $PYANGBIND_PLUGINDIR -o bfd.openconfig-bfd.binding.py -f pybind -p $OC_WORKSPACE/release/models -p $OC_WORKSPACE/third_party/ietf $OC_WORKSPACE/release/models/bfd/openconfig-bfd.yang

Can you help take a look? grepping for async didn't give me anything in the repo.

— Reply to this email directly, view it on GitHub https://github.com/robshakir/pyangbind/issues/302, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFAWOWMSHN2SS22NGTHALMDXQFNMHANCNFSM6AAAAAA2KOE73A . You are receiving this because you are subscribed to this thread.Message ID: @.***>

wenovus commented 11 months ago

Thanks Jose

JoseIgnacioTamayo commented 11 months ago

Fixed in https://github.com/robshakir/pyangbind/pull/285