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

Test start script on MacOS CI #1509

Merged
merged 3 commits into from
Mar 16, 2021
Merged

Test start script on MacOS CI #1509

merged 3 commits into from
Mar 16, 2021

Conversation

aherrmann
Copy link
Member

Closes #1475

Fixes and re-enables the start script tests on both nixpkgs and bindist MacOS CI pipelines.

  • Adds -Won-implicit-function-declaration to zlib.dev in bindist mode. The warning causes build errors on MacOS due to -Werror.
  • Updates the nixpkgs revision from 19.03 to 20.09 to avoid GHC build issues on MacOS (Start script doesn't work with --use-nix #1475). Also updates the GHC version accordingly.

Disables the corresponding warning

```
external/zlib.dev/gzlib.c:252:9: error: implicit declaration of function 'lseek' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
        LSEEK(state->fd, 0, SEEK_END);  /* so gzoffset() is correct */
        ^
```
Older versions don't work on MacOS

```
building '/nix/store/iyr224fd0npizqh3mvckpxjdzc0qr45h-pytest-hook.drv'...
building '/nix/store/q325bf3n7y33nszmsn9pgcnxbp3dpava-python3-3.7.3.drv'...
unpacking sources
unpacking source archive /nix/store/r5ik1sbafp0kr3a7prnaca1lmmqmb0b6-Python-3.7.3.tar.xz
source root is Python-3.7.3
setting SOURCE_DATE_EPOCH to timestamp 1553547549 of file Python-3.7.3/Misc/NEWS
patching sources
applying patch /nix/store/gd8ycd6j3qhcq1qccaqz68zdq5l4c2wf-no-ldconfig.patch
patching file Lib/ctypes/util.py
applying patch /nix/store/ggk7z3dvydrkypxff7d7c6v1slvbriin-darwin-libutil.patch
patching file Modules/posixmodule.c
Hunk #1 succeeded at 5907 (offset 27 lines).
applying patch /nix/store/9kwzs3pplms8sijf55sdryypzvic4x1s-python-3.x-distutils-C++.patch
patching file Lib/_osx_support.py
patching file Lib/distutils/cygwinccompiler.py
patching file Lib/distutils/sysconfig.py
Hunk #1 succeeded at 183 (offset 13 lines).
Hunk #2 succeeded at 202 (offset 13 lines).
Hunk #3 succeeded at 231 (offset 13 lines).
patching file Lib/distutils/unixccompiler.py
Hunk #3 FAILED at 183.
1 out of 3 hunks FAILED -- saving rejects to file Lib/distutils/unixccompiler.py.rej
patching file Makefile.pre.in
Hunk #1 succeeded at 615 (offset 31 lines).
builder for '/nix/store/q325bf3n7y33nszmsn9pgcnxbp3dpava-python3-3.7.3.drv' failed with exit code 1
cannot build derivation '/nix/store/26rq8gd6dh6vzn9rzz56ljqdr1sq90wi-ghc-8.6.4.drv': 1 dependencies couldn't be built
error: build of '/nix/store/26rq8gd6dh6vzn9rzz56ljqdr1sq90wi-ghc-8.6.4.drv' failed
```
@aherrmann aherrmann added the merge-queue merge on green CI label Mar 12, 2021
@aherrmann
Copy link
Member Author

Mergify:

Pull request must be merged manually.

@aherrmann aherrmann merged commit 998e23d into master Mar 16, 2021
@aherrmann aherrmann deleted the start-darwin branch March 16, 2021 09:16
@mergify mergify bot removed the merge-queue merge on green CI label Mar 16, 2021
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.

Start script doesn't work with --use-nix
2 participants