Skip to content

Commit

Permalink
Fix parser error for some r packages
Browse files Browse the repository at this point in the history
YAML with multiple empty entries in an array and comments can't be
parsed by ruamel yaml (NotImplementedError in
ruamel/yaml/tokens.py:84).

The recipe part causing issue looked like this:

```
requirements:
  build:
    - {{ compiler(cxx) }} #  [not win]
    - {{ compiler(c) }} #  [not win]
    - ...
```

after jinja like this:
```
requirements:
  build:
    -                      #  [not win]
    -                    #  [not win]
    - ...
```
  • Loading branch information
epruesse committed Dec 7, 2018
1 parent 1cb4423 commit 3cb4b1f
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions bioconda_utils/update.py
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
- [ ] deal with multi-output packages
- [ ] deal with linux/osx packages (selectors)
- [ ] deal with already open PRs
- [x] deal with already open PRs
- [x] handle remote/local git correctly
- [ ] collect editors from log
- [ ] collect author from GithubHoster
Expand Down Expand Up @@ -221,7 +221,8 @@ class HasSelector(RecipeError):

#: Variables to pass to Jinja when rendering recipe
JINJA_VARS = {
"cran_mirror": "https://cloud.r-project.org"
"cran_mirror": "https://cloud.r-project.org",
"compiler": lambda x: f"compiler_{x}",
}

#: Name of key under ``extra`` containing config
Expand Down

0 comments on commit 3cb4b1f

Please sign in to comment.