-
-
Notifications
You must be signed in to change notification settings - Fork 15.5k
zfs_unstable: 0.8.0-rc3 -> 2019-03-29 #58553
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
Conversation
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 don't feel good about getting off release candidates and applying patches from funny domains for TRIM
Feel free to host the updated patch file yourself. I fail to see why you call it a "funny" domain. Also, afaik, TRIM has always been planned to be included into 0.8.0. Since it's zfs unstable I see no difference to go to latest git master. |
isUnstable = true; | ||
|
||
extraPatches = [ | ||
# in case this gets out of date, just send Mic92 a pull request! |
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.
Please do that instead of deleting the comment.
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.
On a related note, would you be interested in upstreaming this a version of this patch? We have an opportunity to add NixOS to the ZFSoL CI / build farm, and I think not requiring patches would be a step in that direction.
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.
Please do that instead of deleting the comment.
This just got deleted by accident.
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.
@grahamc sounds good to me. Do you have some contacts in that direction?
I guess there's no interest. |
I was interested. The trim patch was merged to master, was it not? Could we not just point to master instead of hosting patches? @grahamc I do like being on a release candidate, but once 0.8 is released, zfsUnstable will have nowhere to point but master, right? It is a little odd to me that zfsUnstable currently points to a version that's over two months old; I think monthly updates would be considered normal for the unstable moniker. |
Motivation for this change
Adding ssd trim patch to zfs unstable: openzfs/zfs#8419
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)