From 9926ce4ad10893d8d0758f7df8162f75ee9b080d Mon Sep 17 00:00:00 2001 From: heinezen Date: Tue, 10 Oct 2023 16:42:50 +0200 Subject: [PATCH 1/3] doc: Clarify version field usage in modpack info. --- doc/media/openage/modpack_definition_file.md | 29 +++++++++++--------- 1 file changed, 16 insertions(+), 13 deletions(-) diff --git a/doc/media/openage/modpack_definition_file.md b/doc/media/openage/modpack_definition_file.md index a9ff6d6a41..3403949dd6 100644 --- a/doc/media/openage/modpack_definition_file.md +++ b/doc/media/openage/modpack_definition_file.md @@ -31,19 +31,22 @@ The following parameters have to be specified. `[info]` contains general information about the modpack. -| Parameter | Data Type | Optional | Description | -| ------------------ | ------------- | -------- | ------------------------------------------------------------------------ | -| `packagename` | String | No | Name of the modpack. | -| `version` | String | No | Internal version number. Must have [semver](https://semver.org/) format. | -| `versionstr` | String | Yes | Human-readable version string. | -| `repo` | String | Yes | Name of the repo where the package is hosted. | -| `alias` | String | Yes | Alias of the modpack. Aliases can be used for replacing other modpacks. | -| `title` | String | Yes | Title used in UI. | -| `description` | String | Yes | Path to a file with a short description (max 500 chars). | -| `long_description` | String | Yes | Path to a file with a detailed description. | -| `url` | String | Yes | Link to the modpack's website. | -| `license` | Array[String] | Yes | License(s) of the modpack. | - +| Parameter | Data Type | Optional | Description | +| ------------------ | ------------- | -------- | ----------------------------------------------------------------------- | +| `packagename` | String | No | Name of the modpack. | +| `version`\* | String | No | The modpack's internal version number. Must use [semver] format. | +| `versionstr`\* | String | Yes | Human-readable version string. | +| `repo` | String | Yes | Name of the repo where the package is hosted. | +| `alias` | String | Yes | Alias of the modpack. Aliases can be used for replacing other modpacks. | +| `title` | String | Yes | Title used in UI. | +| `description` | String | Yes | Path to a file with a short description (max 500 chars). | +| `long_description` | String | Yes | Path to a file with a detailed description. | +| `url` | String | Yes | Link to the modpack's website. | +| `license` | Array[String] | Yes | License(s) of the modpack. | + +[semver]: https://semver.org/ + +\* `version` is used by the engine to determine the most recent version of a modpack. Therefore, it should be bumped when something in the modpack changes (e.g. whenever a new version gets published). `versionstr` is what is displayed to the user and can contain any string, so it can be used to represent any sensible version format. ## [assets] Section From bd17e3a431b898c28525933e640af6f968066b3f Mon Sep 17 00:00:00 2001 From: heinezen Date: Tue, 10 Oct 2023 16:49:00 +0200 Subject: [PATCH 2/3] convert: Fix directory access in API export CLI. --- openage/convert/tool/api_export.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openage/convert/tool/api_export.py b/openage/convert/tool/api_export.py index 36f85abafd..e0d14e703e 100644 --- a/openage/convert/tool/api_export.py +++ b/openage/convert/tool/api_export.py @@ -33,7 +33,7 @@ def main(args, error): del error # unused path = Union().root - path.mount(Directory(args.dir)) + path.mount(Directory(args.dir).root) export_api(path) From 88180ad0c5cde484e337d9e9717baf74df5aaafe Mon Sep 17 00:00:00 2001 From: heinezen Date: Tue, 10 Oct 2023 17:06:06 +0200 Subject: [PATCH 3/3] gamestate: Do not substr on double wildcard. --- libopenage/gamestate/game.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libopenage/gamestate/game.cpp b/libopenage/gamestate/game.cpp index 90dfe5ede3..d31310945d 100644 --- a/libopenage/gamestate/game.cpp +++ b/libopenage/gamestate/game.cpp @@ -67,11 +67,12 @@ void Game::load_data(const std::shared_ptr &mod_manager) { recursive = true; if (parts.size() == 1) { // include = "**" - search = include.substr(0, include.size() - 2); + // start in root directory + search = ""; } else { // include = "path/to/somewhere/**" - // remove the slash '/' too + // remove the wildcard '**' and the slash '/' search = include.substr(0, include.size() - 3); } }