Skip to content

evdi: v1.7.0 -> v1.8.0 #102873

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

evdi: v1.7.0 -> v1.8.0 #102873

wants to merge 1 commit into from
+3 −3

Conversation

samsartor
Copy link

Motivation for this change

Using Linux 5.17 and evdi 1.7 resulted in compilation errors.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
    • i loaded the resulting kernel module if that counts.
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

Sorry, something went wrong.

@ofborg ofborg bot requested a review from eyJhb November 5, 2020 08:00
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 11-100 labels Nov 5, 2020
@eyJhb
Copy link
Member

eyJhb commented Nov 5, 2020

I am tempted to wait for a 1.9.0 release, where they have support for linux kernel 5.9.
But I will test this once I get home, and see if everything works with my dock :)

@eyJhb
Copy link
Member

eyJhb commented Nov 5, 2020

The update breaks DisplayLink. We need to either have two versions of evdi, or wait for a new release... :)

@samsartor samsartor closed this Nov 6, 2020
@eyJhb
Copy link
Member

eyJhb commented Nov 6, 2020

@samsartor there is no need to close it as such, as it can stay open to remind us to check every once in a while :) But completely up to you, to do so :)

@Janik-Haag Janik-Haag added the 12. first-time contribution This PR is the author's first one; please be gentle! label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eyJhb eyJhb

Assignees
No one assigned
Labels
10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 11-100 12. first-time contribution This PR is the author's first one; please be gentle!
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@samsartor @eyJhb @Janik-Haag