Skip to content
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

View skipped recipes #322

Merged

Conversation

deribaucourt
Copy link
Member

Closes #317

This will allow us to track which recipes were marked as skipped by
bitbake.

Skipped recipes are recipes that are not buildable in the current
configuration, but available in the workspace.
We use this variable to determine if the scan result data saved in the
workspace state is up to date. Since this should be modified whenever
the structure of the scan data changes, it makes sense to keep it in the
same file as the scan data structure.

Also added a reminder comment to increment the version number when the
structure changes.
We are going to display these recipes with a description in the
BitbakeRecipesView. It will allow users to see skipped recipes they
would be interested in, and understand why they are skipped.
Now that we keep and track skipped recipe items, we want to know
their source. Adapt the regex to capture the filename and truncate
the skipped reason. It is already captured in the ElementInfo.skipped
from the recipe scan.
This will allow users to know why recipes are not buildable when
they were skipped.

A warning icon and description will clearly present this.

Not much performance impact for recipes array traversal since this
is only done on the hand-selected active recipes.
@deribaucourt
Copy link
Member Author

image

Even though their shouldn't be many active recipes, we want to avoid
looping multiple times through the list of recipes.
The SCAN_COMPLETE event needed to be reordered so that the data is
passed before computing the view.
Copy link
Contributor

@WilsonZiweiWang WilsonZiweiWang left a comment

Choose a reason for hiding this comment

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

LGTM

@deribaucourt deribaucourt merged commit 4c11212 into yoctoproject:staging Sep 18, 2024
5 checks passed
@deribaucourt deribaucourt deleted the view-skipped-recipes branch September 18, 2024 14:49
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.

Show in recipes explorer skipped recipes
2 participants