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

nixpkgs: build aarch64-linux #23638

Merged
merged 1 commit into from Mar 8, 2017

Conversation

Projects
None yet
8 participants
@grahamc
Member

grahamc commented Mar 8, 2017

cc @dezgeg @edolstra

I figure we'll start with nixpkgs and expand to NixOS afterwards.

@mention-bot

This comment has been minimized.

Show comment
Hide comment
@mention-bot

mention-bot Mar 8, 2017

@grahamc, thanks for your PR! By analyzing the history of the files in this pull request, we identified @edolstra, @domenkozar and @peti to be potential reviewers.

@grahamc, thanks for your PR! By analyzing the history of the files in this pull request, we identified @edolstra, @domenkozar and @peti to be potential reviewers.

@fpletz

This comment has been minimized.

Show comment
Hide comment
@fpletz

fpletz Mar 8, 2017

Member

We also need a build machine for hydra if we merge this PR or those builds will stay in the queue. Can we get this done simultaneously?

Member

fpletz commented Mar 8, 2017

We also need a build machine for hydra if we merge this PR or those builds will stay in the queue. Can we get this done simultaneously?

@grahamc

This comment has been minimized.

Show comment
Hide comment
@fpletz

This comment has been minimized.

Show comment
Hide comment
@fpletz

fpletz Mar 8, 2017

Member

Oh, nevermind. Thanks a lot. 👍

Member

fpletz commented Mar 8, 2017

Oh, nevermind. Thanks a lot. 👍

@c0bw3b

This comment has been minimized.

Show comment
Hide comment
@c0bw3b

c0bw3b Mar 8, 2017

Contributor

The beast won't be idle anymore!

Contributor

c0bw3b commented Mar 8, 2017

The beast won't be idle anymore!

@domenkozar

This comment has been minimized.

Show comment
Hide comment
@domenkozar

domenkozar Mar 8, 2017

Member

Looks good to me (code-wise). What's missing? :)

Member

domenkozar commented Mar 8, 2017

Looks good to me (code-wise). What's missing? :)

@edolstra edolstra merged commit 7a08a44 into NixOS:master Mar 8, 2017

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@edolstra

This comment has been minimized.

Show comment
Hide comment
@edolstra

edolstra Mar 8, 2017

Member

Hm, it's giving eval errors apparently for all aarch64 jobs:

in job ‘icecast.aarch64-linux’:
evaluation aborted with the following error message: ‘unsupported system type: aarch64-linux’
...

https://hydra.nixos.org/jobset/nixpkgs/trunk/#tabs-errors

This is odd because the patchelf jobset evaluated fine with an aarch64 job: http://hydra.nixos.org/build/49825197

Member

edolstra commented Mar 8, 2017

Hm, it's giving eval errors apparently for all aarch64 jobs:

in job ‘icecast.aarch64-linux’:
evaluation aborted with the following error message: ‘unsupported system type: aarch64-linux’
...

https://hydra.nixos.org/jobset/nixpkgs/trunk/#tabs-errors

This is odd because the patchelf jobset evaluated fine with an aarch64 job: http://hydra.nixos.org/build/49825197

@domenkozar

This comment has been minimized.

Show comment
Hide comment
Member

domenkozar commented Mar 8, 2017

fpletz added a commit that referenced this pull request Mar 8, 2017

@FRidh

This comment has been minimized.

Show comment
Hide comment
@FRidh

FRidh Mar 19, 2017

Member

This has been merged 11 days ago but one of the jobs, Thunderbird, hasn't actually ever been build successfully. It's nice having aarch64 support but I don't think aarch64 builds should block any of the channels. There's too few people having such systems / the possibility to fix issues.

Member

FRidh commented Mar 19, 2017

This has been merged 11 days ago but one of the jobs, Thunderbird, hasn't actually ever been build successfully. It's nice having aarch64 support but I don't think aarch64 builds should block any of the channels. There's too few people having such systems / the possibility to fix issues.

@grahamc

This comment has been minimized.

Show comment
Hide comment
@grahamc

grahamc Mar 20, 2017

Member
Member

grahamc commented Mar 20, 2017

@dezgeg

This comment has been minimized.

Show comment
Hide comment
@dezgeg

dezgeg Mar 20, 2017

Contributor

Done in 93446a2.

Contributor

dezgeg commented Mar 20, 2017

Done in 93446a2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment