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

Enable separate building and packaging of TFE and the “AdjustableHud” mod #393

Merged
merged 2 commits into from
Mar 24, 2024

Conversation

JakeSmarter
Copy link
Contributor

Improves the life of package maintainers and paves the way for integrated mod distribution.

Conceptually, building the “AdjustableHud” mod should have been its own CMake target. But, since CMake is conceptually flawed in general and thus custom build target support in CMake is either incomplete or just a disgrace to implement, we have to go the ugly route of adding build configuration options. Yes, it is ugly but it works and is obviously so intended by CMake devs.

@JakeSmarter JakeSmarter force-pushed the AdjustableHud_mod branch 2 times, most recently from 526c9a1 to b453ba5 Compare March 11, 2024 13:59
JakeSmarter added a commit to JakeSmarter/io.github.theforceengine.tfe that referenced this pull request Mar 11, 2024
Externalize launch script into a separate file
Enable Wayland support following luciusDXL/TheForceEngine#386
Move installing the launch script into a separate `module`
Use Flatpak Builder environment variables where applicable
Fix git tag pattern checker
Change `buildsystem` to `cmake`
Move editing the Desktop Entry file to `post-install` step
Add Flatpak extension support for mods following luciusDXL/TheForceEngine#393
Externalize the default “AdjustableHud” mod into a separate but bundled Flatpak `.Mod` extension
JakeSmarter added a commit to JakeSmarter/io.github.theforceengine.tfe that referenced this pull request Mar 14, 2024
Externalize launch script into a separate file
Enable Wayland support following luciusDXL/TheForceEngine#386
Move installing the launch script into a separate `module`
Use Flatpak Builder environment variables where applicable
Fix git tag pattern checker
Change `buildsystem` to `cmake`
Move editing the Desktop Entry file to `post-install` step
Add Flatpak extension support for mods following luciusDXL/TheForceEngine#393
Externalize the default “AdjustableHud” mod into a separate but bundled Flatpak `.Mod` extension
Add “Dark Forces Community Patch” mod
JakeSmarter added a commit to JakeSmarter/io.github.theforceengine.tfe that referenced this pull request Mar 14, 2024
Externalize launch script into a separate file
Enable Wayland support following luciusDXL/TheForceEngine#386
Move installing the launch script into a separate `module`
Use Flatpak Builder environment variables where applicable
Fix git tag pattern checker
Change `buildsystem` to `cmake`
Move editing the Desktop Entry file to `post-install` step
Add Flatpak extension support for mods following luciusDXL/TheForceEngine#393
Externalize the default “AdjustableHud” mod into a separate but bundled Flatpak `.Mod` extension
JakeSmarter added a commit to JakeSmarter/io.github.theforceengine.tfe that referenced this pull request Mar 14, 2024
Externalize launch script into a separate file
Enable Wayland support following luciusDXL/TheForceEngine#386
Move installing the launch script into a separate `module`
Use Flatpak Builder environment variables where applicable
Fix git tag pattern checker
Change `buildsystem` to `cmake`
Move editing the Desktop Entry file to `post-install` step
Add Flatpak extension support for mods following luciusDXL/TheForceEngine#393
Externalize the default “AdjustableHud” mod into a separate but bundled Flatpak `.Mod` extension
* Drop deprecated `type` attribute from `id` element
* Add *globally* accessible `icon`
* Fix typo in de `li`
* Add `keywords` synced with Desktop Entry’s `Keywords` key
* `CMakeLists.txt`: Add `ENABLE_TFE` and `ENABLE_ADJUSTABLEHOD_MOD`
   configuration options for package maintainers to be able to build separate
   packages
* Add AppStream component meta data for the built‑in “AdjustableHud” mod
* `README.md`: Add info on packaging TFE and the “AdjustableHud” mod
* `ProjectStructure.md`: Add `Mods` directory and description
@luciusDXL luciusDXL merged commit ec48222 into luciusDXL:master Mar 24, 2024
1 check passed
@JakeSmarter JakeSmarter deleted the AdjustableHud_mod branch April 8, 2024 13:19
JakeSmarter added a commit to JakeSmarter/io.github.theforceengine.tfe that referenced this pull request Apr 12, 2024
Externalize launch script into a separate file
Enable Wayland support following luciusDXL/TheForceEngine#386
Move installing the launch script into a separate `module`
Use Flatpak Builder environment variables where applicable
Fix git tag pattern checker
Change `buildsystem` to `cmake`
Move editing the Desktop Entry file to `post-install` step
Add Flatpak extension support for mods following luciusDXL/TheForceEngine#393
Externalize the default “AdjustableHud” mod into a separate but bundled Flatpak `.Mod` extension
JakeSmarter added a commit to JakeSmarter/io.github.theforceengine.tfe that referenced this pull request Apr 12, 2024
Externalize launch script into a separate file
Enable Wayland support following luciusDXL/TheForceEngine#386
Move installing the launch script into a separate `module`
Use Flatpak Builder environment variables where applicable
Fix git tag pattern checker
Change `buildsystem` to `cmake`
Move editing the Desktop Entry file to `post-install` step
Add Flatpak extension support for mods following luciusDXL/TheForceEngine#393
Externalize the default “AdjustableHud” mod into a separate but bundled Flatpak `.Mod` extension
JakeSmarter added a commit to JakeSmarter/io.github.theforceengine.tfe that referenced this pull request Jan 9, 2025
Externalize launch script into a separate file
Enable Wayland support following luciusDXL/TheForceEngine#386
Move installing the launch script into a separate `module`
Use Flatpak Builder environment variables where applicable
Fix git tag pattern checker
Change `buildsystem` to `cmake`
Add Flatpak extension support for mods following luciusDXL/TheForceEngine#393
Externalize the default “AdjustableHud” mod into a separate but bundled Flatpak `.Mod` extension
JakeSmarter added a commit to JakeSmarter/io.github.theforceengine.tfe that referenced this pull request Jan 9, 2025
Externalize launch script into a separate file
Enable Wayland support following luciusDXL/TheForceEngine#386
Move installing the launch script into a separate `module`
Use Flatpak Builder environment variables where applicable
Fix git tag pattern checker
Change `buildsystem` to `cmake`
Add Flatpak extension support for mods following luciusDXL/TheForceEngine#393
Externalize the default “AdjustableHud” mod into a separate but bundled Flatpak `.Mod` extension
JakeSmarter added a commit to JakeSmarter/io.github.theforceengine.tfe that referenced this pull request Jan 18, 2025
Externalize launch script into a separate file
Enable Wayland support following luciusDXL/TheForceEngine#386
Move installing the launch script into a separate `module`
Use Flatpak Builder environment variables where applicable
Fix git tag pattern checker
Change `buildsystem` to `cmake`
Add Flatpak extension support for mods following luciusDXL/TheForceEngine#393
Externalize the default “AdjustableHud” mod into a separate but bundled Flatpak `.Mod` extension
JakeSmarter added a commit to JakeSmarter/io.github.theforceengine.tfe that referenced this pull request Jan 19, 2025
Externalize launch script into a separate file
Enable Wayland support following luciusDXL/TheForceEngine#386
Move installing the launch script into a separate `module`
Use Flatpak Builder environment variables where applicable
Fix git tag pattern checker
Change `buildsystem` to `cmake`
Add Flatpak extension support for mods following luciusDXL/TheForceEngine#393
Externalize the default “AdjustableHud” mod into a separate but bundled Flatpak `.Mod` extension
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.

2 participants