bug#24491: Avoid recursive require of eshell

[Glenn Morris]

You ought to be able to avoid such problems by using
(with-)eval-after-load instead of foo-load-hook, since the former runs
after provide (effectively).

Frankly all foo-load-hooks are probably historical baggage that should
be obsoleted. There's no need for them to exist given that a general
solution does.