Page MenuHomeFreeBSD

sysutils/fusefs-ntfs: Disable UBLIO
Needs ReviewPublic

Authored by koobs on Aug 10 2019, 2:43 AM.


Group Reviewers
Contributor Reviewers (ports)
sysutils/fusefs-ntfs: Disable UBLIO

Disable* the UBLIO port option as it is unmaintained, experimental code for
which there have been bug reports including data corruption when using it. 

Note: the bits to enable and patch NTFS-3g to use UBLIO have been left in
place to enable interested persons to test and potentially bring it back to
a maintained and stable state. To enable UBLIO, add UBLIO to the ports

While I'm here:


 - Make UUID functionality (and libuuid dependency) OPTION'al (UUID option
   enabled by default)
 - Add CRYPTO option (Enable crypto related code and utilities), enabled by
 - Add DEBUG option
 - Add EXTRAS option (ntfsdecrypt, ntfsrecover, ntfssecaudit, ntfstruncate,
   ntfsusermap, ntfswipe), enabled by default. EXTRAS implies (ntfsdecrypt
   breaks without) CRYPTO enabled.


 - Replace pkgconfigdir configure patch with one that adds a
   --with-pkgconfigdir option to instead. Add USES=autoreconf
   accordingly. Upstreamable
 - Replace configure patch removing the freebsd* error case in
   enable_ntfs_3g check with adding freebsd to list of OS's that get the
   --with-fuse option. Upstreamable.
 - Fix missing AM_PATH_LIBGRYPT macro errors by checking for their presence

 - Use install-strip to strip shared libraries/binaries instead of a custom
   post-install target.
 - Respect user/sysctl CFLAGS in, in particular to allow for
   -Wno-<warning> overrides.
 - Sort pkg-plist
[1] bug 194526, bug 206978

Diff Detail

rP FreeBSD ports repository
No Linters Available
No Unit Test Coverage
Build Status
Buildable 25772
Build 24345: arc lint + arc unit

Event Timeline

koobs created this revision.Aug 10 2019, 2:43 AM
koobs edited the summary of this revision. (Show Details)Aug 10 2019, 2:43 AM
arrowd added a subscriber: arrowd.Aug 12 2019, 9:25 AM
arrowd added inline comments.

Just a suggestion - how about adding # UBLIO to make it clear there is a hidden option?


Extra tab relative to CRYPTO_DESC.


A typo?

mat added a comment.Aug 13 2019, 2:08 PM

Please also convert the file to ucl so that is is only shown on install.

tobik added a subscriber: tobik.Aug 30 2019, 8:15 AM
tobik added inline comments.

This seems weird to me. The option should be discoverable not hidden. Putting up an additional barrier for interested users by hiding it in long documents (README.FreeBSD and the Makefile) that few people will read is artificial and unnecessary. Please just rename the option to something more sensible like UBLIO_EXPERIMENTAL or UBLIO_UNSTABLE and add it back to OPTIONS_DEFINE. The likelihood of users turning it on by accident or due to stale OPTIONS_SET should be minimal to nonexistent. The option could be further warned about by an improved option description and pkg-help.