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

fix(cmd/gf): remove dm driver for command gen dao as it causes building failed for some platforms #3667

Merged
merged 1 commit into from
Jun 27, 2024

Conversation

hailaz
Copy link
Member

@hailaz hailaz commented Jun 27, 2024

No description provided.

@houseme houseme requested a review from gqcn June 27, 2024 10:04
@gqcn gqcn changed the title fix(cmd/gf): 修复因为引入dm驱动导致cli预编译失败的问题 fix(cmd/gf): remove dm driver as it caused building failed for some platforms Jun 27, 2024
@gqcn gqcn changed the title fix(cmd/gf): remove dm driver as it caused building failed for some platforms fix(cmd/gf): remove dm driver for command gen dao as it causes building failed for some platforms Jun 27, 2024
@gqcn gqcn merged commit 31a6ae2 into master Jun 27, 2024
46 checks passed
@gqcn gqcn deleted the fix/releases branch June 27, 2024 13:12
@xcore-fun
Copy link

意思是达梦不能通过命令生成代码吗

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


Does it mean Dameng cannot generate code through commands?

@hailaz
Copy link
Member Author

hailaz commented Jan 8, 2025

意思是达梦不能通过命令生成代码吗

不是,可以自行构建,解开注释然后自己build一下就好了。这里移除的原因是添加后导致部分平台预编译失败。

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


Does it mean Dameng cannot generate code through commands?

No, you can build it yourself, uncomment it and then build it yourself. The reason for removal here is that after adding it, precompilation failed for some platforms.

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.

5 participants