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

Build s390x-unknown-linux-musl target #62

Merged
merged 3 commits into from
Sep 20, 2022
Merged

Build s390x-unknown-linux-musl target #62

merged 3 commits into from
Sep 20, 2022

Conversation

messense
Copy link
Member

@messense messense commented Sep 19, 2022

bors: amd64 --target s390x-unknown-linux-musl

bors bot added a commit that referenced this pull request Sep 19, 2022
@messense
Copy link
Member Author

bors try amd64 --target s390x-unknown-linux-musl

bors bot added a commit that referenced this pull request Sep 19, 2022
@bors
Copy link
Contributor

bors bot commented Sep 19, 2022

try

Build failed:

@messense
Copy link
Member Author

bors retry

bors bot added a commit that referenced this pull request Sep 19, 2022
@bors
Copy link
Contributor

bors bot commented Sep 19, 2022

try

Build failed:

@messense
Copy link
Member Author

bors retry

bors bot added a commit that referenced this pull request Sep 19, 2022
@bors
Copy link
Contributor

bors bot commented Sep 19, 2022

try

Build failed:

@messense
Copy link
Member Author

bors retry

bors bot added a commit that referenced this pull request Sep 19, 2022
@bors
Copy link
Contributor

bors bot commented Sep 19, 2022

try

Build failed:

@messense
Copy link
Member Author

  = note: /usr/local/musl/bin/../lib/gcc/s390x-unknown-linux-musl/9.2.0/../../../../s390x-unknown-linux-musl/bin/ld: /home/rust/src/target/s390x-unknown-linux-musl/debug/deps/libopenssl_sys-70172c64409e3e90.rlib(curve448.o): in function `recode_wnaf':
          /home/rust/src/target/s390x-unknown-linux-musl/debug/build/openssl-sys-1de3ca3326f5bf22/out/openssl-build/build/src/crypto/ec/curve448/curve448.c:575: undefined reference to `__ctzdi2'
          /usr/local/musl/bin/../lib/gcc/s390x-unknown-linux-musl/9.2.0/../../../../s390x-unknown-linux-musl/bin/ld: read-only segment has dynamic relocations
          collect2: error: ld returned 1 exit status

Hmm, not sure what's going on.

@messense
Copy link
Member Author

bors retry

bors bot added a commit that referenced this pull request Sep 19, 2022
@bors
Copy link
Contributor

bors bot commented Sep 19, 2022

try

Build failed:

@messense
Copy link
Member Author

bors try arm64 armv7 --target s390x-unknown-linux-musl

bors bot added a commit that referenced this pull request Sep 19, 2022
```
  = note: /usr/local/musl/bin/../lib/gcc/s390x-unknown-linux-musl/9.2.0/../../../../s390x-unknown-linux-musl/bin/ld: /home/rust/src/target/s390x-unknown-linux-musl/debug/deps/libopenssl_sys-70172c64409e3e90.rlib(curve448.o): in function `recode_wnaf':
          /home/rust/src/target/s390x-unknown-linux-musl/debug/build/openssl-sys-1de3ca3326f5bf22/out/openssl-build/build/src/crypto/ec/curve448/curve448.c:575: undefined reference to `__ctzdi2'
          /usr/local/musl/bin/../lib/gcc/s390x-unknown-linux-musl/9.2.0/../../../../s390x-unknown-linux-musl/bin/ld: read-only segment has dynamic relocations
          collect2: error: ld returned 1 exit status
```
@messense
Copy link
Member Author

bors r+

bors bot added a commit that referenced this pull request Sep 19, 2022
62: Build s390x-unknown-linux-musl target r=messense a=messense

bors: arm64 armv7 --target s390x-unknown-linux-musl

Co-authored-by: messense <[email protected]>
@messense messense marked this pull request as ready for review September 19, 2022 04:39
@bors
Copy link
Contributor

bors bot commented Sep 19, 2022

Build failed:

@bors
Copy link
Contributor

bors bot commented Sep 20, 2022

try

Build failed:

@messense
Copy link
Member Author

bors retry

bors bot added a commit that referenced this pull request Sep 20, 2022
@bors
Copy link
Contributor

bors bot commented Sep 20, 2022

try

Build failed:

@messense
Copy link
Member Author

bors retry

bors bot added a commit that referenced this pull request Sep 20, 2022
@bors
Copy link
Contributor

bors bot commented Sep 20, 2022

try

Build failed:

@messense
Copy link
Member Author

bors retry

bors bot added a commit that referenced this pull request Sep 20, 2022
@bors
Copy link
Contributor

bors bot commented Sep 20, 2022

try

Build failed:

@messense
Copy link
Member Author

bors try armv7 --target s390x-unknown-linux-musl

@messense
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Sep 20, 2022

try

Already running a review

bors bot added a commit that referenced this pull request Sep 20, 2022
@bors
Copy link
Contributor

bors bot commented Sep 20, 2022

try

Build succeeded:

@bors
Copy link
Contributor

bors bot commented Sep 20, 2022

Build succeeded:

@bors bors bot merged commit af0c832 into main Sep 20, 2022
@bors bors bot deleted the s390x branch September 20, 2022 03:31
@paulwratt
Copy link

paulwratt commented Sep 20, 2022

I just checked that last conclusion, in the build script (step 2 of 3) the build docker sub-step has this at line 120. is it normal?

#0 0.093 fatal: Not a valid object name af0c8329a3c6a17b5f2f0730899b9ecba2cf69ff^{commit}

https://github.com/messense/rust-musl-cross/actions/runs/3087155053/jobs/4992234259#step:8:122


There seems to be an issue that generates warnings in the GCC package starting here:

https://github.com/messense/rust-musl-cross/actions/runs/3087155053/jobs/4992234259#step:8:353

EDIT: seems to be various linux.h doing so for each *.c file that includes ./tm.h - In fact that appears to be the only reason the log file is sooooo huuuuuge, as something then re-dumps those in a different format, starting here

https://github.com/messense/rust-musl-cross/actions/runs/3087155053/jobs/4992234259#step:8:822

@messense
Copy link
Member Author

messense commented Sep 20, 2022

I just checked that last conclusion, in the build script (step 2 of 3) the build docker sub-step has this at line 120. is it normal?

https://stackoverflow.com/a/72091309

Trying it out in e57aa65

@paulwratt
Copy link

paulwratt commented Sep 20, 2022

ok, and I just checked which commit it was (and presumed rightly) it was the (last?) br-bot (bors) commit

@paulwratt
Copy link

paulwratt commented Sep 20, 2022

also it seems someone invented "string format police" (the other reason the log is so long).

is there any way to get rid on -Wformat-diag before you compile

it sounds like those header warnings are because there is not #include once or #pragma once in there somewhere

( I realize these are GCC build issues, might not even be part of the musl-cross-make )

EDIT I just posted on musl-cross-make so the warnings might disappear - that should result in a slight speed up too - it depends tho, because any fixes on their end wont affect that 0.9.9 archive (which I see you just updated to current)

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