mbox

[PULL,1.1] qemu-ga build fix for OpenBSD

Message ID 1337885561-10629-1-git-send-email-mdroth@linux.vnet.ibm.com
State New
Headers show

Pull-request

git://github.com/mdroth/qemu.git qga-pull-5-24-12

Message

Michael Roth May 24, 2012, 6:52 p.m. UTC
The following changes since commit aeb29b6459cb9496b38c820f3faff64cf2369d0d:

  audio: Always call fini on exit (2012-05-24 19:35:27 +0400)

are available in the git repository at:
  git://github.com/mdroth/qemu.git qga-pull-5-24-12

Luiz Capitulino (2):
      configure: check if environ is declared
      qemu-ga: Fix missing environ declaration

 configure            |   19 +++++++++++++++++++
 qga/commands-posix.c |    6 +++++-
 2 files changed, 24 insertions(+), 1 deletions(-)

Comments

Michael Roth May 29, 2012, 3:30 p.m. UTC | #1
On Thu, May 24, 2012 at 01:52:39PM -0500, Michael Roth wrote:
> The following changes since commit aeb29b6459cb9496b38c820f3faff64cf2369d0d:
> 
>   audio: Always call fini on exit (2012-05-24 19:35:27 +0400)
> 
> are available in the git repository at:
>   git://github.com/mdroth/qemu.git qga-pull-5-24-12
> 
> Luiz Capitulino (2):
>       configure: check if environ is declared
>       qemu-ga: Fix missing environ declaration
> 
>  configure            |   19 +++++++++++++++++++
>  qga/commands-posix.c |    6 +++++-
>  2 files changed, 24 insertions(+), 1 deletions(-)
> 

Please ignore. There are a couple new patches in the queue that are
dependent on patches in this pull. Will send a new pull request.
Anthony Liguori May 30, 2012, 1:25 a.m. UTC | #2
On 05/29/2012 10:30 AM, Michael Roth wrote:
> On Thu, May 24, 2012 at 01:52:39PM -0500, Michael Roth wrote:
>> The following changes since commit aeb29b6459cb9496b38c820f3faff64cf2369d0d:
>>
>>    audio: Always call fini on exit (2012-05-24 19:35:27 +0400)
>>
>> are available in the git repository at:
>>    git://github.com/mdroth/qemu.git qga-pull-5-24-12
>>
>> Luiz Capitulino (2):
>>        configure: check if environ is declared
>>        qemu-ga: Fix missing environ declaration
>>
>>   configure            |   19 +++++++++++++++++++
>>   qga/commands-posix.c |    6 +++++-
>>   2 files changed, 24 insertions(+), 1 deletions(-)
>>
>
> Please ignore. There are a couple new patches in the queue that are
> dependent on patches in this pull. Will send a new pull request.

Too late.  I applied it last night before you sent this note.

Please rebase your pull request with a patch that fixes the previous patch.

Regards,

Anthony Liguori

>
>
Michael Roth May 30, 2012, 1:54 a.m. UTC | #3
On Tue, May 29, 2012 at 08:25:54PM -0500, Anthony Liguori wrote:
> On 05/29/2012 10:30 AM, Michael Roth wrote:
> >On Thu, May 24, 2012 at 01:52:39PM -0500, Michael Roth wrote:
> >>The following changes since commit aeb29b6459cb9496b38c820f3faff64cf2369d0d:
> >>
> >>   audio: Always call fini on exit (2012-05-24 19:35:27 +0400)
> >>
> >>are available in the git repository at:
> >>   git://github.com/mdroth/qemu.git qga-pull-5-24-12
> >>
> >>Luiz Capitulino (2):
> >>       configure: check if environ is declared
> >>       qemu-ga: Fix missing environ declaration
> >>
> >>  configure            |   19 +++++++++++++++++++
> >>  qga/commands-posix.c |    6 +++++-
> >>  2 files changed, 24 insertions(+), 1 deletions(-)
> >>
> >
> >Please ignore. There are a couple new patches in the queue that are
> >dependent on patches in this pull. Will send a new pull request.
> 
> Too late.  I applied it last night before you sent this note.
> 
> Please rebase your pull request with a patch that fixes the previous patch.

