Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Norway problem in yaml output from xkbcli list #268

Closed
hickford opened this issue Oct 31, 2021 · 2 comments · Fixed by #269
Closed

Norway problem in yaml output from xkbcli list #268

hickford opened this issue Oct 31, 2021 · 2 comments · Fixed by #269

Comments

@hickford
Copy link
Contributor

Anyone who uses YAML long enough will eventually get burned when attempting to abbreviate Norway.

https://news.ycombinator.com/item?id=17359376

YAML output from xkbcli list

- layout: no
  variant: ''
  brief: no
  description: Norwegian
  iso639: [nor, nob, nno]
  iso3166: []

Output of PyYAML's yaml.safe_load

{'layout': False,
 'variant': '',
 'brief': False,
 'description': 'Norwegian',
 'iso639': ['nor', 'nob', 'nno'],
 'iso3166': []},
@hickford
Copy link
Contributor Author

Turns out this is a Yaml 1.1 problem fixed in 1.2. PyYAML only supports the older yaml/pyyaml#116

Better to use parser ruamel.yaml https://yaml.readthedocs.io/en/latest/

@bluetech
Copy link
Member

bluetech commented Nov 1, 2021

Heh that's a funny problem. I've made a change to mention YAML 1.2 in the manpage: e8cb431

whot added a commit to whot/libxkbcommon that referenced this issue Nov 8, 2021
@whot
tools: enclose the layout name in quotes
4834f3d 
Because otherwise the 'no' layout is treated as disagreement with whatever is to
be disagreed with. Fixed in YAML 1.2 but that's not universally supported.

Fixes xkbcommon#268
@whot whot mentioned this issue Nov 8, 2021
Merged
whot added a commit to whot/libxkbcommon that referenced this issue Nov 10, 2021
@whot
tools/list: enclose the the various field names in quotes
c52109c 
Because otherwise the 'no' layout is treated as disagreement with whatever is to
be disagreed with. Fixed in YAML 1.2 but that's not universally supported.

Fixes xkbcommon#268
whot added a commit that referenced this issue Nov 10, 2021
@whot
tools/list: enclose the the various field names in quotes
abb115c 
Because otherwise the 'no' layout is treated as disagreement with whatever is to
be disagreed with. Fixed in YAML 1.2 but that's not universally supported.

Fixes #268
wismill pushed a commit to wismill/libxkbcommon that referenced this issue Dec 16, 2024
@whot @wismill
tools/list: enclose the the various field names in quotes
bb55613 
Because otherwise the 'no' layout is treated as disagreement with whatever is to
be disagreed with. Fixed in YAML 1.2 but that's not universally supported.

Fixes xkbcommon#268
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tools: enclose the layout name in quotes whot/libxkbcommon
2 participants
@hickford @bluetech