Open tuxpoldo opened 10 years ago
The current check_btsync_user method doesn't just check whether btsync-user is installed, but also that the script itself came from the btsync-user package. The proposed solution would mean that if you've got btsync-user installed, you couldn't, for example run a standalone version of btsync and a seperate instance of the script for testing, since the script would always see and use the btsync-user scripts.
Do you think that losing this use case is going to be a problem, or is it worth the trade-off for the better performance?
I would suggest: make the change, but leave the old code commented. If nobody complains, after one year it shows that the use case is not really relevant.
As it's in Git I've just removed it rather than commenting it out, the commit can be reverted down the road if needed.
Can you test it out and check that it works as expected?
Reported by anton@ryzhov.me