qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] docs/devel/build-environment: enhance MSYS2 instructions
@ 2025-03-05 21:38 Pierrick Bouvier
  2025-03-24 17:47 ` Pierrick Bouvier
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Pierrick Bouvier @ 2025-03-05 21:38 UTC (permalink / raw)
  To: qemu-devel
  Cc: Markus Armbruster, Alex Bennée, Thomas Huth,
	Daniel P. Berrangé, Philippe Mathieu-Daudé,
	Pierrick Bouvier

Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
---
 docs/devel/build-environment.rst | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/docs/devel/build-environment.rst b/docs/devel/build-environment.rst
index f133ef2e012..661f6ea8504 100644
--- a/docs/devel/build-environment.rst
+++ b/docs/devel/build-environment.rst
@@ -97,11 +97,11 @@ build QEMU in MSYS2 itself.
 
 ::
 
-    pacman -S wget
+    pacman -S wget base-devel git
     wget https://raw.githubusercontent.com/msys2/MINGW-packages/refs/heads/master/mingw-w64-qemu/PKGBUILD
     # Some packages may be missing for your environment, installation will still
     # be done though.
-    makepkg -s PKGBUILD || true
+    makepkg --syncdeps --nobuild PKGBUILD || true
 
 Build on windows-aarch64
 ++++++++++++++++++++++++
-- 
2.39.5



^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] docs/devel/build-environment: enhance MSYS2 instructions
  2025-03-05 21:38 [PATCH] docs/devel/build-environment: enhance MSYS2 instructions Pierrick Bouvier
@ 2025-03-24 17:47 ` Pierrick Bouvier
  2025-04-28 19:35   ` Pierrick Bouvier
  2025-03-24 18:54 ` Philippe Mathieu-Daudé
  2025-04-30 10:34 ` Daniel P. Berrangé
  2 siblings, 1 reply; 9+ messages in thread
From: Pierrick Bouvier @ 2025-03-24 17:47 UTC (permalink / raw)
  To: qemu-devel
  Cc: Markus Armbruster, Alex Bennée, Thomas Huth,
	Daniel P. Berrangé, Philippe Mathieu-Daudé

On 3/5/25 13:38, Pierrick Bouvier wrote:
> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> ---
>   docs/devel/build-environment.rst | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/devel/build-environment.rst b/docs/devel/build-environment.rst
> index f133ef2e012..661f6ea8504 100644
> --- a/docs/devel/build-environment.rst
> +++ b/docs/devel/build-environment.rst
> @@ -97,11 +97,11 @@ build QEMU in MSYS2 itself.
>   
>   ::
>   
> -    pacman -S wget
> +    pacman -S wget base-devel git
>       wget https://raw.githubusercontent.com/msys2/MINGW-packages/refs/heads/master/mingw-w64-qemu/PKGBUILD
>       # Some packages may be missing for your environment, installation will still
>       # be done though.
> -    makepkg -s PKGBUILD || true
> +    makepkg --syncdeps --nobuild PKGBUILD || true
>   
>   Build on windows-aarch64
>   ++++++++++++++++++++++++

Gentle ping on this trivial change for doc.

Thanks,
Pierrick


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] docs/devel/build-environment: enhance MSYS2 instructions
  2025-03-05 21:38 [PATCH] docs/devel/build-environment: enhance MSYS2 instructions Pierrick Bouvier
  2025-03-24 17:47 ` Pierrick Bouvier
@ 2025-03-24 18:54 ` Philippe Mathieu-Daudé
  2025-04-30 10:34 ` Daniel P. Berrangé
  2 siblings, 0 replies; 9+ messages in thread
From: Philippe Mathieu-Daudé @ 2025-03-24 18:54 UTC (permalink / raw)
  To: Pierrick Bouvier, qemu-devel, Yonggang Luo, Stefan Weil
  Cc: Markus Armbruster, Alex Bennée, Thomas Huth,
	Daniel P. Berrangé

Cc'ing Stefan and Yonggang

