Messages in this thread | | | Date | Fri, 22 Nov 2013 09:01:05 -0500 | Subject | Re: [GIT] Security subsystem updates for 3.13 | From | Josh Boyer <> |
| |
On Thu, Nov 21, 2013 at 11:22 PM, Linus Torvalds <torvalds@linux-foundation.org> wrote: > In short: this is exactly the kind of thing I *don't* enjoy merging, > because the code that needed review was > > - mixed up with other changes that had nothing to do with it > - had no pointers to help me review it > - had zero information about others who had possibly reviewed it before > > and quite frankly, without the extended explanation of what the > changes were for (which I also didn't get initially), I'd probably > have decided half-way that it's not worth the headache. > > In the end, the code didn't look bad, and I didn't find any obvious > problems, but there's very much a reason why I merged this over two > weeks after the first pull request was originally sent to me. > > So guys, make it easier for me to merge these kinds of things, and > *don't* do what happened this time. Ok? Pretty please?
If it makes you feel any more confident, we've been carrying the keys code during the development of Fedora 20 (and rawhide) as patches. They were needed to solve some issues some of the userspace people were seeing that David mentioned in his explanation. The issues that have cropped up were all fixed in the various follow up commits included in the pull and we haven't hit any others that I'm aware of.
josh
| |