-
Notifications
You must be signed in to change notification settings - Fork 257
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NXP BSP LF6.6.3-1.1.0: Update SCR and EULA #1778
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm ok with the PR, and to start the new upstream.
I only want to emphasize the 6.6.3-1.1.0 release is based on Nanbield, so we may want to finish the upstream before scarthgap, perhaps?
SCR/SCR-6.6.3-1.0.0.txt
Outdated
@@ -147,7 +176,7 @@ License File: LICENSE | |||
Package Category: BSP | |||
Type of Content: source | |||
Description and comments: BSP libraries | |||
Release Location: https://github.com/nxp-imx/imx-lib -b lf-6.1.55_2.2.0 | |||
Release Location: https://github.com/nxp-imx/imx-lib -b master |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm only double checking if "master" is ok here
Here [1], we have "lf-6.6.3_1.0.0" instead.
[1] https://github.com/nxp-imx/meta-imx/blob/nanbield-6.6.3-1.0.0/meta-imx-bsp/recipes-bsp/imx-lib/imx-lib_git.bbappend
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like the SCR is wrong. I will follow up internally.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@angolini thanks for notice this, I'm rechecking all "master" cases in SCR and fixing them properly. It seems a bug in meta--imx.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good but you need to update fsl-eula-unpack.bbclass
as well.
c3ca041
to
7e0c97f
Compare
This version is used in NXP BSP LF6.6.3-1.0.0. Signed-off-by: Oleksandr Suvorov <[email protected]>
Fixed wrong master branches for: - imx-lib -b lf-6.6.3_1.0.0 - imx-firmware -b lf-6.6.3_1.0.0 - imx-alsa-plugins -b MM_04.08.03_2312_L6.6.y Signed-off-by: Oleksandr Suvorov <[email protected]>
7e0c97f
to
910dc45
Compare
@thochstein Fixed, thanks! |
This PR is a part of solution to #1777