Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Script command #2805 #2992

Merged
merged 8 commits into from
Feb 16, 2017
Merged

Script command #2805 #2992

merged 8 commits into from
Feb 16, 2017

Commits on Feb 16, 2017

  1. Script command #2805 

    This adds a basic script command. Most of the work involved has to do
with preventing config files from being loaded. Originally we planned on
creating an alternative set of config types for with and without a local
config. That turned out to be prohibitively invasive. Instead, we now
just create a dummy config file instead ~/.stack/script/lts-x.y (or
/nightly-...).

While this addresses reproducibility, it doesn't help with the speed
concerns: script is now about 100ms faster than runghc on my system for
the case where --package is provided, but it's still over a second for
Hello World. The slowdown is inherent right now in checking if the
relevant packages are installed. It would be nice to figure out a way to
optimize the package check.

Also, this should include some integration tests. It should be a simple
matter of a test that includes a bogus stack.yaml and proving that stack
script ignores it.
    @snoyberg
    snoyberg committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    4984a5a View commit details
    Browse the repository at this point in the history

  2. Support the --compile and --optimize options for script command

    @snoyberg
    snoyberg committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    8ed3250 View commit details
    Browse the repository at this point in the history

  3. Remove the NoLocal config type split 

    This was originally done to help implementation of #2805, but we ended
up going a different route.
    @snoyberg
    snoyberg committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    32143ad View commit details
    Browse the repository at this point in the history

  4. Move scriptCmd to new Stack.Script module

    @snoyberg
    snoyberg committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    df44688 View commit details
    Browse the repository at this point in the history

  5. Experimental: parse package list based on imports in scripts #2805

    @snoyberg
    snoyberg committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    29df0c4 View commit details
    Browse the repository at this point in the history

  6. Better error output when script compile fails

    @snoyberg
    snoyberg committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    462fbfe View commit details
    Browse the repository at this point in the history

  7. Switch to logError #3007

    @snoyberg
    snoyberg committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    a8e53fd View commit details
    Browse the repository at this point in the history

  8. hlint suggestions

    @snoyberg
    snoyberg committed Feb 16, 2017
    Configuration menu
    Copy the full SHA
    011e4da View commit details
    Browse the repository at this point in the history