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

Port RJVB's patches to 0.8.2 #1

Open
rdmark opened this issue Jan 27, 2024 · 16 comments
Open

Port RJVB's patches to 0.8.2 #1

rdmark opened this issue Jan 27, 2024 · 16 comments

Comments

@rdmark
Copy link
Owner

rdmark commented Jan 27, 2024

A patchset for stability and fuse functionality was contributed for the baseline 0.8.1 codebase:

https://sourceforge.net/p/afpfs-ng/patches/4/

These patches no longer apply cleanly on 0.8.2 code, so this ticket is for tracking the porting effort.

@rdmark
Copy link
Owner Author

rdmark commented Jan 27, 2024

@rdmark
Copy link
Owner Author

rdmark commented Jan 27, 2024

@RJVB Thanks for contributing the above almost a decade ago. I always say, better late than never. ;)

@RJVB
Copy link

RJVB commented Jan 27, 2024 via email

@rdmark
Copy link
Owner Author

rdmark commented Jan 27, 2024

afpfs-ng-0.8.1-pointer.patch

Already in 0.8.2

@rdmark
Copy link
Owner Author

rdmark commented Jan 27, 2024

bump_fuse_use_version.patch

1180ca0

@rdmark
Copy link
Owner Author

rdmark commented Jan 27, 2024

return-a-value.patch

Already in 0.8.2

@rdmark
Copy link
Owner Author

rdmark commented Jan 28, 2024

RJVBs-patches.patch

e5748dd

RJVBs-patches-2.patch

69fd9db

@rdmark
Copy link
Owner Author

rdmark commented Jan 28, 2024

RJVBs-patches-askpass+fuse_options.patch
RJVBs-patches-subtype+fsname.patch

These patches introduce asprintf() which I believe is a GNU extension. I want to keep support for non-GNU OSes like *BSD and macOS. Needs some more work before adopting.

@RJVB
Copy link

RJVB commented Jan 28, 2024 via email

@rdmark
Copy link
Owner Author

rdmark commented Jan 29, 2024

@RJVB Thanks for the advice!

The portability might not have been as dire as I thought. With minor modifications this builds on Linux and FreeBSD: 8e433ee

There is some instability after a short while after mounting a fuse file system. Probably not related to this change, but more testing is needed.

@RJVB
Copy link

RJVB commented Jan 29, 2024 via email

@rdmark
Copy link
Owner Author

rdmark commented Jan 30, 2024

@RJVB Cheers, thanks for the offer!

My main usecase for afpfs is because I'm the maintainer of Netatalk these days, and it's handy to have a testbed that's not dependent on an actual Mac. :)

FWIW I don't think anything I did broke Fuse. Even going back to the first commit in my fork, the mounted Fuse file system is super unstable. It either freezes the Debian system after 2-3 file operations, OR runs into inexplicable permission denied errors. I need to read up more on Fuse I think to make sure I haven't misconfigured it...

@RJVB
Copy link

RJVB commented Jan 30, 2024 via email

@rdmark
Copy link
Owner Author

rdmark commented Jan 31, 2024

Was afpfs Fuse mounted volumes really that unstable 9 years ago? I want to believe that something else, like subsequent changes in the Fuse library since is causing this…

Anyhow, on a positive note the afpcmd tools work remarkably well (after a few bug fixes by yours truly) and is perfectly serviceable for one-off file transfers.

@RJVB
Copy link

RJVB commented Jan 31, 2024 via email

@RJVB
Copy link

RJVB commented Jan 31, 2024 via email

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

2 participants