Skip to content

libultrahdr: fix tests on big-endian - #533532

Closed
zwang20 wants to merge 1 commit into
NixOS:stagingfrom
zwang20:fix-libultrahdr-2
Closed

libultrahdr: fix tests on big-endian#533532
zwang20 wants to merge 1 commit into
NixOS:stagingfrom
zwang20:fix-libultrahdr-2

Conversation

@zwang20

@zwang20 zwang20 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

This pull request fixes tests on big-endian by incorporating the upstream commit in google/libultrahdr#396, as requested by @OPNA2608 on #533168

Things done

@OPNA2608

OPNA2608 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

The other PR is the only reason that this is even an issue, because there is not a single BE target in the allowed platforms before the CMake patch you want to pull in, and building on BE regardless by ignoring meta.platforms will get you rejected by CMake, so adding the commit to that PR would've made more sense to me tbh.

@zwang20
zwang20 changed the base branch from master to staging June 20, 2026 08:21
@nixpkgs-ci nixpkgs-ci Bot closed this Jun 20, 2026
@nixpkgs-ci nixpkgs-ci Bot reopened this Jun 20, 2026
nixpkgs-branch-check[bot]

This comment was marked as outdated.

@zwang20
zwang20 force-pushed the fix-libultrahdr-2 branch from cb60f8c to 3e9bff5 Compare June 20, 2026 08:23
@nixpkgs-branch-check
nixpkgs-branch-check Bot dismissed their stale review June 20, 2026 08:24

Review dismissed automatically

@zwang20

zwang20 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor Author

I think I will defer this decision to the package maintainer, @yzx9

@yzx9

yzx9 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Thanks for the ping. I would prefer to merge them to make testing easier for the reviewer.

@OPNA2608

Copy link
Copy Markdown
Contributor

7vvppp2h362pis9jxbaf26vajaq6c5dq-libultrahdr-1.4.0.drv.log

CMake Error at CMakeLists.txt:68 (message):
  Architecture: ppc64 not recognized

¯\_(ツ)_/¯

@nixpkgs-ci
nixpkgs-ci Bot requested a review from yzx9 June 20, 2026 08:33
@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-darwin: 101-500 This PR causes between 101 and 500 packages to rebuild on Darwin. 10.rebuild-linux: 501-1000 This PR causes many rebuilds on Linux and should normally target the staging branches. labels Jun 20, 2026
@zwang20

zwang20 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor Author

Closing in favor of #533168

@zwang20 zwang20 closed this Jun 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 101-500 This PR causes between 101 and 500 packages to rebuild on Darwin. 10.rebuild-linux: 501-1000 This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants