diff mbox series

migration-5.2: Add entries for virtual provider and multiconfig changes

Message ID 20250126111850.2609298-1-richard.purdie@linuxfoundation.org
State Under Review
Headers show
Series migration-5.2: Add entries for virtual provider and multiconfig changes | expand

Commit Message

Richard Purdie Jan. 26, 2025, 11:18 a.m. UTC
Add some information to the migration guide about two important potentially
breaking changes in 5.2.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 .../migration-guides/migration-5.2.rst        | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)

[Note, I didn't test the formatting of this, I just wanted to get some text shared
so we don't forget these. Hope that helps!]

Comments

Jörg Sommer Jan. 26, 2025, 12:09 p.m. UTC | #1
Richard Purdie via lists.yoctoproject.org schrieb am So 26. Jan, 11:18 (+0000):
> Add some information to the migration guide about two important potentially
> breaking changes in 5.2.
> 
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>  .../migration-guides/migration-5.2.rst        | 22 +++++++++++++++++++
>  1 file changed, 22 insertions(+)
> 
> [Note, I didn't test the formatting of this, I just wanted to get some text shared
> so we don't forget these. Hope that helps!]
> 
> diff --git a/documentation/migration-guides/migration-5.2.rst b/documentation/migration-guides/migration-5.2.rst
> index e9de6910e..fdf44757a 100644
> --- a/documentation/migration-guides/migration-5.2.rst
> +++ b/documentation/migration-guides/migration-5.2.rst
> @@ -94,6 +94,28 @@ systemd changes
>     not for socket files).
>     Now all service files must be explicitly added to :term:`FILES`.
>  
> +Multiconfig changes
> +~~~~~~~~~~~~~~~~~~~
> +
> +-  The value of BB_CURRENT_MC was changed from 'default' to an empty string for the
> +   default multiconfig configuration to avoid needing to map the values within bitbake.
> +   This was already not happening insome cases so fixes some obscure bugs.
                                     ^^^^^^

I think a space is missing here.


Jörg
Antonin Godard Jan. 27, 2025, 8:23 a.m. UTC | #2
Hi Richard,

Some formatting related changes.

On Sun Jan 26, 2025 at 12:18 PM CET, Richard Purdie via lists.yoctoproject.org wrote:
> Add some information to the migration guide about two important potentially
> breaking changes in 5.2.
>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>  .../migration-guides/migration-5.2.rst        | 22 +++++++++++++++++++
>  1 file changed, 22 insertions(+)
>
> [Note, I didn't test the formatting of this, I just wanted to get some text shared
> so we don't forget these. Hope that helps!]
>
> diff --git a/documentation/migration-guides/migration-5.2.rst b/documentation/migration-guides/migration-5.2.rst
> index e9de6910e..fdf44757a 100644
> --- a/documentation/migration-guides/migration-5.2.rst
> +++ b/documentation/migration-guides/migration-5.2.rst
> @@ -94,6 +94,28 @@ systemd changes
>     not for socket files).
>     Now all service files must be explicitly added to :term:`FILES`.
>  
> +Multiconfig changes
> +~~~~~~~~~~~~~~~~~~~
> +
> +-  The value of BB_CURRENT_MC was changed from 'default' to an empty string for the
> +   default multiconfig configuration to avoid needing to map the values within bitbake.
> +   This was already not happening insome cases so fixes some obscure bugs.
> +
> +Virtual toolchain provider changes
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> +
> +-  Support was added for virtual providers on a per recipe basis for specific key toolchain
> +   providers listed in BB_VIRTUAL_RECIPE_PROVIDERS. As part of that support, the 'cross'
> +   providers were simplified to remove the triplet prefix and generalise the naming, so
> +   for example::
> +
> +     virtual/${TARGET_PREFIX}-binutils - > virtual/cross-bintuils
> +     virtual/${TARGET_PREFIX}-gcc - > virtual/cross-cc
> +     virtual/${TARGET_PREFIX}-g++ - > virtual/cross-c++
> +     virtual/${TARGET_PREFIX}-compilerlibs - > virtual/compilerlibs
> +     virtual/${SDK_PREFIX}-binutils - > virtual/nativesdk-cross-binutils
> +     virtual/${SDK_PREFIX}-gcc - > virtual/nativesdk-cross-cc

The code-block should be indented with three spaces.

Other than that I'm taking note that we are missing documentation for
BB_CURRENT_MC and BB_VIRTUAL_RECIPE_PROVIDERS, but that is already partly
mentioned in https://bugzilla.yoctoproject.org/show_bug.cgi?id=15709.

I'll try working on it soon.

Antonin
diff mbox series

Patch

diff --git a/documentation/migration-guides/migration-5.2.rst b/documentation/migration-guides/migration-5.2.rst
index e9de6910e..fdf44757a 100644
--- a/documentation/migration-guides/migration-5.2.rst
+++ b/documentation/migration-guides/migration-5.2.rst
@@ -94,6 +94,28 @@  systemd changes
    not for socket files).
    Now all service files must be explicitly added to :term:`FILES`.
 
+Multiconfig changes
+~~~~~~~~~~~~~~~~~~~
+
+-  The value of BB_CURRENT_MC was changed from 'default' to an empty string for the
+   default multiconfig configuration to avoid needing to map the values within bitbake.
+   This was already not happening insome cases so fixes some obscure bugs.
+
+Virtual toolchain provider changes
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+-  Support was added for virtual providers on a per recipe basis for specific key toolchain
+   providers listed in BB_VIRTUAL_RECIPE_PROVIDERS. As part of that support, the 'cross'
+   providers were simplified to remove the triplet prefix and generalise the naming, so
+   for example::
+
+     virtual/${TARGET_PREFIX}-binutils - > virtual/cross-bintuils
+     virtual/${TARGET_PREFIX}-gcc - > virtual/cross-cc
+     virtual/${TARGET_PREFIX}-g++ - > virtual/cross-c++
+     virtual/${TARGET_PREFIX}-compilerlibs - > virtual/compilerlibs
+     virtual/${SDK_PREFIX}-binutils - > virtual/nativesdk-cross-binutils
+     virtual/${SDK_PREFIX}-gcc - > virtual/nativesdk-cross-cc
+
 Recipe changes
 ~~~~~~~~~~~~~~