Fix parsing OPAM file with whitespace after key - Githubissues

yunxing / opam-npm

convert opam packages to npm

8 stars 12 forks source link

Fix parsing OPAM file with whitespace after key #5

Closed dittos closed 7 years ago

dittos commented 7 years ago

This bug caused uutf's conflicts to be included as dependency.

https://github.com/ocaml/opam-repository/blob/2ffce85/packages/uutf/uutf.0.9.4/opam

depopts: ["cmdliner"
          "cmdliner" {test} ]
conflicts : ["cmdliner" { < "0.9.6" } ]

-> https://github.com/npm-opam/uutf/blob/fdbc8eb/package.json#L43

    "dependencies": {
        ...
        "@opam-alpha/cmdliner": "< 0.9.6",
        ...
    },

chenglou commented 7 years ago

Seems legit. @yunxing