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

feat: quark builder with cli #8

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

leofvo
Copy link
Contributor

@leofvo leofvo commented Apr 10, 2022

No description provided.

@leofvo leofvo force-pushed the builder-leofvo branch 2 times, most recently from 9cd2ebd to 92fe82e Compare April 22, 2022 09:52
.arg(format!("{}kernel/.config", QUARK_CONFIG_DIR))
.output()
.expect("failed to copy kernel config");

Copy link
Contributor

Choose a reason for hiding this comment

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

You need to make oldconfig after the copy. Make sure it runs without needing any user interaction.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What should be expected from make oldconfig ? It seems to make minor change to .config file.

Copy link
Contributor

Choose a reason for hiding this comment

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

You need to run make oldconfig locally and use the generated .config. Or call make olddefconfig as it will automatically pick the defaults values.

@leofvo leofvo force-pushed the builder-leofvo branch 3 times, most recently from b0e6430 to c77302b Compare April 27, 2022 16:33
@leofvo leofvo force-pushed the builder-leofvo branch from c77302b to 4a9abcb Compare May 2, 2022 14:30
added:
  - makefile to simplify testing
  - basic cli with clap
  - quardle first minimal struct

Signed-off-by: leofvo <[email protected]>
@leofvo leofvo force-pushed the builder-leofvo branch 2 times, most recently from 794a31c to 1e92ff1 Compare May 2, 2022 20:04
@leofvo leofvo requested a review from sameo May 2, 2022 20:04
leofvo added 3 commits May 3, 2022 09:30
    - new: instanciate quardle
    - build: build quardle from instance
    - delete: removed files created to build quardle
    - add_config_file: create the config files associated to the quardle
    - make_archive: create the quardle archive

Signed-off-by: leofvo <[email protected]>
- add_kernel: add kernel binary to quardle

Signed-off-by: leofvo <[email protected]>
- setup: fetch all files needed to build inird image
- add_init_rd: adding initramfs image to quardle

Signed-off-by: leofvo <[email protected]>
@leofvo leofvo force-pushed the builder-leofvo branch from 1e92ff1 to 999973f Compare May 3, 2022 07:34
@leofvo leofvo changed the title draft: quark builder with cli feat: quark builder with cli May 3, 2022
@leofvo
Copy link
Contributor Author

leofvo commented May 3, 2022

Hey @sameo , my PR is ready for review !

@leofvo leofvo force-pushed the builder-leofvo branch 5 times, most recently from 999973f to 4fe07d4 Compare May 6, 2022 13:54
- setup: install kaps sources
- add_kaps: build kaps from sources and add it to quardle

Signed-off-by: leofvo <[email protected]>
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