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 osip and exosip repo #4654

Closed
wants to merge 21 commits into from
Closed

add osip and exosip repo #4654

wants to merge 21 commits into from

Conversation

hahahahbenny
Copy link
Contributor

添加了xtd的repo安装脚本

@hahahahbenny hahahahbenny changed the title add xtd repo add osip and exosip repo Jul 17, 2024
end)

on_test(function (package)
assert(package:has_cincludes("eXosip2/eXosip.h"))
Copy link
Member

Choose a reason for hiding this comment

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

用 has_cfuncs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的

@waruqi
Copy link
Member

waruqi commented Jul 17, 2024

还有commit太多了,重新理下,只用自己的账号 commit force push 下

@waruqi
Copy link
Member

waruqi commented Jul 17, 2024

另外, pr 拆分下,一个包一个 patch 单独记录和 ci test

@hahahahbenny
Copy link
Contributor Author

好的,那我开另一个pr来提交osip和exosip的包

@Issues-translate-bot
Copy link

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


There are too many commits. Let’s reorganize them and only use your own account to commit force push.

@Issues-translate-bot
Copy link

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


In addition, when pr is split, each package and patch are recorded separately and ci test

@Issues-translate-bot
Copy link

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


Okay, then I will open another PR to submit the osip and exosip packages.

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.

None yet

4 participants