On 5/3/25 22:38, Pierrick Bouvier wrote:
> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> ---
>   docs/devel/build-environment.rst | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/devel/build-environment.rst b/docs/devel/build-environment.rst
> index f133ef2e012..661f6ea8504 100644
> --- a/docs/devel/build-environment.rst
> +++ b/docs/devel/build-environment.rst
> @@ -97,11 +97,11 @@ build QEMU in MSYS2 itself.
>   
>   ::
>   
> -    pacman -S wget
> +    pacman -S wget base-devel git
>       wget https://raw.githubusercontent.com/msys2/MINGW-packages/refs/heads/master/mingw-w64-qemu/PKGBUILD
>       # Some packages may be missing for your environment, installation will still
>       # be done though.
> -    makepkg -s PKGBUILD || true
> +    makepkg --syncdeps --nobuild PKGBUILD || true
>   
>   Build on windows-aarch64
>   ++++++++++++++++++++++++



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] docs/devel/build-environment: enhance MSYS2 instructions
  2025-03-24 17:47 ` Pierrick Bouvier
@ 2025-04-28 19:35   ` Pierrick Bouvier
  2025-04-30 10:09     ` Thomas Huth
  0 siblings, 1 reply; 9+ messages in thread
From: Pierrick Bouvier @ 2025-04-28 19:35 UTC (permalink / raw)
  To: qemu-devel, qemu-trivial
  Cc: Markus Armbruster, Alex Bennée, Thomas Huth,
	Daniel P. Berrangé, Philippe Mathieu-Daudé

On 3/24/25 10:47 AM, Pierrick Bouvier wrote:
> On 3/5/25 13:38, Pierrick Bouvier wrote:
>> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>> ---
>>    docs/devel/build-environment.rst | 4 ++--
>>    1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/docs/devel/build-environment.rst b/docs/devel/build-environment.rst
>> index f133ef2e012..661f6ea8504 100644
>> --- a/docs/devel/build-environment.rst
>> +++ b/docs/devel/build-environment.rst
>> @@ -97,11 +97,11 @@ build QEMU in MSYS2 itself.
>>    
>>    ::
>>    
>> -    pacman -S wget
>> +    pacman -S wget base-devel git
>>        wget https://raw.githubusercontent.com/msys2/MINGW-packages/refs/heads/master/mingw-w64-qemu/PKGBUILD
>>        # Some packages may be missing for your environment, installation will still
>>        # be done though.
>> -    makepkg -s PKGBUILD || true
>> +    makepkg --syncdeps --nobuild PKGBUILD || true
>>    
>>    Build on windows-aarch64
>>    ++++++++++++++++++++++++
> 
> Gentle ping on this trivial change for doc.
>

Another gentle ping on this trivial doc change.


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] docs/devel/build-environment: enhance MSYS2 instructions
  2025-04-28 19:35   ` Pierrick Bouvier
@ 2025-04-30 10:09     ` Thomas Huth
  2025-04-30 14:34       ` Pierrick Bouvier
  0 siblings, 1 reply; 9+ messages in thread
From: Thomas Huth @ 2025-04-30 10:09 UTC (permalink / raw)
  To: Pierrick Bouvier, qemu-devel, qemu-trivial
  Cc: Markus Armbruster, Alex Bennée, Daniel P. Berrangé,
	Philippe Mathieu-Daudé

On 28/04/2025 21.35, Pierrick Bouvier wrote:
> On 3/24/25 10:47 AM, Pierrick Bouvier wrote:
>> On 3/5/25 13:38, Pierrick Bouvier wrote:
>>> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>>> ---
>>>    docs/devel/build-environment.rst | 4 ++--
>>>    1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/docs/devel/build-environment.rst b/docs/devel/build- 
>>> environment.rst
>>> index f133ef2e012..661f6ea8504 100644
>>> --- a/docs/devel/build-environment.rst
>>> +++ b/docs/devel/build-environment.rst
>>> @@ -97,11 +97,11 @@ build QEMU in MSYS2 itself.
>>>    ::
>>> -    pacman -S wget
>>> +    pacman -S wget base-devel git
>>>        wget https://raw.githubusercontent.com/msys2/MINGW-packages/refs/ 
>>> heads/master/mingw-w64-qemu/PKGBUILD
>>>        # Some packages may be missing for your environment, installation 
>>> will still
>>>        # be done though.
>>> -    makepkg -s PKGBUILD || true
>>> +    makepkg --syncdeps --nobuild PKGBUILD || true
>>>    Build on windows-aarch64
>>>    ++++++++++++++++++++++++
>>
>> Gentle ping on this trivial change for doc.
>>
> 
> Another gentle ping on this trivial doc change.

Not really my turf, I don't have much clue about the MSYS2 environment, but 
since there were no objections, I can add it to my next PR.

  Thomas



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] docs/devel/build-environment: enhance MSYS2 instructions
  2025-03-05 21:38 [PATCH] docs/devel/build-environment: enhance MSYS2 instructions Pierrick Bouvier
  2025-03-24 17:47 ` Pierrick Bouvier
  2025-03-24 18:54 ` Philippe Mathieu-Daudé
