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

Choice of the cache directory #36

Merged
merged 1 commit into from Jun 5, 2016

Conversation

Projects
None yet
3 participants
@tYYGH
Copy link
Contributor

tYYGH commented Jun 18, 2015

My server has a very small /tmp, which caused MegaFuse to fail after a time. Unfortunately, I saw that "/tmp" was hardcoded.
I have next to no knowledge in C++, but with the help of others, I managed to adapt the software to my needs. So far it seems to work fine.
If you don't spot problems in the changes, I would be glad if you would include these.

@matteoserva matteoserva merged commit 5d16474 into matteoserva:master Jun 5, 2016

char filename[] = "/tmp/mega.XXXXXX";
close (mkstemp(filename));
std::string filename = Config::getInstance()->CACHEPATH + "/mega.XXXXXX";
close (mkstemp(&filename[0]));

This comment has been minimized.

@fenollp

fenollp Sep 29, 2016

shouldn't that be filename.c_str()?

This comment has been minimized.

@tYYGH

tYYGH Sep 29, 2016

Contributor

Well… possibly ;-)
I did what I could with my Java knowledge and a fair bit of research+help.
Feel free to submit corrections :-)
However, this pull-request is closed, as it has been merged into MegaFuse already. So I suppose you’ll have to create another pull-request.
Cheers,

This comment has been minimized.

@fenollp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment