Public testing of using yaml config with kde-builder
This mr is intended for testers of the new feature - the yaml format for kde-builder config. See #109 (closed)
Test plan
- Ensure you have updated
repo-metadata
to at least after this commit. This commit adds build configs (formerly known as "module definitions") in yaml format.
For this, either run kde-builder with enabled metadata updates (i.e. without--no-metadata/-M
), or update it explicitly:
$ kde-builder --metadata-only
Alternatively, you can git pull
in the metadata direcrory (~/.local/state/sysadmin-repo-metadata/
) manually.
- Use the kde-builder as you would normally do, f.e. open a Konsole, and try to build some project, like ark.
Expected behavior
The tool will instruct you what command to run to convert the config:
❯ kde-builder ark
* The /home/andrew/.config/kde-builder.yaml config was not found, but the legacy ksb config /home/andrew/.config/kdesrc-buildrc was found.
Please, convert your legacy config to the new format with the following command:
/home/andrew/Development/kde-builder/scripts/convert_ksb_config.py /home/andrew/.config/kdesrc-buildrc /home/andrew/.config/kde-builder.yaml
Runtime Error: Missing config.
After running this command, rerun the kde-builder:
❯ /home/andrew/Development/kde-builder/scripts/convert_ksb_config.py /home/andrew/.config/kdesrc-buildrc /home/andrew/.config/kde-builder.yaml
Converting /home/andrew/.config/kdesrc-buildrc to /home/andrew/.config/kde-builder.yaml
❯ kde-builder ark
Fetching remote changes to sysadmin-repo-metadata
Merging sysadmin-repo-metadata changes from branch master
Holding performance profile
Building extra-cmake-modules from frameworks (1/40)
Fetching remote changes to extra-cmake-modules
Merging extra-cmake-modules changes from branch master
No changes to extra-cmake-modules source code, but proceeding to build anyway.
0.0% Compiling...
...
ToDo
- Update documentation, add note to changelog
- Update code, that references the "old" names.
Feedback
Please report in case you got any issue.
Tagging people who might want to test this mr: @duha @merritt @bcooksley @tcanabrava @thiagosueto @nclarius
Edited by Andrew Shark