Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Bad version number in 2.1.0 #10

Closed
zoggy opened this issue Oct 14, 2016 · 2 comments · Fixed by #11
Closed

Bad version number in 2.1.0 #10

zoggy opened this issue Oct 14, 2016 · 2 comments · Fixed by #11

Comments

@zoggy
Copy link

zoggy commented Oct 14, 2016

After installing base64 with opam:

$ ocamlfind query base64 -format "%v"
v2.1.1

The version number is not the same (2.1.1 instead of 2.1.0) and having 'v' at the beginning of the version number is not standard.
hannesm added a commit to hannesm/ocaml-base64 that referenced this issue Oct 14, 2016
@hannesm
use VERSION_NUM instead of VERSION to get rid of leading 'v', partially
a47a6b0 
fixes mirage#10
@hannesm hannesm mentioned this issue Oct 14, 2016
Merged
@hannesm
Copy link
Member

hannesm commented Oct 14, 2016

I fixed the META issue in #11. The 2.1.1 vs 2.1.0 -- maybe we should rename the thing in opam-repository (the url of 2.1.0 there points to v2.1.1.tbz)?

@hannesm
Copy link
Member

hannesm commented Oct 17, 2016

this issue is not fixed, only a future release (or a local patch in opam-repository) will fix the META issue...

@hannesm hannesm reopened this Oct 17, 2016
samoht added a commit to samoht/opam-repository that referenced this issue Oct 18, 2016
@samoht
Remove base64 2.1.0 as it has not been released
583377a 
Which explains why the URL was wrong. Fixes mirage/ocaml-base64#10
@samoht samoht mentioned this issue Oct 18, 2016
Merged
dsheets pushed a commit to ocaml/opam-repository that referenced this issue Oct 18, 2016
@samoht @dsheets
Remove base64 2.1.0 as it has not been released (#7642)
86bc6a0 
Which explains why the URL was wrong. Fixes mirage/ocaml-base64#10
fdopen added a commit to fdopen/opam-repository-mingw that referenced this issue Oct 18, 2016
@fdopen
Remove base64 2.1.0 as it has not been released (#7642)
681024a 
Which explains why the URL was wrong. Fixes mirage/ocaml-base64#10
@hannesm hannesm mentioned this issue Nov 7, 2016
Merged
@zoggy zoggy mentioned this issue Nov 30, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

use VERSION_NUM instead of VERSION to get rid of leading 'v' hannesm/ocaml-base64
2 participants
@hannesm @zoggy