adulau / Forban

Forban is a p2p application for link-local and local area networks. Forban works independently from the Internet and uses only the local area capabilities to announce, discover, search or share files. Forban relies on HTTP and it is "opportunistic".
http://www.foo.be/forban/
134 stars 24 forks source link

.pid File option -> OpenWRT integration #7

Closed MaStr closed 12 years ago

MaStr commented 12 years ago

Hi, I need an option for placing the pid files into another place on folder. As I mentioned on the other thread, I wanted to place the /var folder to /tmp , that it gets removed after reboot. This seems to cause other problems.

Please, can u please create an option, where the pid files should be located?

Thank you.

Matthias, btw: remove the blank space on "indexrebuild" option (I ran into an error because I only removed the # ;) ) it's an noob-failure point ;)

adulau commented 12 years ago

Hi Matthias,

Sure, I can update that. But wouldn't be more useful to add an option in forbanctl to force restart of Forban even if a pid is still present (also cross-checking if Forban is still running or not )? A "forbanctl forcestart" is that fine for you? Like that, you don't need to update the config but just use a forcestart for each reboot/restart in the init script. I'll check and try to commit this in the next hours. This could be useful for the people using Forban in other "standalone" mode.

See ya and thank you for your tests.

adulau

MaStr commented 12 years ago

Hi adulau,

yes, it is possible option. Personally, I like clean starts/stops more than a forced mode. But yes, it is even more easier (and less dependencies, which is good, on the other side).

Matthias

adulau commented 12 years ago

Hi Matthias.

I added a cleanup mode in forbanctl that can be called before the start on the OpenWRT box. Should be useful for all platforms. I tested on two standalone systems and it worked. By the way, I added you in the AUTHORS file. If this is fine for your package tool. I'll tag this Forban to the next version.

Thank you.

MaStr commented 12 years ago

Thank you, I'll test it soon