Re: [Libguestfs] [v2v PATCH v2 2/3] lib/utils: make "chown_for_libvirt_rhbz_1045069" fail hard

2023-06-30 Thread Laszlo Ersek
On 6/30/23 09:38, Richard W.M. Jones wrote:
> On Fri, Jun 30, 2023 at 09:32:33AM +0200, Laszlo Ersek wrote:
>> On 6/29/23 19:16, Richard W.M. Jones wrote:
>>> On Thu, Jun 29, 2023 at 06:40:27PM +0200, Laszlo Ersek wrote:
>>
 Either way, I can restore the parens. Do you want me to submit a v3?
>>>
>>> I think the parens are clearer.  Don't need a v3, thanks!
>>
>> Can I add your R-b that way (with the parens reinstated) to this patch
>> as well?
> 
> Yes please!

Thanks, commit range 97d8c28b7eb1..dcfea1b9b5d0.

Laszlo

___
Libguestfs mailing list
Libguestfs@redhat.com
https://listman.redhat.com/mailman/listinfo/libguestfs



Re: [Libguestfs] [v2v PATCH v2 2/3] lib/utils: make "chown_for_libvirt_rhbz_1045069" fail hard

2023-06-30 Thread Richard W.M. Jones
On Fri, Jun 30, 2023 at 09:32:33AM +0200, Laszlo Ersek wrote:
> On 6/29/23 19:16, Richard W.M. Jones wrote:
> > On Thu, Jun 29, 2023 at 06:40:27PM +0200, Laszlo Ersek wrote:
> 
> >> Either way, I can restore the parens. Do you want me to submit a v3?
> > 
> > I think the parens are clearer.  Don't need a v3, thanks!
> 
> Can I add your R-b that way (with the parens reinstated) to this patch
> as well?

Yes please!

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
libguestfs lets you edit virtual machines.  Supports shell scripting,
bindings from many languages.  http://libguestfs.org
___
Libguestfs mailing list
Libguestfs@redhat.com
https://listman.redhat.com/mailman/listinfo/libguestfs



Re: [Libguestfs] [v2v PATCH v2 2/3] lib/utils: make "chown_for_libvirt_rhbz_1045069" fail hard

2023-06-30 Thread Laszlo Ersek
On 6/29/23 19:16, Richard W.M. Jones wrote:
> On Thu, Jun 29, 2023 at 06:40:27PM +0200, Laszlo Ersek wrote:

>> Either way, I can restore the parens. Do you want me to submit a v3?
> 
> I think the parens are clearer.  Don't need a v3, thanks!

Can I add your R-b that way (with the parens reinstated) to this patch
as well?

Thanks!
Laszlo
___
Libguestfs mailing list
Libguestfs@redhat.com
https://listman.redhat.com/mailman/listinfo/libguestfs