-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Let the documentation reflect that there is no vs/master anymore #3220
Let the documentation reflect that there is no vs/master anymore #3220
Conversation
We used to have that `make vcxproj` hack, but a hack it is. In the meantime, we have a much cleaner solution: using CMake, either explicitly, or even more conveniently via Visual Studio's built-in CMake support (simply open Git's top-level directory via File>Open>Folder...). Let's let the `README` reflect this. Signed-off-by: Johannes Schindelin <[email protected]>
Let's not recommend the `vs/master` branch anymore (which we stopped updating), but instead recommend the much better CMake-based way to build Git in Visual Studio. Signed-off-by: Johannes Schindelin <[email protected]>
We no longer update `vs/master` and will probably drop it soon altogether (besides, it has been the `vs/main` branch for the last part of its active life). So let's not describe it in this document any longer. Signed-off-by: Johannes Schindelin <[email protected]>
0d7a7f7
to
1aaa8f9
Compare
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
While I fully support the changes, my recent experience is that the VS2019 integration isn't working properly when started from a clean slate. If I understand correctly, the cause is that VS2019 has changed it's default generator to 'Ninja' and doesn't therefore set the MSVC property needed for the CMake initialisation. Also the default ARCH setting isn't being picked up in the cmake code when it's set in the install_vcpkg.bat. It can be tricky to test as it's awkward to ensure a truly clean setup of a fresh VS2019 etc. |
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
That may be so, but we should fix the experience, then. I could imagine that the recent ARM64 work might have broken the Ninja-based build process, but the good news is that we're unlikely to change this part of the code base anytime soon (except to fix the bug you reported). Would you mind walking through the process from scratch and opening a new ticket with the findings? |
I hope to do that. Time has been a bit limited of late. I had been making some notes in #3176 and have some local notes trying to get to the bottom of the issue (lots of StackOverflow links ;-) My aim is that the code should work straight out of the tin to encourage those who want to investigate the code who don't yet feel able to make the leap to being 'contributors' (which links to also wanting to get Sourcetrail to integrate a little more easily.) |
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
…ter-anymore Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
…ter-anymore Let the documentation reflect that there is no vs/master anymore
…ter-anymore Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
…ter-anymore Let the documentation reflect that there is no vs/master anymore
…ter-anymore Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
…ter-anymore Let the documentation reflect that there is no vs/master anymore
…ter-anymore Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
Let the documentation reflect that there is no vs/master anymore
While rebasing Git for Windows onto v2.32.0-rc0, I noticed that the documentation was stale:
vs/master
(or more appropriately,vs/main
) is no longer updated, and was deprecated in favor of using CMake already a long time ago.