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

skip builds on manual download for several packages #192463

Closed
wants to merge 1 commit into from
Closed

skip builds on manual download for several packages #192463

wants to merge 1 commit into from

Conversation

superherointj
Copy link
Contributor

@superherointj superherointj commented Sep 22, 2022

Goal is to avoid this (for all versions of every package on list):
image

Notice how many of these packages are listed in large builds:


Pending:

  • Confirm setting hydraPlatforms to [] has intended effect in nixpkgs-review results.

superherointj:

I seek advice on how to solve requireFile noise issue in nixpkgs-review results.
I naively thought that by setting meta.hydraPlatforms to [] for each package would solve issue, but I'm uncertain of that behavior. (if you could confirm that would be nice.)

Mic92:

That's a tricky one
Because it's not the derivation itself that fails but any dependency to that.
I think what could work is to mark requireFile by something as broken
So that eval would fail of this.
I think ofborg could even do this so it does not even show up in the eval cache


Skip builds (at hydra/nixpkgs-review) on manual download for:

  • cisco-packet-tracer
  • citrix-workspace
  • cov-build
  • everspace
  • fpga/lattice-diamond
  • iaca
  • ib-tws
  • javacard-devkit
  • kdbplus
  • kinect-audio-setup
  • koboredux
  • mathematica
  • oraclejdk
  • planetaryannihilation
  • sm64ex
  • snapdragon-profiler
  • sqldeveloper
  • tensorrt
  • ue4
  • vessel
  • wolfram-engine
  • worldofgoo
  • wtk

Pending:

  • airwave requires vstsdk368_08_11_2017_build_121.zip
  • ib-controller

Notes:

@superherointj superherointj marked this pull request as draft September 22, 2022 17:51
@superherointj superherointj marked this pull request as ready for review September 22, 2022 18:11
@ofborg ofborg bot added 8.has: clean-up 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux labels Sep 22, 2022
@superherointj superherointj marked this pull request as draft September 23, 2022 10:22
@superherointj superherointj requested a review from Mic92 September 23, 2022 10:28
@superherointj superherointj deleted the skip-builds-on-manual-download branch October 4, 2022 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: clean-up 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant