adapt new rust build standard #82
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mokeyish
reviewed
Mar 24, 2023
| $(eval $(call smartdns-rust/templates)) | ||
| define Package/smartdns-rust/install | ||
| $(INSTALL_DIR) $(1)/usr/sbin | ||
| $(INSTALL_BIN) $(PKG_BUILD_DIR)/target/$(RUSTC_TARGET_ARCH)/stripped/smartdns $(1)/usr/sbin/ |
There was a problem hiding this comment.
$(1) 是 prefix 么？是的话 /usr 可以不要的。
例如，
- 当 prefix 是
/时，这时二进制程序在/sbin - 当 prefix 是
/usr/local时，，这时二进制程序在/usr/local/sbin
There was a problem hiding this comment.
需/usr,否则编译后安装的目录是/sbin/而不是/usr/sbin/
|
这种文件最好能够自动更新。例如上面的 hash。 例如，挂载到 git-hook 上，提交代码自动更新。 |
可以考虑用github的dependabot |
|
你能想办法，写个脚本自动生成那些版号，hash 之类的么？类似 c 语言里的 configure 命令，它也不是写死的，而是编译的时候自动创建的。 可以看这个 它在编译前，有个 autogen 命令。 |
autogen干了啥？ 这个是做Makefile的，和rust兼容吗？感觉还是在github上用dependabot处理掉比较好 |
随意吧，反正是由脚本动态生成 makefile 文件。在构建前，先调脚本生成 makefile，不用每次都要跟随着版本号，改这个文件。倒也用不着 dependabot |
没操作过，如何做？ |
mokeyish
force-pushed
the
main
branch
7 times, most recently
from
cf07c81
to
9c7b741
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.