[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qemu: Broken -smb with latest SAMBA package. (U
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH] qemu: Broken -smb with latest SAMBA package. (Unsupported security=share option) |
Date: |
Mon, 04 Nov 2013 15:04:32 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2013-11-04 14:55, Michael Tokarev wrote:
> 04.11.2013 00:06, Jan Kiszka wrote:
>> On 2013-11-01 11:10, Michael Tokarev wrote:
> []
>>> If Jan picks it up, that's fine. If not, I think it can go
>>> to the trivial patches queue.
>>
>> Works fine, applied to queues/slirp.
>
> Okay, thank you Jan.
>
>> But this is not a trivial patch as the fix is not obvious for a reader
>> (unless you know smb.conf semantics by heart).
>
> It's trivial for my understanding. If we require that every
> change going to -trivial should be obvious to everyone, we
> should just close it right away.
Then we may need -less-trivial, because - to my understanding - -trivial
was once set up according to the rule that (most) QEMU hackers should be
able to understand that a trivial change is at least mostly harmless.
> And this area does not have an active maintainer anyway, at least
> according to MAINTAINERS and ./scripts/get_maintainer.pl.
Yeah, I think we have some holes there. That smb configuration
conceptually belongs to slirp is right, just not clear documented in our
script.
Jan
signature.asc
Description: OpenPGP digital signature