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

[CI] musl build fails #158

Closed
azerupi opened this issue Aug 1, 2016 · 10 comments
Closed

[CI] musl build fails #158

azerupi opened this issue Aug 1, 2016 · 10 comments
Labels
A-Infrastructure Area: CI, Releases E-Easy Experience: Easy

Comments

@azerupi
Copy link
Contributor

azerupi commented Aug 1, 2016

@japaric hey, sorry to annoy you again with this 😕

Recently my musl build started failing and I am not sure what is going on. The travis log says:

error: linking with `cc` failed: exit code: 1

And a little further

note: /usr/bin/ld: BFD (GNU Binutils for Ubuntu) 2.22 internal error, aborting at ../../bfd/reloc.c line 443 in bfd_get_reloc_size
/usr/bin/ld: Please report this bug.

collect2: ld returned 1 exit status

error: aborting due to previous error
error: Could not compile `mdbook`.

Do you have an idea on how I might solve this?

@japaric
Copy link
Member

japaric commented Aug 1, 2016

This is probably rust-lang/rust#34978. I'd suggest to just disable (allowed_failures) the musl target on nightly until that's fixed. The other solution is to upgrade the binutils in the Travis builder, which is not trivial.

@azerupi
Copy link
Contributor Author

azerupi commented Aug 1, 2016

Thanks! It seems to be related to that issue indeed

I'd suggest to just disable (allowed_failures) the musl target on nightly until that's fixed.

It's already affecting beta too, so I guess I will allow failure for all channels for the musl build until it gets fixed.

Thanks for your help :)

@code-ape
Copy link
Contributor

code-ape commented Aug 5, 2016

Is there a published artifact for the musl build of mdBook? I've been building my own to run on alpine Docker containers and just saw this in passing.

@azerupi
Copy link
Contributor Author

azerupi commented Aug 5, 2016

There is, but not for the latest release due to this bug.
You can find them under the releases section: https://github.com/azerupi/mdBook/releases

@code-ape
Copy link
Contributor

code-ape commented Aug 5, 2016

Oh, ok! Thanks. I just thought that there may have been some other reason since the stable build for musl succeeded in the build you referenced and after that build Travis CI doesn't appeal to run the musl builds at all any more.

@azerupi
Copy link
Contributor Author

azerupi commented Aug 5, 2016

Since the bug was also present in beta, I anticipated that it would propagate to stable unless it was fixed before the next release. So I just allowed all musl builds to fail.

Travis CI doesn't appeal to run the musl builds at all any more.

I noticed that too, but I am not sure why... I don't have a ton of experience with Travis and it's possible that I changed something that I shouldn't have. The behavior I wanted was to run the musl builds but not complain if they failed.

@japaric
Copy link
Member

japaric commented Aug 5, 2016

I noticed that too, but I am not sure why... I don't have a ton of experience with Travis and it's possible that I changed something that I shouldn't have. The behavior I wanted was to run the musl builds but not complain if they failed.

@azerupi

I just looked at azerupi@02c5c97 and you should have not removed any line from .travis.yml to get the behavior you describe (IOW, only adding the build jobs to allowed_failures was necessary)

@azerupi
Copy link
Contributor Author

azerupi commented Aug 5, 2016

Ah, I see. It all makes sense now.

Thanks a lot for taking a look! :)

@lilianmoraru
Copy link

This should work again(at least on nightly).

@azerupi
Copy link
Contributor Author

azerupi commented May 16, 2017

This is probably fixed since a long time. I think, if it is not already done, we can re-add the musl builds to fail CI if they error. Once that's done and CI is still green we can close this issue.

@azerupi azerupi added the E-Easy Experience: Easy label May 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Infrastructure Area: CI, Releases E-Easy Experience: Easy
Projects
None yet
Development

No branches or pull requests

4 participants