@ 2025-04-30 10:34 ` Daniel P. Berrangé
  2 siblings, 0 replies; 9+ messages in thread
From: Daniel P. Berrangé @ 2025-04-30 10:34 UTC (permalink / raw)
  To: Pierrick Bouvier
  Cc: qemu-devel, Markus Armbruster, Alex Bennée, Thomas Huth,
	Philippe Mathieu-Daudé

On Wed, Mar 05, 2025 at 01:38:53PM -0800, Pierrick Bouvier wrote:

The commit message really ought to explain why '--syncdeps --nobuild'
is improving on the current '-s'

> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> ---
>  docs/devel/build-environment.rst | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/devel/build-environment.rst b/docs/devel/build-environment.rst
> index f133ef2e012..661f6ea8504 100644
> --- a/docs/devel/build-environment.rst
> +++ b/docs/devel/build-environment.rst
> @@ -97,11 +97,11 @@ build QEMU in MSYS2 itself.
>  
>  ::
>  
> -    pacman -S wget
> +    pacman -S wget base-devel git
>      wget https://raw.githubusercontent.com/msys2/MINGW-packages/refs/heads/master/mingw-w64-qemu/PKGBUILD
>      # Some packages may be missing for your environment, installation will still
>      # be done though.
> -    makepkg -s PKGBUILD || true
> +    makepkg --syncdeps --nobuild PKGBUILD || true
>  
>  Build on windows-aarch64
>  ++++++++++++++++++++++++
> -- 
> 2.39.5
> 

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] docs/devel/build-environment: enhance MSYS2 instructions
  2025-04-30 10:09     ` Thomas Huth
@ 2025-04-30 14:34       ` Pierrick Bouvier
  2025-04-30 17:54         ` Thomas Huth
  0 siblings, 1 reply; 9+ messages in thread
From: Pierrick Bouvier @ 2025-04-30 14:34 UTC (permalink / raw)
  To: Thomas Huth, qemu-devel, qemu-trivial
  Cc: Markus Armbruster, Alex Bennée, Daniel P. Berrangé,
	Philippe Mathieu-Daudé

On 4/30/25 3:09 AM, Thomas Huth wrote:
> On 28/04/2025 21.35, Pierrick Bouvier wrote:
>> On 3/24/25 10:47 AM, Pierrick Bouvier wrote:
>>> On 3/5/25 13:38, Pierrick Bouvier wrote:
>>>> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>>>> ---
>>>>     docs/devel/build-environment.rst | 4 ++--
>>>>     1 file changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/docs/devel/build-environment.rst b/docs/devel/build-
>>>> environment.rst
>>>> index f133ef2e012..661f6ea8504 100644
>>>> --- a/docs/devel/build-environment.rst
>>>> +++ b/docs/devel/build-environment.rst
>>>> @@ -97,11 +97,11 @@ build QEMU in MSYS2 itself.
>>>>     ::
>>>> -    pacman -S wget
>>>> +    pacman -S wget base-devel git
>>>>         wget https://raw.githubusercontent.com/msys2/MINGW-packages/refs/
>>>> heads/master/mingw-w64-qemu/PKGBUILD
>>>>         # Some packages may be missing for your environment, installation
>>>> will still
>>>>         # be done though.
>>>> -    makepkg -s PKGBUILD || true
>>>> +    makepkg --syncdeps --nobuild PKGBUILD || true
>>>>     Build on windows-aarch64
>>>>     ++++++++++++++++++++++++
>>>
>>> Gentle ping on this trivial change for doc.
>>>
>>
>> Another gentle ping on this trivial doc change.
> 
> Not really my turf, I don't have much clue about the MSYS2 environment, but
> since there were no objections, I can add it to my next PR.
>

Thank you, that's appreciated.

