[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Collaborating on the HFS code



Hello!

Since I have posted about my initial work on my updated »hfsprogs« [1]
package based on the new »hfs« [2] package, multiple people send me patches
for improvement off-list.

Since it's easier and more productive to collaborate on this project in public,
I would like to ask everyone to submit patches via pull request on the Github
project.

The ultimate goal should be to have a minimal patch set which can be applied
on top of Apple's »hfs« package to get a decent »hfsprogs« package for Linux.

The smaller the patch set, the easier it will be to keep the package up to date.

As a bonus, I think we should also refresh my patch to re-add support for classic
HFS, see [3].

There is currently one limitation with the HFS patch as it doesn't allow setting
a custom volume name and hard-wires it to "untitled" [4]. This is because the
original code uses an Apple-specifc function to convert UTF-8 into Mac encoding [5],
if someone has a suggestion how to implement this encoding conversion on Linux, I
would be happy to integrate this change into the (current) »hfsprogs« package as
this would fix an open bug [6].

Any input is greatly appreciated as I know many here have lots of expertise on Mac
programming.

Thanks,
Adrian

> [1] https://buildd.debian.org/status/package.php?p=hfsprogs&suite=sid
> [2] https://github.com/glaubitz/hfs/commits/linux
> [3] https://salsa.debian.org/debian/hfsprogs/-/blob/master/debian/patches/0005-Re-add-support-for-creating-legacy-HFS-filesystems.patch
> [4] https://salsa.debian.org/debian/hfsprogs/-/blob/master/debian/patches/0005-Re-add-support-for-creating-legacy-HFS-filesystems.patch#L319
> [5] https://github.com/apple-opensource/diskdev_cmds/blob/491.6/newfs_hfs.tproj/makehfs.c#L598
> [6] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1003038

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913


Reply to: