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
Add version management #90
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cannot generate new build after removed
rm build/bin/qitmeer-wallet
make
make: `build' is up to date.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One suggestion
By your current implementation , we cannot view qitmeer version and the binary location by make command as Qitmeer does.
Done building.
qitmeer version 0.9.2+dev-ac02016 (Go version go1.16.3))
Run "./build/bin/qitmeer" to launch.
So if I made a build before, there is no way for me to find where the build is.
make: `build' is up to date.
Makefile
Outdated
|
||
EXECUTABLES=$(UNIX_EXECUTABLES) $(WIN_EXECUTABLES) | ||
|
||
COMPRESSED_EXECUTABLES=$(UNIX_EXECUTABLES:%=%.tar.gz) $(WIN_EXECUTABLES:%.exe=%.zip) $(WIN_EXECUTABLES:%.exe=%.cn.zip) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you don't need .*win.cn
package, it exists for the historical reason to contain different startup scripts and useful only for the qitmeer
. please cleanup the unnecessary compressed package.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
Makefile
Outdated
%.cn.zip: %.exe | ||
@echo Build $(@).cn.zip | ||
@echo zip $(EXECUTABLE)-$(VERSION)-$(OS)-$(ARCH) | ||
@zip -j $(EXECUTABLE)-$(VERSION)-$(OS)-$(ARCH).cn.zip "$<" script/win/start.bat |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should not work since the file does not exist.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
@rm -f *.zip | ||
@rm -f *.tar.gz | ||
@rm -f ./build/bin/qitmeer-waellt | ||
@rm -rf ./build/release | ||
|
||
webui: statik npm | ||
cd assets/src && npm run build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
webui
is required for a clean build. please re-arrange your makefile
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
No description provided.