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

Send bitbake -e <package> output to the server for further processing #52

Conversation

WilsonZiweiWang
Copy link
Contributor

@WilsonZiweiWang WilsonZiweiWang commented Jan 5, 2024

This PR adds an option in the contextual menu to trigger bitbake some-recipe -e in the terminal and uses the results to provide more robust information about the environment of the selected recipe such as the complete include history, the current final value for a variable, etc. sends the results to the server for further processing.

Edit: This PR only prepares the parsing.
Edit-2: The actual processing will be in other PRs.

Ticket No. 10429

@WilsonZiweiWang WilsonZiweiWang added the help wanted Extra attention is needed label Jan 5, 2024
@WilsonZiweiWang WilsonZiweiWang self-assigned this Jan 5, 2024
@WilsonZiweiWang WilsonZiweiWang changed the title Scan recipe envrionment for better language service Scan recipe environment for better language service Jan 5, 2024
@WilsonZiweiWang WilsonZiweiWang removed the help wanted Extra attention is needed label Jan 8, 2024
@WilsonZiweiWang WilsonZiweiWang force-pushed the Feature-10429-parse-recipe-in-menu branch from 6a1dbea to b41ef61 Compare January 9, 2024 21:21
client/src/ui/BitbakeCommands.ts Outdated Show resolved Hide resolved
client/src/ui/BitbakeCommands.ts Outdated Show resolved Hide resolved
client/package.json Outdated Show resolved Hide resolved
client/src/lib/src/types/requests.ts Outdated Show resolved Hide resolved
client/src/lib/src/types/requests.ts Outdated Show resolved Hide resolved
client/src/ui/BitbakeTaskProvider.ts Show resolved Hide resolved
client/src/ui/BitbakeTaskProvider.ts Outdated Show resolved Hide resolved
client/src/ui/BitbakeCommands.ts Outdated Show resolved Hide resolved
client/src/ui/BitbakeTerminal.ts Outdated Show resolved Hide resolved
@WilsonZiweiWang WilsonZiweiWang force-pushed the Feature-10429-parse-recipe-in-menu branch from 382f61c to 089ef22 Compare January 10, 2024 18:51
@WilsonZiweiWang WilsonZiweiWang force-pushed the Feature-10429-parse-recipe-in-menu branch from 089ef22 to 62ef6f9 Compare January 11, 2024 16:20
@WilsonZiweiWang WilsonZiweiWang force-pushed the Feature-10429-parse-recipe-in-menu branch from d4c56a0 to 95860da Compare January 12, 2024 15:45
@WilsonZiweiWang WilsonZiweiWang changed the base branch from staging-next to staging January 12, 2024 15:45
@WilsonZiweiWang WilsonZiweiWang marked this pull request as ready for review January 12, 2024 15:45
@WilsonZiweiWang WilsonZiweiWang force-pushed the Feature-10429-parse-recipe-in-menu branch from 95860da to b1d001c Compare January 12, 2024 16:08
@WilsonZiweiWang WilsonZiweiWang changed the title Scan recipe environment for better language service Send bitbake -e <package> output to the server for further processing through contextual menu Jan 17, 2024
@WilsonZiweiWang WilsonZiweiWang changed the title Send bitbake -e <package> output to the server for further processing through contextual menu Send bitbake -e <package> output to the server for further processing Jan 26, 2024
Copy link
Member

@deribaucourt deribaucourt left a comment

Choose a reason for hiding this comment

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

Nearly there, a few refactoring suggestions

client/src/driver/BitbakeRecipeScanner.ts Outdated Show resolved Hide resolved
client/src/driver/BitbakeRecipeScanner.ts Outdated Show resolved Hide resolved
client/src/driver/BitbakeRecipeScanner.ts Outdated Show resolved Hide resolved
client/src/ui/BitbakeCommands.ts Show resolved Hide resolved
server/src/tree-sitter/analyzer.ts Show resolved Hide resolved
server/src/tree-sitter/analyzer.ts Show resolved Hide resolved
@WilsonZiweiWang WilsonZiweiWang force-pushed the Feature-10429-parse-recipe-in-menu branch from ee0019c to 00d2129 Compare January 29, 2024 22:09
server/src/server.ts Outdated Show resolved Hide resolved
…m one by one in a queue-like manner

This should allow the scan requests, that are added during the task execution, to be handled one by one.
The current uri is re-assigned before the execution to ensure the correct uri is received by the server.
@WilsonZiweiWang WilsonZiweiWang force-pushed the Feature-10429-parse-recipe-in-menu branch from 00d2129 to 2cec61b Compare January 30, 2024 16:34
@WilsonZiweiWang WilsonZiweiWang merged commit 6329fa5 into yoctoproject:staging Jan 31, 2024
2 checks passed
@WilsonZiweiWang WilsonZiweiWang deleted the Feature-10429-parse-recipe-in-menu branch January 31, 2024 15:52
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.

2 participants