Re: [PATCH(sbuild) 1/1] sbuild: Add changelog entry

["Bezdeka, Florian" <florian.bezdeka@siemens.com>]

On Wed, 2022-05-18 at 18:12 +0300, Anton Mikanovich wrote:
> Migration from buildchroot to sbuild will cause API change.
> Add changelog entry which also includes do_install_builddeps task
> remove mentioning.

Uh... Wait... Is that sbuild stuff already merged? 

I'm expecting quite some trouble in downstream layers when trying to
update ISAR next time. I would vote for the following to avoid a
"migrate all at once" scenario:

- Apply all pending patches (except sbuild stuff)
- Prepare a new "stable release 0.9" and maintain that "branch" for
  some time. 
- branch out for ISAR 1.0, apply sbuild stuff there

That would allow downstream layers to test and migrate in multiple
iterations against the 1.0 release. I guess sbuild will discover some
build problems. If layers have more such problems updating all at once
can be hard.

Further ideas?

Best regards,
Florian

> 
> Signed-off-by: Anton Mikanovich <amikan@ilbers.de>
> ---
>  RECIPE-API-CHANGELOG.md | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/RECIPE-API-CHANGELOG.md b/RECIPE-API-CHANGELOG.md
> index f3b3035..092292b 100644
> --- a/RECIPE-API-CHANGELOG.md
> +++ b/RECIPE-API-CHANGELOG.md
> @@ -397,3 +397,16 @@ New conversions can be added by defining CONVERSION_CMD_type.
>      - the conversions appends its own type, e.g. the output file of a conversion `xz`
>        would be ${IMAGE_FULLNAME}.${type}.xz
>      - a final chown is appended automatically
> +
> +### Buildchroot no longer used for package building
> +
> +Packages are now built with sbuild which takes care of dependency
> +installation.
> +The task do_install_builddeps has been removed.
> +
> +The migration to sbuild also means that all changes in the rootfs made during
> +package building will not be shared between the build sessions of different
> +packages and will be lost after a given build session finishes.
> +
> +Any package build requirements for the rootfs should be satisfied in the
> +Debian way via package dependencies.
> -- 
> 2.17.1
>