Merge ~jocave/checkbox/+git/checkbox-configure:wrapper-naming-nonclassic into ~checkbox-dev/checkbox/+git/support:master
Proposed by
Jonathan Cave
| Status: | Superseded |
|---|---|
| Proposed branch: | ~jocave/checkbox/+git/checkbox-configure:wrapper-naming-nonclassic |
| Merge into: | ~checkbox-dev/checkbox/+git/support:master |
| Diff against target: |
1701 lines (+1479/-0) 37 files modified
.bumpversion.cfg (+17/-0) .gitignore (+23/-0) checkbox-provider-template/manage.py.in (+16/-0) checkbox-provider-template/units/device/jobs.pxu (+30/-0) checkbox-provider-template/units/test-plan.pxu.in (+119/-0) config/config_vars (+32/-0) configure.ac (+111/-0) launchers/checkbox-cli-wrapper (+12/-0) launchers/checkbox-cli-wrapper_classic (+6/-0) launchers/checkbox-slave (+5/-0) launchers/configure.in (+69/-0) launchers/configure.in_classic (+58/-0) launchers/hook-wrapper (+5/-0) launchers/shell-wrapper (+7/-0) launchers/test-runner-automated.in (+19/-0) launchers/test-runner.in (+12/-0) launchers/wrapper_local.in (+43/-0) launchers/wrapper_local.in_classic (+41/-0) launchers_v2/checkbox-cli-wrapper (+3/-0) launchers_v2/configure.in (+69/-0) launchers_v2/configure.in_classic (+58/-0) launchers_v2/shell-wrapper (+4/-0) launchers_v2/test-runner-automated.in (+19/-0) launchers_v2/test-runner.in (+12/-0) launchers_v2/wrapper_local.in (+47/-0) launchers_v2/wrapper_local.in_classic (+45/-0) snap/hooks/configure.in (+64/-0) snap/hooks/install.in (+4/-0) snap/hooks_classic/configure.in (+53/-0) snap/hooks_classic/install.in (+16/-0) snap/hooks_classic/remove (+5/-0) snap/plugins/checkbox_provider.py.in (+69/-0) snap/plugins/checkbox_provider.py.in_v2 (+92/-0) snap/snapcraft.yaml.in (+83/-0) snap/snapcraft.yaml.in_v2 (+93/-0) snap/snapcraft_classic.yaml.in (+57/-0) snap/snapcraft_classic.yaml.in_v2 (+61/-0) |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Checkbox Developers | Pending | ||
|
Review via email:
|
|||
Description of the change
Although the use of "checkbox-cli" as the name of the wrapper script worked on classic, on non-classic it ended up stuck in an exec loop. Could possibly have been solved in other ways, but it makes more sense to me to just keep the name of the wrapper script as something different to the underlying command.
To post a comment you must log in.
There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
