Page MenuHomeFreeBSD

Update fusefs-lkl to recent version
ClosedPublic

Authored by cem on Aug 16 2016, 5:26 AM.

Details

Summary

Update fusefs-lkl from a 4.3.0 based master to more recent master.

This brings in support for extended filesystem options, needed for non-trivial
BTRFS configurations.

Test Plan

I successfully mounted an ext4 filesystem and manipulated files in it.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

cem retitled this revision from to Update fusefs-lkl to recent version.
cem updated this object.
cem edited the test plan for this revision. (Show Details)
cem added reviewers: mat, bapt, bdrewery.
sysutils/fusefs-lkl/files/patch-lib_Build
7–10 ↗(On Diff #19317)

These use Linux-specific epoll and invoke Linux-specific tun/tap device configuration. I believe they could be ported to FreeBSD by a motivated individual somewhat familiar with tun(4)/tap(4). They are not needed for lklfuse, so omitting them is fine for now IMO.

sysutils/fusefs-lkl/files/patch-lib_posix-host.c
7–18 ↗(On Diff #19317)

These I intend to send upstream.

sysutils/fusefs-lkl/Makefile
33 ↗(On Diff #19317)

If this is so that it gets bc from its BUILD_DEPENDS, it should be LOCALBASE, not PREFIX.

44 ↗(On Diff #19317)

This should be rephrased (11+), or the OSVERSION check changed, (1200000)

cem marked 2 inline comments as done.

Fix bc/ld PATH and rephrase OSVERSION check now that CURRENT is 12.

Update to newer git master.

  • Remove patch-lib_posix-host.c, which has been upstreamed.
mat edited edge metadata.

Looks good.

Maybe the OSVERSION check could be checked, 1100000 seems to be rather odd. it is what 11 was just after stable/10 was branched, so, 10 should work, but does not, so, I wonder if it should be changed to be the right one. See https://www.freebsd.org/doc/en/books/porters-handbook/versions.html for the list of all versions.

This revision is now accepted and ready to land.Aug 17 2016, 11:53 AM
This revision was automatically updated to reflect the committed changes.