parsing not robust
Bug #1477875 reported by
Olli Ries
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Snapcraft |
Fix Released
|
Medium
|
Sergio Schvezov | ||
Bug Description
my package.yaml has a \t in the last empty line and snapcraft exists with:
yaml.scanner.
found character '\t' that cannot start any token
in "snap/meta/
while correct, I think it shouldn't bail on an (apparently) empty line
Related branches
lp:~sergiusens/snapcraft/yaml-tabs
- Ted Gould (community): Approve
-
Diff: 47 lines (+29/-0)2 files modifiedsnapcraft/tests/test_yaml.py (+25/-0)
snapcraft/yaml.py (+4/-0)
Revision history for this message
| Olli Ries (ories) wrote | #1 |
| tags: | added: first-steps |
Revision history for this message
| Sergio Schvezov (sergiusens) wrote | #2 |
| Changed in snapcraft: | |
| status: | New → Fix Committed |
| Changed in snapcraft: | |
| milestone: | none → 0.2 |
| assignee: | nobody → Sergio Schvezov (sergiusens) |
| Changed in snapcraft: | |
| importance: | Undecided → Medium |
| Changed in snapcraft: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
But according to the yaml spec it is invalid, I can make it exit cleanly