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

[R] [CI] Windows build fails in packaging job #15110

Closed
jonkeane opened this issue Dec 29, 2022 · 3 comments · Fixed by #15111
Closed

[R] [CI] Windows build fails in packaging job #15110

jonkeane opened this issue Dec 29, 2022 · 3 comments · Fixed by #15111

Comments

@jonkeane
Copy link
Member

Describe the bug, including details regarding any error messages, version, and platform.

One example: https://github.com/ursacomputing/crossbow/actions/runs/3799348531/jobs/6462070137

C:\rtools42\x86_64-w64-mingw32.static.posix\bin/ld.exe: ../windows/arrow-10.0.1.100000336/lib/x64-ucrt/libcurl.a(libcurl_la-version.o):(.text+0x1bc): undefined reference to `nghttp2_version'
collect2.exe: error: ld returned 1 exit status
no DLL was created
ERROR: compilation failed for package 'arrow'
* removing 'D:/a/_temp/Library/arrow'
Warning message:
In install.packages("arrow", type = "source", repos = sub("file://",  :
  installation of package 'arrow' had non-zero exit status

I suspect this has to do with #14235 (and I'll try to fix it in a PR in a sec)

Component(s)

Continuous Integration, R

@jonkeane
Copy link
Member Author

Though a counter-argument that this has to do with #14235 is that this is a new failure as of today (and that PR has been merged for a few weeks already!)

@jonkeane
Copy link
Member Author

And interestingly(?) even with the patch in #15111 nghttp2 was installed by mngw in the first failing build:

https://github.com/ursacomputing/crossbow/actions/runs/3799348531/jobs/6461793749#step:7:185

==> Installing missing dependencies...
resolving dependencies...
looking for conflicting packages...

Packages (13) mingw-w64-i686-boost-1.67.0-9002  mingw-w64-i686-libssh2-1.10.1-9800  mingw-w64-i686-nghttp2-1.51.0-1  mingw-w64-i686-openssl-1.1.1.m-9800  mingw-w64-i686-aws-sdk-cpp-1.7.365-1  mingw-w64-i686-brotli-1.0.9-4  mingw-w64-i686-curl-7.84.0-9000  mingw-w64-i686-libutf8proc-2.4.0-2  mingw-w64-i686-lz4-1.8.2-1  mingw-w64-i686-re2-20200801-1  mingw-w64-i686-snappy-1.1.7-2  mingw-w64-i686-thrift-0.13.0-1  mingw-w64-i686-zstd-1.4.4-1

Total Download Size:    26.87 MiB
Total Installed Size:  350.90 MiB

Though it's not in the last passing build:

https://github.com/ursacomputing/crossbow/actions/runs/3792761794/jobs/6449247350#step:7:185

==> Installing missing dependencies...
resolving dependencies...
looking for conflicting packages...

Packages (12) mingw-w64-i686-boost-1.67.0-9002  mingw-w64-i686-libssh2-1.10.1-9800  mingw-w64-i686-openssl-1.1.1.m-9800  mingw-w64-i686-aws-sdk-cpp-1.7.365-1  mingw-w64-i686-brotli-1.0.9-4  mingw-w64-i686-curl-7.83.1-9000  mingw-w64-i686-libutf8proc-2.4.0-2  mingw-w64-i686-lz4-1.8.2-1  mingw-w64-i686-re2-20200801-1  mingw-w64-i686-snappy-1.1.7-2  mingw-w64-i686-thrift-0.13.0-1  mingw-w64-i686-zstd-1.4.4-1

Total Download Size:    26.70 MiB
Total Installed Size:  350.17 MiB

@jonkeane
Copy link
Member Author

Also of note: mingw went from curl 7.83.1-9000 to 7.84.0-9000.

assignUser pushed a commit that referenced this issue Dec 29, 2022
Add new dependencies to windows builds following #14235
* Closes: #15110

Authored-by: Jonathan Keane <jkeane@gmail.com>
Signed-off-by: Jacob Wujciak-Jens <jacob@wujciak.de>
@assignUser assignUser added this to the 11.0.0 milestone Dec 29, 2022
EpsilonPrime pushed a commit to EpsilonPrime/arrow that referenced this issue Jan 5, 2023
…#15111)

Add new dependencies to windows builds following apache#14235
* Closes: apache#15110

Authored-by: Jonathan Keane <jkeane@gmail.com>
Signed-off-by: Jacob Wujciak-Jens <jacob@wujciak.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants