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 launcher config #234

Open
wants to merge 67 commits into
base: master
Choose a base branch
from
Open

add launcher config #234

wants to merge 67 commits into from

Conversation

fesily
Copy link
Contributor

@fesily fesily commented Mar 17, 2023

signature两个配置module和version,module需要用户填写,version自动从luaVersion字段读取

将attach的luaversion默认设置为unknown,允许launcher自动判断版本

将luadbug和launcher添加为module黑名单
将调试器目录下的所有module为黑名单

@actboy168
Copy link
Owner

package_json.lua的格式全乱了

extension/js/configurationProvider.js Outdated Show resolved Hide resolved
extension/script/frontend/proxy.lua Outdated Show resolved Hide resolved
@actboy168
Copy link
Owner

由launcher加载调试器dll的代码也可以包含进来,这很有用。

@fesily
Copy link
Contributor Author

fesily commented Mar 18, 2023

package_json.lua的格式全乱了

设置中保存自动format,我先添加一个pull来完成code-format的功能把现有的lua文件都format一遍

@actboy168
Copy link
Owner

format是个大工程,尤其是在lua 的format还不是很成熟的情况下。所以你还是先别改吧。

@fesily
Copy link
Contributor Author

fesily commented Mar 18, 2023

format是个大工程,尤其是在lua 的format还不是很成熟的情况下。所以你还是先别改吧。

嗯,就先启用设置,保证以后format的结果

@fesily
Copy link
Contributor Author

fesily commented Mar 18, 2023

把我搞懵了,package_json.lua文件用的是windows的换行符号,怪不得一format整个文件变了,你是如何打算的,统一使用哪种?

@fesily
Copy link
Contributor Author

fesily commented Mar 18, 2023

由launcher加载调试器dll的代码也可以包含进来,这很有用。

单纯加载调试器?还是要把导入表修复的部分也加入?

@actboy168
Copy link
Owner

只加载就好

@actboy168
Copy link
Owner

actboy168 commented Mar 18, 2023

把我搞懵了,package_json.lua文件用的是windows的换行符号,怪不得一format整个文件变了,你是如何打算的,统一使用哪种?

用lf吧,我记得我的git配置是会强制转为lf的

@fesily
Copy link
Contributor Author

fesily commented Mar 18, 2023

把我搞懵了,package_json.lua文件用的是windows的换行符号,怪不得一format整个文件变了,你是如何打算的,统一使用哪种?

用lf吧,我记得我的git配置是会强制转为lf的

主干上直接format成lf吧,pull里就不做这个操作了

@fesily fesily marked this pull request as ready for review April 6, 2023 07:05
@fesily fesily requested a review from actboy168 April 6, 2023 07:17
src/launcher/autoattach/lua_version.cpp Outdated Show resolved Hide resolved
extension/script/debugger.lua Outdated Show resolved Hide resolved
src/launcher/config/config.cpp Outdated Show resolved Hide resolved
src/launcher/main.cpp Outdated Show resolved Hide resolved
src/launcher/autoattach/lua_module.cpp Outdated Show resolved Hide resolved
src/launcher/main.cpp Outdated Show resolved Hide resolved
@fesily fesily requested a review from actboy168 April 7, 2023 05:11
extension/script/debugger.lua Outdated Show resolved Hide resolved
@fesily fesily requested a review from actboy168 April 7, 2023 07:07
@fesily
Copy link
Contributor Author

fesily commented Apr 21, 2023

最近在忙其他的,review咋样,还有需要哪里改进的吗 @actboy168

@actboy168
Copy link
Owner

我最近也挺忙的,所以还没空review

@fesily
Copy link
Contributor Author

fesily commented Apr 22, 2023

我最近也挺忙的,所以还没空review

好的,等你忙完

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.

2 participants