Upgrade to latest version of nom
jonhoo opened this issue · comments
The version of nom currently used by nom-sql is fairly old. In and of itself, this isn't so bad (though the newer nom presumably has performance and correctness fixes), but it causes some pain with dependencies. Specifically, the old version of num depends on regex 0.1, whereas regex is now at 0.2 (which is incompatible according to Rust's semver). regex 0.1 in turn has a bunch of outdated dependencies (compare 0.1 to current). These in turn pull in more old dependencies, etc. For crates that use nom-sql this causes many dependencies to be pulled in twice (one old, one new), which has a detrimental effect on compilation time and space usage.
I know fixing this is a bit annoying, as the nom api has changed a fair bit, but it's probably eventually worth doing.
Done as per #14!