>    Thomas
> 



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] docs/devel/build-environment: enhance MSYS2 instructions
  2025-04-30 14:34       ` Pierrick Bouvier
@ 2025-04-30 17:54         ` Thomas Huth
  2025-04-30 18:12           ` Pierrick Bouvier
  0 siblings, 1 reply; 9+ messages in thread
From: Thomas Huth @ 2025-04-30 17:54 UTC (permalink / raw)
  To: Pierrick Bouvier, qemu-devel, qemu-trivial
  Cc: Markus Armbruster, Alex Bennée, Daniel P. Berrangé,
	Philippe Mathieu-Daudé

On 30/04/2025 16.34, Pierrick Bouvier wrote:
> On 4/30/25 3:09 AM, Thomas Huth wrote:
>> On 28/04/2025 21.35, Pierrick Bouvier wrote:
>>> On 3/24/25 10:47 AM, Pierrick Bouvier wrote:
>>>> On 3/5/25 13:38, Pierrick Bouvier wrote:
>>>>> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>>>>> ---
>>>>>     docs/devel/build-environment.rst | 4 ++--
>>>>>     1 file changed, 2 insertions(+), 2 deletions(-)
>>>>>
>>>>> diff --git a/docs/devel/build-environment.rst b/docs/devel/build-
>>>>> environment.rst
>>>>> index f133ef2e012..661f6ea8504 100644
>>>>> --- a/docs/devel/build-environment.rst
>>>>> +++ b/docs/devel/build-environment.rst
>>>>> @@ -97,11 +97,11 @@ build QEMU in MSYS2 itself.
>>>>>     ::
>>>>> -    pacman -S wget
>>>>> +    pacman -S wget base-devel git
>>>>>         wget https://raw.githubusercontent.com/msys2/MINGW-packages/refs/
>>>>> heads/master/mingw-w64-qemu/PKGBUILD
>>>>>         # Some packages may be missing for your environment, installation
>>>>> will still
>>>>>         # be done though.
>>>>> -    makepkg -s PKGBUILD || true
>>>>> +    makepkg --syncdeps --nobuild PKGBUILD || true
>>>>>     Build on windows-aarch64
>>>>>     ++++++++++++++++++++++++
>>>>
>>>> Gentle ping on this trivial change for doc.
>>>>
>>>
>>> Another gentle ping on this trivial doc change.
>>
>> Not really my turf, I don't have much clue about the MSYS2 environment, but
>> since there were no objections, I can add it to my next PR.
>>
> 
> Thank you, that's appreciated.

Could you still please provide a proper patch description, as Daniel suggested?

  Thanks,
   Thomas



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] docs/devel/build-environment: enhance MSYS2 instructions
  2025-04-30 17:54         ` Thomas Huth
@ 2025-04-30 18:12           ` Pierrick Bouvier
  0 siblings, 0 replies; 9+ messages in thread
From: Pierrick Bouvier @ 2025-04-30 18:12 UTC (permalink / raw)
  To: Thomas Huth, qemu-devel, qemu-trivial
  Cc: Markus Armbruster, Alex Bennée, Daniel P. Berrangé,
	Philippe Mathieu-Daudé

On 4/30/25 10:54 AM, Thomas Huth wrote:
> On 30/04/2025 16.34, Pierrick Bouvier wrote:
>> On 4/30/25 3:09 AM, Thomas Huth wrote:
>>> On 28/04/2025 21.35, Pierrick Bouvier wrote:
>>>> On 3/24/25 10:47 AM, Pierrick Bouvier wrote:
>>>>> On 3/5/25 13:38, Pierrick Bouvier wrote:
>>>>>> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>>>>>> ---
>>>>>>      docs/devel/build-environment.rst | 4 ++--
>>>>>>      1 file changed, 2 insertions(+), 2 deletions(-)
>>>>>>
>>>>>> diff --git a/docs/devel/build-environment.rst b/docs/devel/build-
>>>>>> environment.rst
>>>>>> index f133ef2e012..661f6ea8504 100644
>>>>>> --- a/docs/devel/build-environment.rst
>>>>>> +++ b/docs/devel/build-environment.rst
>>>>>> @@ -97,11 +97,11 @@ build QEMU in MSYS2 itself.
>>>>>>      ::
>>>>>> -    pacman -S wget
>>>>>> +    pacman -S wget base-devel git
>>>>>>          wget https://raw.githubusercontent.com/msys2/MINGW-packages/refs/
>>>>>> heads/master/mingw-w64-qemu/PKGBUILD
>>>>>>          # Some packages may be missing for your environment, installation
>>>>>> will still
>>>>>>          # be done though.
>>>>>> -    makepkg -s PKGBUILD || true
>>>>>> +    makepkg --syncdeps --nobuild PKGBUILD || true
>>>>>>      Build on windows-aarch64
>>>>>>      ++++++++++++++++++++++++
>>>>>
>>>>> Gentle ping on this trivial change for doc.
>>>>>
>>>>
>>>> Another gentle ping on this trivial doc change.
>>>
>>> Not really my turf, I don't have much clue about the MSYS2 environment, but
>>> since there were no objections, I can add it to my next PR.
>>>
>>
>> Thank you, that's appreciated.
> 
> Could you still please provide a proper patch description, as Daniel suggested?
>

Sure.
I just sent v2: 20250430181047.2043492-1-pierrick.bouvier@linaro.org

>    Thanks,
>     Thomas
> 

Thanks,
Pierrick


^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2025-04-30 18:12 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-05 21:38 [PATCH] docs/devel/build-environment: enhance MSYS2 instructions Pierrick Bouvier
2025-03-24 17:47 ` Pierrick Bouvier
2025-04-28 19:35   ` Pierrick Bouvier
2025-04-30 10:09     ` Thomas Huth
2025-04-30 14:34       ` Pierrick Bouvier
2025-04-30 17:54         ` Thomas Huth
2025-04-30 18:12           ` Pierrick Bouvier
2025-03-24 18:54 ` Philippe Mathieu-Daudé
2025-04-30 10:34 ` Daniel P. Berrangé

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).