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

Sqlite error on master after fd59ca3 #2824

Closed
enolan opened this issue Dec 3, 2016 · 1 comment
Closed

Sqlite error on master after fd59ca3 #2824

enolan opened this issue Dec 3, 2016 · 1 comment

Comments

@enolan
Copy link
Contributor

enolan commented Dec 3, 2016

Versions of stack after fd59ca3 (found via bisect) give this error when trying to run stack build with Docker support turned on:

Database/Persist/Sqlite.hs:(103,12)-(118,9): Missing field in record construction connUpsertSql

That commits upgrades persistent and persistent-template to 2.6 and 2.5.1.6 respectively. Maybe it's a bug in the code generation? Database.Persist.Sqlite is part of persistent-sqlite.

Steps to reproduce

For example:

  1. Checkout current master. As of writing it's 737ee75. Anything between that and fd59ca3 will also likely fail.
  2. stack new foo simple
  3. Add docker: {enable: true} to stack.yaml
  4. stack build

Expected

Project builds successfully.

Actual

Error message quoted above. Here's the output of stack build -v:

$ /home/enolan/mystuff/code/stack/.stack-work/install/x86_64-linux/lts-6.25/7.10.3/bin/stack build -v
Version 1.3.1, Git revision 737ee751654cba4b2547b50628c9a5a78ddc6f8b (4364 commits) x86_64 hpack-0.15.0
2016-12-02 18:59:49.348992: [debug] Checking for project config at: /home/enolan/junk/stack-simple-template/stack.yaml
@(Stack/Config.hs:863:9)
2016-12-02 18:59:49.349326: [debug] Loading project config file stack.yaml
@(Stack/Config.hs:881:13)
2016-12-02 18:59:49.352096: [debug] Run process: /usr/bin/docker --version
@(System/Process/Read.hs:306:3)
2016-12-02 18:59:49.362680: [debug] Process finished in 10ms: /usr/bin/docker --version
@(System/Process/Read.hs:306:3)
2016-12-02 18:59:49.363090: [debug] Run process: /usr/bin/docker inspect fpco/stack-build:lts-7.10
@(System/Process/Read.hs:306:3)
2016-12-02 18:59:49.377911: [debug] Process finished in 12ms: /usr/bin/docker inspect fpco/stack-build:lts-7.10
@(System/Process/Read.hs:306:3)
2016-12-02 18:59:49.381740: [debug] Run process: /usr/bin/docker run -v /home/enolan/mystuff/code/stack/.stack-work/install/x86_64-linux/lts-6.25/7.10.3/bin/stack:/tmp/stack sha256:b74d3f6b2adf6bbeb857e72588e51eee3dcfb3dd8f17aa9d95d9d06dcfff1ff5 /tmp/stack --version
@(System/Process/Read.hs:306:3)
2016-12-02 18:59:49.971777: [debug] Process finished in 589ms: /usr/bin/docker run -v /home/enolan/mystuff/code/stack/.stack-work/install/x86_64-linux/lts-6.25/7.10.3/bin/stack:/tmp/stack sha256:b74d3f6b2adf6bbeb857e72588e51eee3dcfb3dd8f17aa9d95d9d06dcfff1ff5 /tmp/stack --version
@(System/Process/Read.hs:306:3)
Database/Persist/Sqlite.hs:(103,12)-(118,9): Missing field in record construction connUpsertSql

Stack version

$ stack --version
Version 1.3.1, Git revision 737ee751654cba4b2547b50628c9a5a78ddc6f8b (4364 commits) x86_64 hpack-0.15.0

Method of installation

stack build in a Git checkout.

@mgsloan
Copy link
Contributor

mgsloan commented Dec 10, 2016

Thanks for catching this!

Pinging @borsboom , I think b37e2f4 and 164cb55 should be included in the upcoming release. Otherwise, docker support is broken

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants