[Qemu-devel] [PATCH] slirp: Add Samuel Thibault's staging tree for slirp

Samuel Thibault posted 1 patch 7 years, 8 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180531194843.21172-1-samuel.thibault@ens-lyon.org
Test checkpatch passed
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test s390x passed
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
[Qemu-devel] [PATCH] slirp: Add Samuel Thibault's staging tree for slirp
Posted by Samuel Thibault 7 years, 8 months ago
Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 41cd3736a9..8822ae9b70 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1675,6 +1675,7 @@ S: Maintained
 F: slirp/
 F: net/slirp.c
 F: include/net/slirp.h
+T: git http://people.debian.org/~sthibault/qemu.git slirp
 T: git git://git.kiszka.org/qemu.git queues/slirp
 
 Stubs
-- 
2.17.0


Re: [Qemu-devel] [PATCH] slirp: Add Samuel Thibault's staging tree for slirp
Posted by Samuel Thibault 7 years, 8 months ago
Ping? I'm not sure who I am supposed to get a review from, or if I have
to have one at all?

Samuel

Samuel Thibault, le jeu. 31 mai 2018 21:48:43 +0200, a ecrit:
> Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 41cd3736a9..8822ae9b70 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1675,6 +1675,7 @@ S: Maintained
>  F: slirp/
>  F: net/slirp.c
>  F: include/net/slirp.h
> +T: git http://people.debian.org/~sthibault/qemu.git slirp
>  T: git git://git.kiszka.org/qemu.git queues/slirp
>  
>  Stubs
> -- 
> 2.17.0
> 

Re: [Qemu-devel] [PATCH] slirp: Add Samuel Thibault's staging tree for slirp
Posted by Philippe Mathieu-Daudé 7 years, 8 months ago
Hi Samuel,

On 06/06/2018 10:14 AM, Samuel Thibault wrote:
> Ping? I'm not sure who I am supposed to get a review from, or if I have
> to have one at all?

I think you don't need since your are the maintainer ;)

> 
> Samuel
> 
> Samuel Thibault, le jeu. 31 mai 2018 21:48:43 +0200, a ecrit:
>> Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
>> ---
>>  MAINTAINERS | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 41cd3736a9..8822ae9b70 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -1675,6 +1675,7 @@ S: Maintained
>>  F: slirp/
>>  F: net/slirp.c
>>  F: include/net/slirp.h
>> +T: git http://people.debian.org/~sthibault/qemu.git slirp

I tested it but got:

warning: redirecting to https://people.debian.org/~sthibault/qemu.git/

Can you use the https scheme?

Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

>>  T: git git://git.kiszka.org/qemu.git queues/slirp
>>  
>>  Stubs
>> -- 
>> 2.17.0
>>
> 

Re: [Qemu-devel] [PATCH] slirp: Add Samuel Thibault's staging tree for slirp
Posted by Thomas Huth 7 years, 8 months ago
On 06.06.2018 15:14, Samuel Thibault wrote:
> Ping? I'm not sure who I am supposed to get a review from, or if I have
> to have one at all?
> 
> Samuel
> 
> Samuel Thibault, le jeu. 31 mai 2018 21:48:43 +0200, a ecrit:
>> Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
>> ---
>>  MAINTAINERS | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 41cd3736a9..8822ae9b70 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -1675,6 +1675,7 @@ S: Maintained
>>  F: slirp/
>>  F: net/slirp.c
>>  F: include/net/slirp.h
>> +T: git http://people.debian.org/~sthibault/qemu.git slirp
>>  T: git git://git.kiszka.org/qemu.git queues/slirp

FWIW:

Acked-by: Thomas Huth <thuth@redhat.com>