[docs] Bring back the Options page, and clean up LLM policy (#3649)

Signed-off-by: crueter <crueter@eden-emu.dev>
Reviewed-on: https://git.eden-emu.dev/eden-emu/eden/pulls/3649

docs/Options.md

@@ -0,0 +1,100 @@
+# CMake Options
+
+To change these options, add `-DOPTION_NAME=NEWVALUE` to the command line.
+
+- On Qt Creator, go to Project -> Current Configuration
+
+Notes:
+
+- Defaults are marked per-platform.
+- "Non-UNIX" just means Windows/MSVC and Android (yes, macOS is UNIX
+- Android generally doesn't need to change anything; if you do, go to `src/android/app/build.gradle.kts`
+- To set a boolean variable to on, use `ON` for the value; to turn it off, use `OFF`
+- If a variable is mentioned as being e.g. "ON" for a specific platform(s), that means it is defaulted to OFF on others
+- TYPE is always boolean unless otherwise specified
+- Format:
+  - `OPTION_NAME` (TYPE DEFAULT) DESCRIPTION
+
+## Options
+
+### Dependencies
+
+These options control dependencies.
+
+- `YUZU_USE_BUNDLED_FFMPEG` (ON for non-UNIX) Download a pre-built and configured FFmpeg
+- `YUZU_USE_EXTERNAL_FFMPEG` (ON for Solaris) Build FFmpeg from source
+- `YUZU_DOWNLOAD_ANDROID_VVL` (ON) Download validation layer binary for Android
+- `YUZU_DOWNLOAD_TIME_ZONE_DATA` (ON) Always download time zone binaries
+  - Currently, build fails without this
+- `YUZU_TZDB_PATH` (string) Path to a pre-downloaded timezone database (useful for nixOS and Gentoo)
+- `YUZU_USE_BUNDLED_MOLTENVK` (ON, macOS only) Download bundled MoltenVK lib
+- `YUZU_USE_BUNDLED_OPENSSL` (ON for MSVC, Android, Solaris, and OpenBSD) Download bundled OpenSSL build
+- `YUZU_USE_EXTERNAL_SDL2` (OFF) Compiles SDL2 from source
+- `YUZU_USE_BUNDLED_SDL2` (ON for MSVC) Download a prebuilt SDL2
+
+### Miscellaneous
+
+- `ENABLE_WEB_SERVICE` (ON) Enable multiplayer service
+- `ENABLE_WIFI_SCAN` (OFF) Enable WiFi scanning (requires iw on Linux) - experimental
+- `ENABLE_CUBEB` (ON) Enables the cubeb audio backend
+  - This option is subject for removal.
+- `YUZU_TESTS` (ON) Compile tests - requires Catch2
+- `ENABLE_LTO` (OFF) Enable link-time optimization
+  - Not recommended on Windows
+  - UNIX may be better off appending `-flto=thin` to compiler args
+- `USE_FASTER_LINKER` (OFF) Check if a faster linker is available
+  - Not recommended outside of Linux
+
+### Flavors
+
+These options control executables and build flavors.
+
+- `YUZU_LEGACY` (OFF): Apply patches to improve compatibility on some older GPUs at the cost of performance
+- `NIGHTLY_BUILD` (OFF): This is only used by CI. Do not use this unless you're making your own distribution and know what you're doing.
+- `YUZU_STATIC_BUILD` (OFF) Attempt to build using static libraries if possible
+  - Not supported on Linux
+  - Automatically set if `YUZU_USE_BUNDLED_QT` is on for non-Linux
+- `ENABLE_UPDATE_CHECKER` (OFF) Enable update checking functionality
+- `YUZU_DISABLE_LLVM` (OFF) Do not attempt to link to the LLVM demangler
+  - Really only useful for CI or distribution builds
+
+**Desktop only**:
+
+- `YUZU_CMD` (ON) Compile the SDL2 frontend (eden-cli)
+- `YUZU_ROOM` (OFF) Compile dedicated room functionality into the main executable
+- `YUZU_ROOM_STANDALONE` (OFF) Compile a separate executable for room functionality
+- `YUZU_STATIC_ROOM` (OFF) Compile the room executable *only* as a static, portable executable
+  - This is only usable on Alpine Linux.
+
+### Desktop
+
+The following options are desktop only.
+
+- `ENABLE_LIBUSB` (ON) Enable the use of the libusb input frontend (HIGHLY RECOMMENDED)
+- `ENABLE_OPENGL` (ON) Enable the OpenGL graphics frontend
+  - Unavailable on Windows/ARM64
+  - You probably shouldn't turn this off.
+
+### Qt
+
+Also desktop-only, but apply strictly to Qt
+
+- `ENABLE_QT` (ON) Enable the Qt frontend (recommended)
+- `ENABLE_QT_TRANSLATION` (OFF) Enable translations for the Qt frontend
+- `YUZU_USE_BUNDLED_QT` (ON for MSVC) Download bundled Qt binaries
+  - Not recommended on Linux. For Windows and macOS, the provided build is statically linked.
+- `YUZU_QT_MIRROR` (string) What mirror to use for downloading the bundled Qt libraries
+- `YUZU_USE_QT_MULTIMEDIA` (OFF) Use QtMultimedia for camera support
+- `YUZU_USE_QT_WEB_ENGINE` (OFF) Use QtWebEngine for web applet implementation (requires the huge QtWebEngine dependency; not recommended)
+- `USE_DISCORD_PRESENCE` (OFF) Enables Discord Rich Presence (Qt frontend only)
+
+### Retired Options
+
+The following options were a part of Eden at one point, but have since been retired.
+
+- `ENABLE_OPENSSL` - MbedTLS was fully replaced with OpenSSL in [#3606](https://git.eden-emu.dev/eden-emu/eden/pulls/3606), because OpenSSL straight-up performs better.
+- `ENABLE_SDL2` - While technically possible to *not* use SDL2 on desktop, this is **NOT** a supported configuration under any means, and adding this matrix to our build system was not worth the effort.
+- `YUZU_USE_CPM` - This option once had a purpose, but that purpose has long since passed us by. *All* builds use CPMUtil to manage dependencies now.
+  - If you want to *force* the usage of system dependencies, use `-DCPMUTIL_FORCE_SYSTEM=ON`.
+
+See `src/dynarmic/CMakeLists.txt` for additional options--usually, these don't need changed

docs/README.md

@@ -3,6 +3,7 @@
 This contains documentation created by developers. This contains build instructions, guidelines, instructions/layouts for [cool stuff we made](./CPMUtil), and more.
 
 - **[General Build Instructions](Build.md)**
+- **[CMake Options](Options.md)**
 - **[Cross Compiling](CrossCompile.md)**
 - **[Development Guidelines](Development.md)**
 - **[Dependencies](Deps.md)**

docs/policies/AI.md

@@ -17,7 +17,9 @@ AI is notorious for hallucinating facts out of thin air and sometimes outright l
 - Code that works, but is extraordinarily verbose or not nearly as efficient as it can be
 - Code that works well and is written well, but solves a different problem than was intended, or solves the same problem but in a completely incorrect way that will break other things horribly.
 
-Human-written code will, without exception, always be of infinitely higher quality when properly researched and implemented by someone both familiar with the surrounding code and the programming language in use. LLMs may produce a "good enough" result, but this result is often subpar. Keep in mind: all code is held under a standard of excellence. If your code sucks, it will be rejected. AI-generated code just so happens to be a particularly sucky genre of code, and will thus be held to this same standard.
+Human-written code will, without exception, always be of infinitely higher quality when properly researched and implemented by someone familiar with *both* the surrounding code and the programming language in use. LLMs may produce a "good enough" result, but this result is often subpar.
+
+**All code is held under a STRICT STANDARD OF EXCELLENCE**. AI code is no different, and since it often produces subpar or outright terrible code, it will often fail to meet this excellence standard.
 
 On a lesser-known note, LLM outputs often contain unicode symbols such as emojis or the arrow symbol. Please don't put Unicode symbols in your code. It messes with many an IDE, and the three people viewing your code on Lynx will be very unhappy.
 
@@ -25,7 +27,8 @@ On a lesser-known note, LLM outputs often contain unicode symbols such as emojis
 
 ## Acceptable Use
 
-- As stated previously, AI is good in a few *very specific* cases. In these cases, it's usually fine to use AI, as long as you **explicitly provide notice that it was used**.
+As stated previously, AI is good in a few *very specific* cases. In these cases, it's usually fine to use AI, as long as you **explicitly provide notice that it was used**.
+
 - Anything directly outside of the realm of the code written in your PR or patch is none of our business.
   - This primarily covers research.
   - However, we *still* strongly discourage this for the reasons mentioned above.
@@ -104,4 +107,6 @@ This consolidates profile removal behavior, fixes potential race conditions in t
 
 This has all of the same problems as the other one. Needlessly verbose, doesn't address *what* it actually fixes ("consolidates profile removal behavior"... okay, why? What does it fix?), etc. It even has the bonus of totally hallucinating the addition of a method!
 
+On a more "philosophical" note, LLMs tend to be geared towards *corporate language*, as that's what they're trained on. This is why AI-generated commit messages feel like "word salad", and typically pad out the commit message to make it *look* like a lot of things were changed (trust me, it's like that in the corporate world). They typically also drift towards unneeded buzzwords and useless implementation details.
+
 **Don't use AI for commit messages**.