-
Notifications
You must be signed in to change notification settings - Fork 2
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
sync path modifications, cargo bump, minor cleanups #21
base: master
Are you sure you want to change the base?
Conversation
kallisti5
commented
Nov 8, 2023
- Resolves build failure mentioned here: Can’t compile smartstring bodil/smartstring#31
actually, running into other random build failures as well. Might be time for a full |
* Resolves a few misc build errors.
6b9a760
to
f337320
Compare
One reason i'm fiddling here is I want to add a froms3 😸 |
I think it might be time for a full Do you want write access to this repo? |
I'll see if I can make this into a setting over christmas. |
I should note, it's not easy to "convert" to a new sync design. I swapped over to this design by moving files around on mfs:
(.sync matching the state tracking dir) However, ftp2mfs with these modifications doesn't catch the existing data..
Maybe the context directory for the state file is wrong
|
Ah! it works! You have to fully erase any existing /temp/XXX folders when you convert to the new layout. Then it works as expected...
|
From a design standpoint, maybe add the ability to specify a custom state tracking directory? Then you can place it at an alternative location not directly within the sync path.
|