-
Notifications
You must be signed in to change notification settings - Fork 632
[DEVOPS-1113] Buildkite: Add Windows build #3880
Conversation
9d7c8bd
to
e947292
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Some minor suggestions.
Hmm... not so fast I guess:
|
copy log-configs\daedalus.yaml daedalus\log-config-prod.yaml | ||
copy lib\configuration.yaml daedalus\ | ||
copy lib\*genesis*.json daedalus\ | ||
copy cardano-launcher.exe daedalus\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are we in $env:WORK_DIR
?
Can we do copy $env:WORK_DIR\cardano-launcher.exe daedalus\
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes we are in $env:WORK_DIR
but I don't want to touch this bit because it works and produces a zip with the expected layout.
This is a squash of the changes on the buildkite/windows branch, without deleting appveyor.yml, without the setup script, and with ghc 8.2.2 instead of 8.4.4. Original authors: Jacob Mitchell <jacob.mitchell@iohk.io> Samuel Leathers <samuel.leathers@iohk.io> Rodney Lorrimar <rodney.lorrimar@iohk.io>
e947292
to
a312801
Compare
Also, are the ~6min buidkite/windows build seem suspicious. Shouldn't a full build take much much longer? |
Thanks - I force-pushed the review fixes. The build failed quickly in 6 minutes because I accidentally left a ghc-8.4.4 option on the build command line. |
Description
This is a squash of the changes in #3802, without deleting
appveyor.yml
, without the setup script, and with ghc 8.2.2 instead of 8.4.4.Linked issue
https://iohk.myjetbrains.com/youtrack/issue/DEVOPS-1113
https://iohk.myjetbrains.com/youtrack/issue/DEVOPS-1138
Type of change
QA Steps