stack equivalent of cabal check?

[Blaisorblade]

I might be missing something, but stack check fails and I can't find an equivalent syntax (stack lint fails too). This came up in #2500.

[sjakobi]

stack sdist and stack upload include executing checks defined in the Cabal library which, I assume, should be similar to what cabal check does.

If necessary, it shouldn't be difficult to make that functionality available through a separate command.

[Blaisorblade]

I haven't checked whether the code is in Cabal or cabal-install, but after the setup in #2500, both cabal check and cabal sdist have better output than stack sdist.

Specifically, stack sdist only says that LICENSE can't be found, not that it's mentioned in license-file (which was genuinely non-obvious in #2500), nor any of the extra info from cabal-install.

$ cabal check
The following warnings are likely affect your build negatively:
* The 'license-file' field refers to the file 'LICENSE' which does not exist.

Hackage would reject this package.
$ cabal sdist
Distribution quality errors:
The 'license-file' field refers to the file 'LICENSE' which does not exist.
Note: the public hackage server would reject this package.
Warning: Cannot run preprocessors. Run 'configure' command first.
Building source dist for foo-0.1.0.0...
LICENSE: copyFile: does not exist (No such file or directory)
$ stack sdist
Getting file list for /Users/pgiarrusso/AeroFS/Repos/stack-tatooine/foo/
Building sdist tarball for /Users/pgiarrusso/AeroFS/Repos/stack-tatooine/foo/
/Users/pgiarrusso/AeroFS/Repos/stack-tatooine/foo/LICENSE: getFileStatus: does not exist (No such file or directory)

[Blaisorblade]

By chance I found this old ticket on the same idea:
#285 (comment)

[sjakobi]

I have opened #2507 to track improving the stack sdist output.

By chance I found this old ticket on the same idea:
#285 (comment)

See my comment there.

[agrafix]

An alternative would be to add a --dry-runoder --dry flag to stack upload that does everything but the actual upload?

[mgsloan]

There's now an error message mentioning to use "cabal check", that should be changed once this is implemented - #3355