-
Notifications
You must be signed in to change notification settings - Fork 29
Windows 2022 Dockerfile #197
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
base: main
Are you sure you want to change the base?
Conversation
Thanks, maybe we can mention this somewhere? And I think @6543, you were the one mostly pushing windows support in the past? Maybe you can check this out? |
RUN commands should be merged together to minimize layer count |
Hi, Before making any optimizations, I would like to know if there is a desire to integrate Windows support officially into Woodpecker. I think we could make a We can build a Windows image from a Linux OS if we only use
|
Sure, just added a comment to not forget that |
FYI, we use this image successfully for a number of projects. It is worth switching to the latest version of git for windows: this contains git lfs v3.7.0, which contains a patch for using netrc on windows (which I actually made for using this plugin on windows). https://github.com/git-for-windows/git/releases/tag/v2.50.0.windows.2 |
It seems this is working for multiple people, so I guess we can try to officially support it. However, we need to be able to make sure that this is maintained. As none of us woodpecker maintainers will use this afaik, you would have to take care of this. We do this similar for the bitbucket datacenter support in the core project. If anybody complains about something with this dockerfile, we would ping and mention you and expect that you fix the issues. I can try to give this a review but if there's somebody with more experience and knowledge of windows docker that would be very good as well. |
me: windows + local exec windows containers are hard to maintain as you have to target each windows kernel version as different platform ... :/ |
not a bad idea ... we just need good source of the base image for the most common windows kernels currently out there |
Fix issue: #187