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

mesa: 18.1.7 -> 18.3.1 [backport] #53218

Closed
wants to merge 2 commits into from

Conversation

rnhmjoj
Copy link
Contributor

@rnhmjoj rnhmjoj commented Jan 2, 2019

Motivation for this change

18.3.1 brings openGL 4.5 compatibility profile to the radeonsi and 4.3 to virgl besides many bugfixes.
I don't know if this is allowed to be backported since it's a major update and could potentially break stuff. Anyway, I have been running this on my system for a couple of weeks with no issues.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change
  • Tested compilation of all pkgs that depend on this change (mass rebuild)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@@ -89,29 +93,22 @@ let self = stdenv.mkDerivation {
./symlink-drivers.patch
./missing-includes.patch # dev_t needs sys/stat.h, time_t needs time.h, etc.-- fixes build w/musl
./disk_cache-include-dri-driver-path-in-cache-key.patch
];
] ++ lib.optional stdenv.isDarwin ./darwin-clock-gettime.patch;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The file is missing, which caused the ofborg eval failure.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ops, sorry.

@samueldr
Copy link
Member

Oops, this seems really dry as a review... Let's add some words.


Hi! Thanks for the backport, let's fix the basic issues and then look at whether it's right or not as a backport.

@vcunat
Copy link
Member

vcunat commented Jan 13, 2019

Hmm, right, it probably is a bit more risky than what we usually pick.

r-ryantm and others added 2 commits January 14, 2019 12:40
Semi-automatic update generated by
https://github.com/ryantm/nixpkgs-update tools. This update was made
based on information from
https://repology.org/metapackage/libdrm/versions

(cherry picked from commit fe667c9d63a1dbc82e4b58b64d7dc64b4e6410c5)
Reason: needed to update mesa
@vcunat
Copy link
Member

vcunat commented Jan 14, 2019

I'm thinking of the option to provide newer mesa drivers as a non-default option. (libdrm bump itself probably isn't risky)

@samueldr samueldr dismissed their stale review January 14, 2019 16:34

Dimissed since it now evals

@rnhmjoj
Copy link
Contributor Author

rnhmjoj commented Mar 26, 2019

I'm closing this since we now have mesa 18.3.4 in release-19.03.

@rnhmjoj rnhmjoj closed this Mar 26, 2019
@vcunat
Copy link
Member

vcunat commented Mar 26, 2019

Well, 18.09 will be alive for several weeks at least, but there's probably not enough motivation to get this inside...

@rnhmjoj rnhmjoj deleted the release-18.09 branch September 14, 2019 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants