magit/magit — comment on commit b57ac592b6e685c6249b3ca0617fa62b20fa01c3
File:  line 0
By sigma:

actually, I think the correct fix would be to change
`(eval-when-compile (require 'server))` to `(require 'server)`

Is there a reason why we `eval-when-compile` this ?
At the very least, if we absolutely want to load it lazily, we should `require` it at the very beginning of `magit-server-running-p`, since it seems to be the only place where it's used