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 usenom-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.
ms705
commented
6 years ago
The version of
nomcurrently used bynom-sqlis 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 onregex 0.1, whereasregexis now at0.2(which is incompatible according to Rust's semver).regex 0.1in turn has a bunch of outdated dependencies (compare 0.1 to current). These in turn pull in more old dependencies, etc. For crates that usenom-sqlthis 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.