Closed zeromake closed 1 year ago
@zeromake 哇,uwp的支持终于来了! 如果没有其他的补充,我现在可以合并了吗?
@xfangfang 可以了,你看看 ci 触发的分支是否需要改。
@zeromake 感觉可以把dev也加进来
@zeromake 打扰一下,我刚刚尝试了ci的结果,发现了下面几个问题:
execute_process(COMMAND git describe --tags
TIMEOUT 5
OUTPUT_VARIABLE GIT_TAG_VERSION
OUTPUT_STRIP_TRAILING_WHITESPACE
)
execute_process(COMMAND git rev-parse --short HEAD
TIMEOUT 5
OUTPUT_VARIABLE GIT_TAG_SHORT
OUTPUT_STRIP_TRAILING_WHITESPACE
)
add_definitions(-DBUILD_TAG_VERSION=${GIT_TAG_VERSION} -DBUILD_TAG_SHORT=${GIT_TAG_SHORT})
前四条是我可以做的,当然也欢迎提交pr❤️
第五条我不太清楚具体的图标规格,这里有两张svg图片可以使用:https://github.com/xfangfang/wiliwili/blob/yoga/resources/svg/cn.xfangfang.wiliwili.svg https://github.com/xfangfang/wiliwili/blob/yoga/resources/svg/cn.xfangfang.wiliwili.s.svg
第六条估计需要修改一下xmake的构建脚本。
@xfangfang 1-4是已经做好了吗? 图标好整,两个 define 也好整的(可以在xmake里调用 git 来添加)
key.pfx这个本来应该是找微软那边上传以后就不用自行导入了,全屏的问题应该是 sdl 之前没做全屏,初始化没有做全屏检查。
1-4 都还没做
key.pfx 的问题我是想着上传应用还不知道是什么时候的事,在这之前如果有其他用户想要尝试的话可以更方便的下载测试。
on_config(function (target)
target:add("defines", "BUILD_TAG_VERSION=$(shell git describe --tags)")
target:add("defines", "BUILD_TAG_SHORT=$(shell git rev-parse --short HEAD)")
end)
xmake 可以通过如上方式添加编译宏
@dragonflylee 感谢提醒,zeromake 在另外一个pr里已经做好了这部分的内容。
如果对 wiliwili 感兴趣的话欢迎参与到开发中来,很遗憾代码写得可能不是特别好,如果关于要做的哪个功能有什么疑问欢迎随时和我联系。
@dragonflylee
on_config
会多次执行吗,我现在是放 before_build
里做的,$(shell)
这个语法我倒是没有注意到我明天看看在 windows 下的效果,os.iorunv
需要自己去换行。
@dragonflylee
on_config
会多次执行吗,我现在是放before_build
里做的,$(shell)
这个语法我倒是没有注意到我明天看看在 windows 下的效果,os.iorunv
需要自己去换行。
https://xmake.io/#/zh-cn/manual/project_target?id=targeton_config
测了一下确实只执行一次