I'm pretty sure `ghc` is a build input of every package built with `cabal` by default; passing that argument here is probably not necessary. Also, `Cabal` is a core package and implicitly available in every build. Generally speaking, that line should not be called buildInputs but `buildTools` because otherwise it conflicts with the inputs computed by cabal.mkDerivation.