-
Notifications
You must be signed in to change notification settings - Fork 1
Update Setup guide #6
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -5,29 +5,44 @@ Install newest version [here](https://www.python.org/downloads/). | |||||
|
|
||||||
| # Compilers | ||||||
|
|
||||||
| You will need GCC and Clang compilers. | ||||||
| You will need GCC and Clang compilers and some components of Visual Studio (Yes, patching game engine that was originally built with Visual Studio requires it's components). | ||||||
|
|
||||||
| ## Clang | ||||||
|
|
||||||
| Clang compiler installation: | ||||||
| * Goto [github releases of llvm](https://github.com/llvm/llvm-project/releases/tag/llvmorg-18.1.8) | ||||||
| * Download **clang+llvm-18.1.8-x86_64-pc-windows-msvc.tar.xz** | ||||||
| * Unpack into preferred location | ||||||
|
|
||||||
| * Goto [github releases of llvm](https://github.com/llvm/llvm-project/releases) | ||||||
| * Download latest one with **-x86_64-pc-windows-msvc** suffix (or installer **-win64.exe**) | ||||||
| * Install into preferred location | ||||||
| * You need path to `/bin/clang++.exe` | ||||||
|
|
||||||
| ## Visual Studio | ||||||
|
|
||||||
| * Goto [Visual Studio]( https://visualstudio.microsoft.com) official website | ||||||
| * Download latest version and install it | ||||||
| * Via *Visual Studio Installer* install 'Desktop development with C++' | ||||||
|
|
||||||
| ## GCC | ||||||
|
|
||||||
| GCC compiler installation: | ||||||
| * Install [MSYS2](https://www.msys2.org/) | ||||||
| * Install GCC x32/x64 via MSYS console: | ||||||
| * x32: pacman -S mingw-w64-i686-gcc | ||||||
| * x64: pacman -S mingw-w64-x86_64-gcc | ||||||
|
|
||||||
| * Install [chocolatey](https://chocolatey.org/) | ||||||
| * Run `choco install mingw --x86 -y --no-progress` which is gonna be installed at `C:\ProgramData\mingw64`. | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Use formal language in documentation. Replace the informal "gonna be" with "will be" for better documentation quality. ✏️ Proposed fix-* Run `choco install mingw --x86 -y --no-progress` which is gonna be installed at `C:\ProgramData\mingw64`.
+* Run `choco install mingw --x86 -y --no-progress` which will be installed at `C:\ProgramData\mingw64`.As per coding guidelines: The word 'gonna' is informal and should be avoided in documentation. 📝 Committable suggestion
Suggested change
🧰 Tools🪛 LanguageTool[style] ~30-~30: The word ‘gonna’ is informal. (GONNA) 🤖 Prompt for AI Agents |
||||||
| * You need paths to `/mingw32/bin/g++.exe` and `/mingw32/bin/ld.exe` | ||||||
|
Comment on lines
+29
to
31
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain🏁 Script executed: # First, locate and read the SETUP.md file to understand the full context
fd -t f "SETUP.md" | head -5Repository: FAForever/fa-python-binary-patcher Length of output: 86 🏁 Script executed: # Read the SETUP.md file to see lines 29-31 in context
cat -n SETUP.mdRepository: FAForever/fa-python-binary-patcher Length of output: 2331 🌐 Web query:
💡 Result: With Chocolatey installed in the default location, For the community
To confirm on your machine, check:
Citations:
🌐 Web query:
💡 Result: For Chocolatey packages, the extracted “real” files typically live under:
|
||||||
|
|
||||||
| After everything installed you need to clone patches repo (right now only 4z0t's fork is compatible). | ||||||
| # Patcher | ||||||
|
|
||||||
| After everything installed you need to clone patches repo. | ||||||
|
|
||||||
| Now you can setup build script: | ||||||
|
|
||||||
| ```bat | ||||||
| python main.py [Path to patches folder] [Path to clang++.exe] [Path to ld.exe] [Path to g++.exe] | ||||||
| ``` | ||||||
| After you successfully build, you have to test what you've got somehow. | ||||||
|
|
||||||
| After you successfully build, you have to test what you've got. | ||||||
| You have 2 options: | ||||||
|
|
||||||
| * Running patched game with files from FAF client gamedata | ||||||
| * Running patched game with files from FAF repo | ||||||
|
|
||||||
|
|
@@ -40,4 +55,4 @@ In preferences set default C++ formatter to `ms-vscode.cpptools`. | |||||
|
|
||||||
| # Decompiler | ||||||
|
|
||||||
| TODO | ||||||
| TODO | ||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix grammar: use possessive "its" instead of contraction "it's".
The phrase "requires it's components" should be "requires its components" (possessive form, not contraction).
✏️ Proposed fix
📝 Committable suggestion
🤖 Prompt for AI Agents