Will rebase-resend it in a few. The PULL on the list currently just has 2
additional patches (1 with a dependency on this pull), so the patches
here are still up-to-date.

> 
> Regards,
> 
> Anthony Liguori
> 
> >
> >
>
Anthony Liguori May 30, 2012, 2:37 a.m. UTC | #4
On 05/30/2012 09:54 AM, Michael Roth wrote:
> On Tue, May 29, 2012 at 08:25:54PM -0500, Anthony Liguori wrote:
>> On 05/29/2012 10:30 AM, Michael Roth wrote:
>>> On Thu, May 24, 2012 at 01:52:39PM -0500, Michael Roth wrote:
>>>> The following changes since commit aeb29b6459cb9496b38c820f3faff64cf2369d0d:
>>>>
>>>>    audio: Always call fini on exit (2012-05-24 19:35:27 +0400)
>>>>
>>>> are available in the git repository at:
>>>>    git://github.com/mdroth/qemu.git qga-pull-5-24-12
>>>>
>>>> Luiz Capitulino (2):
>>>>        configure: check if environ is declared
>>>>        qemu-ga: Fix missing environ declaration
>>>>
>>>>   configure            |   19 +++++++++++++++++++
>>>>   qga/commands-posix.c |    6 +++++-
>>>>   2 files changed, 24 insertions(+), 1 deletions(-)
>>>>
>>>
>>> Please ignore. There are a couple new patches in the queue that are
>>> dependent on patches in this pull. Will send a new pull request.
>>
>> Too late.  I applied it last night before you sent this note.
>>
>> Please rebase your pull request with a patch that fixes the previous patch.
>
> Will rebase-resend it in a few. The PULL on the list currently just has 2
> additional patches (1 with a dependency on this pull), so the patches
> here are still up-to-date.

I see a 5-29-12 branch but I don't see a pull request.

Regards,

Anthony Liguori

>
>>
>> Regards,
>>
>> Anthony Liguori
>>
>>>
>>>
>>
>
Michael Roth May 30, 2012, 3:21 a.m. UTC | #5
On Wed, May 30, 2012 at 10:37:09AM +0800, Anthony Liguori wrote:
> On 05/30/2012 09:54 AM, Michael Roth wrote:
> >On Tue, May 29, 2012 at 08:25:54PM -0500, Anthony Liguori wrote:
> >>On 05/29/2012 10:30 AM, Michael Roth wrote:
> >>>On Thu, May 24, 2012 at 01:52:39PM -0500, Michael Roth wrote:
> >>>>The following changes since commit aeb29b6459cb9496b38c820f3faff64cf2369d0d:
> >>>>
> >>>>   audio: Always call fini on exit (2012-05-24 19:35:27 +0400)
> >>>>
> >>>>are available in the git repository at:
> >>>>   git://github.com/mdroth/qemu.git qga-pull-5-24-12
> >>>>
> >>>>Luiz Capitulino (2):
> >>>>       configure: check if environ is declared
> >>>>       qemu-ga: Fix missing environ declaration
> >>>>
> >>>>  configure            |   19 +++++++++++++++++++
> >>>>  qga/commands-posix.c |    6 +++++-
> >>>>  2 files changed, 24 insertions(+), 1 deletions(-)
> >>>>
> >>>
> >>>Please ignore. There are a couple new patches in the queue that are
> >>>dependent on patches in this pull. Will send a new pull request.
> >>
> >>Too late.  I applied it last night before you sent this note.
> >>
> >>Please rebase your pull request with a patch that fixes the previous patch.
> >
> >Will rebase-resend it in a few. The PULL on the list currently just has 2
> >additional patches (1 with a dependency on this pull), so the patches
> >here are still up-to-date.
> 
> I see a 5-29-12 branch but I don't see a pull request.

It went to aliguori@us.ibm.com but I missed qemu-devel on the
cc list =/ That pull was from before your reply and is stale now anyway though,
rebased pull request qga-pull-5-29-12-v2 is incoming.

> 
> Regards,
> 
> Anthony Liguori
> 
> >
> >>
> >>Regards,
> >>
> >>Anthony Liguori
> >>
> >>>
> >>>
> >>
> >
>