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

update(guest): fix baremetal nix recipes #3

Merged
merged 20 commits into from
Jan 22, 2024
Merged

Conversation

Diogo21Costa
Copy link
Member

PR Description

This PR introduces changes to the guests Nix build recipes. Moreover, it includes the renaming of the following recipes:

Old name New name Guest Sources Support Test Framework
baremetal-guest.nix baremetal-remote.nix Baremetal Remote No
- baremetal-local.nix Baremetal Local No
baremetal-bao-tf.nix baremetal-remote-tf.nix Baremetal Remote Yes
baremetal-tf.nix baremetal-local-tf.nix Baremetal Local Yes

Furthermore, beyond just renaming files, this PR extends the recipes' versatility by making them more generic and applicable. It eliminates static paths and removes redundant build steps that were too closely tied to specific use-cases.

For simplicity's sake, I've included a folder in the commit d792258 that offers a quick guide on testing each of these refined Nix recipes. Please note that this specific commit is intended to be removed before merging it into the main branch.

Please provide your feedback on these changes.

Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
Signed-off-by: Diogo Costa <diogoandreveigacosta@gmail.com>
@ESCristiano ESCristiano self-requested a review January 4, 2024 15:58
Signed-off-by: Miguel Silva <miguelafsilva5@gmail.com>
Co-authored-by: Diogo Costa <diogoandreveigacosta@gmail.com>
@Diogo21Costa Diogo21Costa force-pushed the update/guests-recipes branch from 0ff9a3a to b22a784 Compare January 22, 2024 14:05
@miguelafsilva5 miguelafsilva5 self-assigned this Jan 22, 2024
@miguelafsilva5 miguelafsilva5 merged commit 6c5f2bc into main Jan 22, 2024
2 checks passed
@miguelafsilva5 miguelafsilva5 deleted the update/guests-recipes branch January 22, 2024 14:08
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