-
Notifications
You must be signed in to change notification settings - Fork 475
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add capability metadata for attributes #794
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps also add validation each enum capabilities value in the list start with the prefix of the attribute name
There is validation already on that, so at specific attribute only corresponding enum values can be added, or you are talking about different check ? Please put any file ending on *.cap in meta folder and remove @Skip tag from that file, and type "make" this will allow to test your scenario |
I cloned meta dir from your fork, metacap branch Creating serialzie for sai_acl_resource_t Use of uninitialized value in list assignment at serialize.pm line 363 (#2)
Any idea what is wrong ? |
P.S. : otherwise I see there is already good validation for using incorrect enum value |
As you can see build succeeded on this branch, so probably you are mergin this commit with your code which is not synced with master? |
No description provided.