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

ext4 support dependency error #47

Closed
lherschi opened this issue Apr 17, 2023 · 10 comments
Closed

ext4 support dependency error #47

lherschi opened this issue Apr 17, 2023 · 10 comments

Comments

@lherschi
Copy link

README, automount and Makefile in the FreeBSD ports tree refer to the wrong fuse module for ext4 support.

README:
sysutils/fusefs-ext2 -> sysutils/fusefs-lkl

automount:
sysutils/fusefs-ext4fuse -> sysutils/fusefs-lkl

Makefile:
fusefs-ext2>=0:sysutils/fusefs-ext2 -> fusefs-lkl>=0:sysutils/fusefs-lkl

@lherschi
Copy link
Author

lherschi commented Apr 17, 2023

EXT4_RUN_DEPENDS in the Makefile should even better become

EXT4_RUN_DEPENDS= fusefs-lkl>=0:sysutils/fusefs-lkl \
                  e2fsprogs-core>=0:sysutils/e2fsprogs-core

@vermaden
Copy link
Owner

Will look into that.

@outpaddling
Copy link

I can confirm this. I created an ext4 fs on a USB stick and it would not mount until I installed fusefs-lkl.

@outpaddling
Copy link

@vermaden
Copy link
Owner

Sorry for being really late on this one.

Implemented along with several other fixes/features.

Please test:

Regards,
vermaden

@lherschi
Copy link
Author

lherschi commented Aug 23, 2023

In the automount file, the reference to fusefs-ext4fuse (in __usage()) is still present. The README has not changed at all and the FreeBSD ports makefile is not even in this source tree.

@vermaden
Copy link
Owner

I only modified https://github.com/vermaden/automount/blob/master/automount for now.

I tested it and it works for me.

If other report it works - I will update README and send a request to update the package/port in FreeBSD Ports tree.

... and I will update the usage() in a moment.

@vermaden
Copy link
Owner

The usage() is also updated now.

@vermaden
Copy link
Owner

Done.

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

No branches or pull requests

3 participants