launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #32423
[Merge] ~alvarocs/launchpad:use-craft-platforms-for-snap into launchpad:master
The proposal to merge ~alvarocs/launchpad:use-craft-platforms-for-snap into launchpad:master has been updated.
Description changed to:
Fixes short notation format bugs: https://bugs.launchpad.net/launchpad/+bug/2077005 & https://forum.snapcraft.io/t/call-for-testing-snapcraft-8-7-0/45762
Changes include:
Build planning
- lp/snappy/adapters/buildarch.py: reformat code to use craft-platforms API for 'platforms' recipe format.
- lp/snappy/adapters/tests/test_buildarch.py: update existing tests with platform name assertion, extend test suite.
Snap model
- lp/snappy/interfaces/snapbuild.py, model/snapbuild.py - store craft_platform on SnapBuild storm object.
- lp/snappy/model/snap.py - pass platform name through 'requestBuild' and 'requestBuildsFromJob'.
- lp/snappy/tests/test_snap.py - add tests to verify 'craft_platform' is passed through.
Build behaviour
- lp/snappy/model/snapbuildbehaviour.py - pass 'craft_platform' as extra argument to the builder.
- lp/snappy/tests/test_snapbuildbehaviour.py - test coverage for 'craft_platform' argument.
For more details, see:
https://code.launchpad.net/~alvarocs/launchpad/+git/launchpad/+merge/485084
--
Your team Launchpad code reviewers is requested to review the proposed merge of ~alvarocs/launchpad:use-craft-platforms-for-snap into launchpad:master.
References