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

Windows builds #1884

Closed
wants to merge 2 commits into from
Closed

Conversation

newhoggy
Copy link
Contributor

No description provided.

- uses: actions/checkout@v1

- name: Install pkgconfiglite
if: matrix.os == 'windows-latest'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ach, that's the trick to run a step on a given platform, I was missing it somewhere else :)

run: cabal update

- name: Select build directory
run: echo "::set-env name=CABAL_BUILDDIR::dist"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this pick the current directory? On windows I had problems with length of paths names, so I always build somewhere very close to the root, e.g. C://d.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So far this has been working. Shortening the dist-newstyle to dist was just enough to squeeze in under the path name limit.

@newhoggy newhoggy force-pushed the windows-builds branch 2 times, most recently from 4816378 to 0d53ec0 Compare October 1, 2020 04:44
@newhoggy newhoggy closed this Oct 13, 2020
@newhoggy newhoggy deleted the windows-builds branch October 29, 2020 04:04
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

Successfully merging this pull request may close these issues.

2 participants