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

fix: partitioning: disable orphan_file (FEATURE_C12) for ext4 filesystems on 1.47+ e2fsprogs host #6809

Conversation

rpardini
Copy link
Member

fix: partitioning: disable orphan_file (FEATURE_C12) for ext4 filesystems on 1.47+ e2fsprogs host

  • fix: partitioning: disable orphan_file (FEATURE_C12) for ext4 filesystems on 1.47+ e2fsprogs host
    • otherwise:
      fsck.ext4 -a -C0 /dev/mmcblk0p1
      /dev/mmcblk0p1 has unsupported feature(s): FEATURE_C12
      e2fsck: Get a newer version of e2fsck!
      
    • orphan_file can only be disabled on recent-enough e2fsprogs, thus check version and compare

@github-actions github-actions bot added size/small PR with less then 50 lines Framework Framework components labels Jun 27, 2024
…esystems on 1.47+ e2fsprogs host

- otherwise:
  ```
  fsck.ext4 -a -C0 /dev/mmcblk0p1
  /dev/mmcblk0p1 has unsupported feature(s): FEATURE_C12
  e2fsck: Get a newer version of e2fsck!
  ```
- `orphan_file` can only be disabled on recent-enough e2fsprogs, thus check version and compare
@rpardini rpardini force-pushed the pr/fix-partitioning-disable-orphanfile-FEATUREC12-for-ext4-filesystems-on-147-e2fsprogs-host branch from 6841f43 to b0270f6 Compare June 27, 2024 07:37
@rpardini
Copy link
Member Author

To clarify: this fixes a problem that only happens if you use for example trixie to build a jammy image -- when the host supports (and defaults to enabling) orphan_file, but the target doesn't know about it.

@rpardini rpardini marked this pull request as ready for review June 27, 2024 08:29
@rpardini rpardini requested a review from a team as a code owner June 27, 2024 08:29
@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge 08 Milestone: Third quarter release labels Jun 27, 2024
@ColorfulRhino
Copy link
Collaborator

Do you know if this also happens when building a bookworm image or is it only jammy?

Maybe we should put some sort of hint/Todo in the code to clarify that this was only a temporary workaround until jammy image is not supported anymore. Otherwise someone will stumble over the code in 5 years and wonder why it's there and hasn't been removed.

@rpardini
Copy link
Member Author

rpardini commented Jun 27, 2024

Do you know if this also happens when building a bookworm image or is it only jammy?

It's not only Jammy. Any distro with e2fsprogs > 1.47 as build host, and any distro with e2fsprogs < 1.47 as target will have this problem. (There is also a kernel interaction, as orphan_file needs support in the kernel too; but I think it's >5.15)

@rpardini
Copy link
Member Author

someone will stumble over the code in 5 years and wonder why it's there and hasn't been removed.

git blame is that person's friend, and it will point to this PR ;-)

@rpardini rpardini merged commit 1ff783d into armbian:main Jun 27, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
08 Milestone: Third quarter release Framework Framework components Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines
Development

Successfully merging this pull request may close these issues.

3 participants