Skip to content

move e2fsprogs dependency from fork to upstream#1254

Open
qkaiser wants to merge 3 commits intomainfrom
1252-fix-e2fsprogs-dependency
Open

move e2fsprogs dependency from fork to upstream#1254
qkaiser wants to merge 3 commits intomainfrom
1252-fix-e2fsprogs-dependency

Conversation

@qkaiser
Copy link
Contributor

@qkaiser qkaiser commented Sep 2, 2025

Our debugfs fix is present in the debian trixie version and in latest Ubuntu. We can therefore go back to installing it from upstream rather than relying on our own fork.

Which means we can revert c13a053 and move to Debian Trixie for the Docker container.

@qkaiser qkaiser requested a review from vlaci September 2, 2025 10:07
@qkaiser qkaiser self-assigned this Sep 2, 2025
@qkaiser qkaiser added the dependencies Pull requests that update a dependency file label Sep 2, 2025
@qkaiser
Copy link
Contributor Author

qkaiser commented Sep 2, 2025

ubuntu-latest at Github is actually Ubuntu Noble (24.04) which is still on e2fsprogs 1.47.0-2 while we need 1.47.2-2.

@e3krisztian
Copy link
Contributor

It is very strange, that the failure https://github.com/onekey-sec/unblob/actions/runs/17400208989/job/49391774625?pr=1254#step:5:1047 happens only in one of the python environments, not all of them.
If it were an rdump-only problem, it should be a problem with all of the supported python versions.

@qkaiser
Copy link
Contributor Author

qkaiser commented Sep 29, 2025

It is very strange, that the failure https://github.com/onekey-sec/unblob/actions/runs/17400208989/job/49391774625?pr=1254#step:5:1047 happens only in one of the python environments, not all of them. If it were an rdump-only problem, it should be a problem with all of the supported python versions.

It's affecting all of them, it's just that the others got cancelled. If a failure is observed in one of the tests, the others are cancelled.

@vlaci
Copy link
Contributor

vlaci commented Oct 16, 2025

ubuntu-latest at Github is actually Ubuntu Noble (24.04) which is still on e2fsprogs 1.47.0-2 while we need 1.47.2-2.

we may run the tests inside docker or force install e2fsprogs from debian

@qkaiser qkaiser force-pushed the 1252-fix-e2fsprogs-dependency branch 8 times, most recently from 95e44c1 to a221185 Compare January 30, 2026 15:09
@qkaiser
Copy link
Contributor Author

qkaiser commented Jan 30, 2026

Chose to run tests in docker container with Ubuntu 25.10 but building dev dependencies (taplo, specifically) is .. difficult.

@qkaiser qkaiser force-pushed the 1252-fix-e2fsprogs-dependency branch 3 times, most recently from 7867a75 to 4221e65 Compare February 6, 2026 16:37
@qkaiser qkaiser requested a review from kukovecz February 6, 2026 16:37
@qkaiser
Copy link
Contributor Author

qkaiser commented Feb 6, 2026

We need this #1368 first

@qkaiser qkaiser force-pushed the 1252-fix-e2fsprogs-dependency branch 4 times, most recently from 10dc348 to d24edae Compare February 10, 2026 11:35
@qkaiser qkaiser enabled auto-merge February 10, 2026 11:35
@qkaiser qkaiser force-pushed the 1252-fix-e2fsprogs-dependency branch 2 times, most recently from 3a04593 to 40e7bc1 Compare February 11, 2026 08:14
Our debugfs fix¹ is present in the debian trixie version² and in latest
Ubuntu³. We can therefore go back to installing it from upstream rather
than relying on our own fork.

¹:
tytso/e2fsprogs@56a444e
²:
https://metadata.ftp-master.debian.org/changelogs//main/e/e2fsprogs/e2fsprogs_1.47.2-3_changelog
³: https://launchpad.net/ubuntu/+source/e2fsprogs
@qkaiser qkaiser force-pushed the 1252-fix-e2fsprogs-dependency branch 11 times, most recently from 90197a2 to 00d6416 Compare February 11, 2026 11:38
@qkaiser qkaiser force-pushed the 1252-fix-e2fsprogs-dependency branch from 00d6416 to 990c497 Compare February 11, 2026 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants