It's a nit, but those diffs would be a bit easier to read if you'd change as few lines as possible, i.e. by appending the `optionals (self.pname != "hscolour") [hscolour]` bit at the end of the attribute, instead of putting it before the `[ghc Cabal] ++ self.extraBuildInputs` part.