在 2017年10月02日 20:19, Burton, Ross 写道:
On 29 September 2017 at 01:27, Yi Zhao <yi.z...@windriver.com <mailto:yi.z...@windriver.com>> wrote:

    Move wd_keepalive.service from watchdog package to watchdog-keepalive
    package. Also remove debian specific code from the service file.


Remember to say *why*.  Reading the diff tells us that you are putting wd-keepalive into a separate package, but doesn't explain *why*.

My patch just move wd_keepalive.service  file from wathdog package to watchdog-keepalive package to fix commit c14550deb824b3ef8087dede22d664879e85006d .  The package splitting was introduced by commit 7a79aba22be0ca0bc3535dea64a79b697013337d by Fabio Berton and reason is in the log message.

Yi



I suspect its because you should only be running one of them and if they both start then they'll race.  I also suspect the real solution is to have them either one or both disabled by default instead, as it appears to be perfectly reasonable to start with one and then swap to the other.

Ross

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to