skvadrik / re2c

Lexer generator for C, C++, Go and Rust.

Home Page:https://re2c.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Version number of re2c-2.0 is still 1.3

marryjianjian opened this issue · comments

project(re2c VERSION 1.3 HOMEPAGE_URL "https://re2c.org/")

I'm glad to see re2c-2.0 released (supported generate go! Wowwwww!!)
when I compile re2c-2.0 by myself, I wonder which verison I'm using. And I found -v prints 1.3
image

Maybe someone just forgot. But as a new big version, it may cause misunderstanding.

Thanks! I fixed this in 9311e52 (forgot to update the release script for the new CMake build system) and released minor bugfix release 2.0.1 to avoid confusion: https://re2c.org/releases/release_notes.html#release-2-0-1.

Well, I don't know how release files packed. But it seems that re2c-2.0.1.tar.xz 's files different from others (2.0.1.zip and 2.0.1.tar.gz ).
There is no CMake build system in re2c-2.0.1.tar.xz.

True, CMake files are missing from the release tarball re2c-2.0.1.tar.xz. This is a consequence of supporting two build systems: release tarball is packed with Autotools (make dist), and it does not need CMake files. I need to add them in EXTRA_DIST.

Zip files are different because they are packed automatically with Github (when I push release commit with a tag).

Thanks!
And I have another question about CMake build system in Build instructions which I didn't find it in this repository.
I found that RE2C_BUILD_RE2GO is needed rather than the option RE2C_BUILD_GOLANG mentioned in above instructions to build binary re2go. It seems it's typo but it does not affect use go backend with --lang go

Good catch, I corrected the error in documentation. I also pushed a fix that adds CMake files to the distribution tarball: 77da086, and another fix that enables Go support by default: ec654f1 (but it still can be turned off with a configuration option).

I plan to make a bugfix release 2.0.2 in about a week, to wait if any more bugs get reported. Thanks for your help!

I did't find any problem further more.

Maybe it's time to close this issue.

Thanks you help!