-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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: update to 21.1.7. #32483
mesa: update to 21.1.7. #32483
Conversation
Woops i forgot to fix the revision |
1974292
to
f30dcac
Compare
srcpkgs/mesa/template
Outdated
version=21.1.6 | ||
revision=2 | ||
version=21.1.7 | ||
revision=1 | ||
wrksrc="mesa-${version}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's drop this line, wrksrc="$pkgname-$version"
is the default, so we don't have to overwrite it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, this is vestigial from when the package was named libGL
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll get to this in the morning but just to clarify you are recommending removing the wrksrc line entirely? Since the default behavior makes it unnecessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we don't need to specify variables that are unnecessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay it's spitting out new errors haha
So i'm not sure why that failed to fetch the archive for the i686 build because it fetches fine on my systems. |
f30dcac
to
fce320b
Compare
|
Sorry I was out at the time. I intended to do this this morning. Looks like it already merged? |
Yup, no worries ;) |
General
Have the results of the proposed changes been tested?