Skip to content

Commit

Permalink
lua-packages/luasocket: Fix wrong platforms value
Browse files Browse the repository at this point in the history
This has surfaced due to 505d7be and
the fixup commit 77f5a50.

The individial platform attributes are a list rather than single
elements, so in this case we got:

with platforms; [darwin linux freebsd illumos]

Which results in:

[ ["x86_64-darwin"]
  ["i686-linux" "x86_64-linux" "armv5tel-linux" "armv6l-linux"
   "armv7l-linux" "aarch64-linux" "mips64el-linux"]
  ["i686-freebsd" "x86_64-freebsd"]
  ["x86_64-solaris"]
]

So if you don't have allowBroken set in nixpkgs config, you end up with
an evaluation error because meta.platforms is expected to be a list
instead of a list of lists which we got here.

Signed-off-by: aszlig <aszlig@redmoonstudios.org>
Cc: @mornfall, @vcunat
  • Loading branch information
aszlig committed Feb 19, 2017
1 parent 4b833fa commit 7c9f4ee
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkgs/top-level/lua-packages.nix
Original file line number Diff line number Diff line change
Expand Up @@ -239,7 +239,7 @@ let

meta = with stdenv.lib; {
homepage = "http://w3.impa.br/~diego/software/luasocket/";
platforms = with platforms; [darwin linux freebsd illumos];
platforms = with platforms; darwin ++ linux ++ freebsd ++ illumos;
maintainers = with maintainers; [ mornfall ];
};
};
Expand Down

1 comment on commit 7c9f4ee

@vcunat
Copy link
Member

@vcunat vcunat commented on 7c9f4ee Feb 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch!

Please sign in to comment.