Simplify Nix expressions #329
|
|
@ -56,3 +56,11 @@ jobs:
|
|||
path: dist-newstyle/sdist/brittany-*.tar.gz
|
||||
name: brittany-${{ github.sha }}.tar.gz
|
||||
- run: cabal check
|
||||
|
||||
|
||||
nix:
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- uses: cachix/install-nix-action@v12
|
||||
- uses: actions/checkout@v2
|
||||
- run: nix-build
|
||||
|
|
|
|||
16
README.md
|
|
@ -70,7 +70,7 @@ log the size of the input, but _not_ the full input/output of requests.)
|
|||
|
||||
- via `nix`:
|
||||
~~~.sh
|
||||
nix build -f release.nix # or 'nix-build -f release.nix'
|
||||
nix build
|
||||
nix-env -i ./result
|
||||
~~~
|
||||
|
||||
|
|
@ -102,6 +102,20 @@ log the size of the input, but _not_ the full input/output of requests.)
|
|||
aura -A brittany
|
||||
~~~~
|
||||
|
||||
# Development tips
|
||||
|
||||
## Run a hoogle server
|
||||
|
||||
To host a local Hoogle server with all of Brittany's dependencies run:
|
||||
|
||||
```sh
|
||||
echo brittany.cabal |
|
||||
$(nix-build '<nixpkgs>' --no-link -A entr)/bin/entr -r -- \
|
||||
sh -c "nix-shell --run 'hoogle server --local'"
|
||||
```
|
||||
|
||||
This will watch `brittany.cabal` for changes and restart the server when new dependencies are added there.
|
||||
|
||||
# Editor Integration
|
||||
|
||||
#### Sublime text
|
||||
|
|
|
|||
52
default.nix
|
|
@ -1,38 +1,18 @@
|
|||
|
|
||||
{ mkDerivation, aeson, base, butcher, bytestring, cmdargs
|
||||
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
, containers, czipwith, data-tree-print, deepseq, directory, extra
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
, filepath, ghc, ghc-boot-th, ghc-exactprint, ghc-paths, hspec
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
, monad-memo, mtl, multistate, neat-interpolation, parsec, pretty
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
, random, safe, semigroups, stdenv, strict, syb, text, transformers
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
, uniplate, unsafe, yaml
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
{ nixpkgsSrc ? builtins.fetchTarball {
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
url =
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
"https://github.com/nixos/nixpkgs/archive/069f183f16c3ea5d4b6e7625433b92eba77534f7.tar.gz"; # nixos-unstable
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
sha256 = "1by9rqvr2k6iz2yipf89yaj254yicpwq384ijgyy8p71lfxbbww2";
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
}, pkgs ? import nixpkgsSrc { }, compiler ? null, forShell ? pkgs.lib.inNixShell
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
}:
|
||||
mkDerivation {
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
pname = "brittany";
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
version = "0.11.0.0";
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
src = ./.;
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
isLibrary = true;
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
isExecutable = true;
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
libraryHaskellDepends = [
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
aeson base butcher bytestring cmdargs containers czipwith
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
data-tree-print deepseq directory extra filepath ghc ghc-boot-th
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
ghc-exactprint ghc-paths monad-memo mtl multistate
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
neat-interpolation pretty random safe semigroups strict syb text
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
transformers uniplate unsafe yaml
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
];
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
executableHaskellDepends = [
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
aeson base butcher bytestring cmdargs containers czipwith
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
data-tree-print deepseq directory extra filepath ghc ghc-boot-th
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
ghc-exactprint ghc-paths monad-memo mtl multistate
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
neat-interpolation pretty safe semigroups strict syb text
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
transformers uniplate unsafe yaml
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
];
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
testHaskellDepends = [
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
aeson base butcher bytestring cmdargs containers czipwith
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
data-tree-print deepseq directory extra filepath ghc ghc-boot-th
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
ghc-exactprint ghc-paths hspec monad-memo mtl multistate
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
neat-interpolation parsec pretty safe semigroups strict syb text
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
transformers uniplate unsafe yaml
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
];
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
homepage = "https://github.com/lspitzner/brittany/";
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
description = "Haskell source code formatter";
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
license = stdenv.lib.licenses.agpl3;
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
let
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
haskellPackages = if compiler == null then
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
pkgs.haskellPackages
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
else
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
pkgs.haskell.packages.${compiler};
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
in haskellPackages.developPackage {
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
name = "brittany";
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
root = pkgs.nix-gitignore.gitignoreSource [ ] ./.;
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
returnShellEnv = forShell;
|
||||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
}
|
||||
|
|
|
|||
|
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
Should this have Should this have `name = "brittany"`, or does it not matter?
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
Review
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed. It doesn't *really* matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
|
||||
5
pkgs.nix
|
|
@ -1,5 +0,0 @@
|
|||
{
|
||||
url = "https://github.com/nixos/nixpkgs.git";
|
||||
ref = "release-18.09";
|
||||
rev = "b9fa31cea0e119ecf1867af4944ddc2f7633aacd";
|
||||
}
|
||||
|
|
@ -1,5 +0,0 @@
|
|||
{ pkgs ? import (fetchGit (import ./pkgs.nix)) {}
|
||||
, compiler ? "ghc822"
|
||||
}:
|
||||
|
||||
pkgs.haskell.packages.${compiler}.callPackage ./shell.nix {}
|
||||
13
shell.nix
|
|
@ -1,13 +0,0 @@
|
|||
{ pkgs ? import (fetchGit (import ./pkgs.nix)) {}
|
||||
, compiler ? "ghc822"
|
||||
}:
|
||||
|
||||
pkgs.haskell.packages.${compiler}.developPackage {
|
||||
root = ./.;
|
||||
name = "brittany";
|
||||
overrides = with pkgs.haskell.lib; self: super: {
|
||||
};
|
||||
source-overrides = {
|
||||
ghc-exactprint = "0.5.8.0";
|
||||
};
|
||||
}
|
||||
Should this have
name = "brittany", or does it not matter?Should this have
name = "brittany", or does it not matter?It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.
It doesn't really matter, in this case it'll use the enclosing directory name. However it should be "brittany" to make sure the same derivation is generated regardless of the directory name. Fixed.