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.

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 macdylibbundler package #4596

Merged
merged 13 commits into from
Jul 8, 2024
Merged

add macdylibbundler package #4596

merged 13 commits into from
Jul 8, 2024

Conversation

hahahahbenny
Copy link
Contributor

添加了macdylibbundler的安装脚本

add_files("src/*.cpp")
add_includedirs("src")

add_headerfiles("src/*.h")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

就这么几行,就没必要单独一个 port 了。。直接 on_install 里,io.writefile 写下不就好了么。。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

因为参考了11zip的写法,所以就直接添加了一个port。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

之前不太熟悉这种写法,学习了,以后如果行数比较少就直接在on install里处理

@waruqi
Copy link
Member

waruqi commented Jul 8, 2024

还有为啥会有这么多不相关的 commit

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


And why are there so many irrelevant commits?

@hahahahbenny
Copy link
Contributor Author

不好意思,今天早上出去了,所以没看到你的建议

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


Sorry, I went out this morning, so I didn’t see your suggestion.

@waruqi waruqi merged commit b9a3128 into xmake-io:dev Jul 8, 2024
65 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants