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

Add btrfs #206

Closed
wants to merge 2 commits into from
Closed

Add btrfs #206

wants to merge 2 commits into from

Conversation

yodatak
Copy link

@yodatak yodatak commented Nov 12, 2022

I'm not god at c but i try to add a simple way to btrfs

I'm doing this for help batocera go full btrfs like fedora does

#158

Copy link
Member

@michaelolbrich michaelolbrich left a comment

Choose a reason for hiding this comment

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

Please squash the two commits and add your signed-off-by.

}

static cfg_opt_t btrfs_opts[] = {
CFG_STR("label", NULL, CFGF_NONE),
Copy link
Member

Choose a reason for hiding this comment

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

This should have an extraargs options like the other filesystems.

exec_test_set_prereq fsck.btrfs
test_expect_success mkfs_btrfs,fsck_btrfs "btrfs" "
run_genimage_root btrfs.config test.btrfs &&
fsck.btrfs images/test.btrfs
Copy link
Member

Choose a reason for hiding this comment

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

fsck.btrfs does not actually do anything, so just drop it here. I suppose we cannot actually validate the result...

@yodatak
Copy link
Author

yodatak commented May 7, 2023

If you want to take this PR take it i don't have time to make the changes :/

@yodatak yodatak closed this Jun 18, 2024
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