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.

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

Targets outside of root dir don't build #366

Closed
chrisdone opened this issue Jun 21, 2015 · 1 comment
Closed

Targets outside of root dir don't build #366

chrisdone opened this issue Jun 21, 2015 · 1 comment
Assignees
Labels
Milestone

Comments

@chrisdone
Copy link
Member

@chrisdone chrisdone commented Jun 21, 2015

Noticed the other day in passing, reporting before I forget:

Expected

stack build to build all targets, including those outside of the directory hierarchy where stack.yaml resides.

Actual

Only builds them when they are dependencies or specifically asked for. I think this is a bug from my original code in fpbuild.

chris@retina:~$ mkdir new-template
chris@retina:~$ cd new-template/
chris@retina:~/new-template$ stack new
[snip]
Wrote project config to: /home/chris/new-template/stack.yaml
chris@retina:~/new-template$ mkdir ../other-dir
chris@retina:~/new-template$ cd ../other-dir/
chris@retina:~/other-dir$ mv ../new-template/stack.yaml .
chris@retina:~/other-dir$ cat stack.yaml
flags: {}
packages:
- ../new-template
extra-deps: []
resolver: lts-2.14
chris@retina:~/other-dir$ stack build
chris@retina:~/other-dir$ stack build ../new-template/
new-template-0.1.0.0: configure
Configuring new-template-0.1.0.0...
[snip]
chris@retina:~/other-dir$ 
@snoyberg
Copy link
Contributor

@snoyberg snoyberg commented Jul 7, 2015

I'll take this one, I think I know how to solve it easily.

@snoyberg snoyberg added the ready label Jul 7, 2015
@snoyberg snoyberg closed this in 8fc9378 Jul 7, 2015
@snoyberg snoyberg removed the ready label Jul 7, 2015
dysinger added a commit that referenced this issue Jul 8, 2015
* master:
  #279 Add retries (default 3) to verifiedDownload
  Upper limit on number of flag combinations to test #543
  Flip order of build and test/bench opts parser
  Move `boptsCoverage` and `boptsNoTests` to `TestOpts`
  Replace `boptsAdditionalArgs` with fields in `FinalAction`
  Add benchmark and test options to FinalAction
  Implement `stack bench --benchmark-arguments`
  Fix #537
  Targets outside of root dir don't build (fixes #366)
  Add changelog for #517
  Add --no-run-tests parameter to stack test.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants