-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Does not parse some versions properly #4
Comments
I think one issues is that SemVer incorrectly suggests that periods can't be allowed in the pre-release component although their examples have them. |
Testing using OSX FAIL./semver.sh 1.0.2-rc.1 1.0.2
1.0.2-rc.1 -> M: 1.1 m:0.1 p:2.1 s:-rc.1
1.0.2 -> M: 1 m:0 p:2 s:
./semver.sh: line 29: [: 1.1: integer expression expected
./semver.sh: line 33: [: 0.1: integer expression expected
./semver.sh: line 37: [: 2.1: integer expression expected
1.0.2-rc.1 == 1.0.2 -> 1.
./semver.sh: line 64: [: 1.1: integer expression expected
./semver.sh: line 68: [[: 1.1: syntax error: invalid arithmetic operator (error token is ".1")
./semver.sh: line 72: [[: 1.1: syntax error: invalid arithmetic operator (error token is ".1")
1.0.2-rc.1 < 1.0.2 -> 0.
./semver.sh: line 29: [: 1.1: integer expression expected
./semver.sh: line 33: [: 0.1: integer expression expected
./semver.sh: line 37: [: 2.1: integer expression expected
./semver.sh: line 64: [: 1.1: integer expression expected
./semver.sh: line 68: [[: 1.1: syntax error: invalid arithmetic operator (error token is ".1")
./semver.sh: line 72: [[: 1.1: syntax error: invalid arithmetic operator (error token is ".1")
1.0.2-rc.1 > 1.0.2 -> 1. 1.0.2-rc.1 == 1.0.2 -> 1 I created a simplified semver compare in bash here: https://gist.github.com/Ariel-Rodriguez/9e3c2163f4644d7a389759b224bfe7f3
|
Example:
5.5.9-1ubuntu4.1
.The text was updated successfully, but these errors were encountered: