On Wednesday 17 February 2016 11:36:01 Dominik Heidler wrote:
Am 17.02.2016 um 09:39 schrieb Sven Eckelmann:
Your patch also doesn't explain why this feature may be required.
I'm running an alfred master server with this patch, so that I can run a script that will inject data of legacy clients (that don't have an alfred slave, but need to be crawled).
Sorry for the late reply. In generally I think this change is acceptable, however I'd like to request the following changes and to resend your patch:
* Please write a proper commit message in the patch, explaining your scenario and why you need it * Write a comment above the change you made to explain that clients should set the mac to 00:00:00:00:00:00 by default, unless they want alfred to use its mac for them.
Thank you! Simon