Skip to content

Public testing of using yaml config with kde-builder

Andrew Shark requested to merge work/ashark/yaml_config_mr into master

This mr is intended for testers of the new feature - the yaml format for kde-builder config. See #109 (closed)

Test plan

  1. 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.

  1. 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

Merge request reports

Loading