Skip to content

base:classes: Fix path for recipeinfo license file#1613

Merged
quaresmajose merged 1 commit intofoundriesio:mainfrom
angolini:fix_recipeinfo_path
Apr 2, 2025
Merged

base:classes: Fix path for recipeinfo license file#1613
quaresmajose merged 1 commit intofoundriesio:mainfrom
angolini:fix_recipeinfo_path

Conversation

@angolini
Copy link
Copy Markdown
Contributor

There are a fix in the DEPLOY_DIR/licenses/recipe/recipeinfo path generation in Scarthgap. The archtecture was included to avoid overlaping.

This patch uses the same mechanism used in license.bbclass to deploy the license files to search for them in image-license-checker.bbclass.

We can have references such as
https://patchwork.yoctoproject.org/project/oe-core/patch/[email protected]/ yoctoproject/poky@6cc2a36

@angolini angolini requested review from a team and quaresmajose March 31, 2025 19:27
@angolini
Copy link
Copy Markdown
Contributor Author

@quaresmajose , if you know a better way to solve this, please let me know

Copy link
Copy Markdown
Contributor

@Tim-Anderson Tim-Anderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think lines 109 and 110 need to be out dented to not be in the for loop

@angolini angolini force-pushed the fix_recipeinfo_path branch from 20e6f46 to fc70e02 Compare March 31, 2025 20:42
There are a fix in the DEPLOY_DIR/licenses/recipe/recipeinfo path
generation in Scarthgap. The archtecture was included to avoid
overlaping.

This patch uses the same mechanism used in license.bbclass to deploy the
license files to search for them in image-license-checker.bbclass.

We can have references such as
https://patchwork.yoctoproject.org/project/oe-core/patch/[email protected]/
yoctoproject/poky@6cc2a36

Signed-off-by: Daiane Angolini <[email protected]>
@angolini angolini force-pushed the fix_recipeinfo_path branch from fc70e02 to 223d270 Compare March 31, 2025 20:48
Copy link
Copy Markdown
Contributor

@Tim-Anderson Tim-Anderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@quaresmajose quaresmajose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@quaresmajose quaresmajose merged commit 523102e into foundriesio:main Apr 2, 2025
1 check passed
@angolini angolini deleted the fix_recipeinfo_path branch April 10, 2025 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants