On Tue, Nov 07, 2017 at 03:22:02PM +0000, Peter Maydell wrote: > On 7 November 2017 at 14:36, Daniel P. Berrange <berra...@redhat.com> wrote: > > The following changes since commit 5853e92207193e967abf5e4c25b4a551c7604725: > > > > Merge remote-tracking branch 'remotes/pmaydell/tags/pull-cocoa-20171107' > > into staging (2017-11-07 12:19:48 +0000) > > > > are available in the git repository at: > > > > git://github.com/berrange/qemu pull-2017-11-06-2 > > > > for you to fetch changes up to 10a7b7e6fd9f250f0506568345d7b4d2ab52889d: > > > > sockets: avoid leak of listen file descriptor (2017-11-07 14:10:20 +0000) > > > > ---------------------------------------------------------------- > > Pull IO 2017/11/06 v2 > > > > ---------------------------------------------------------------- > > Daniel P. Berrange (1): > > sockets: avoid leak of listen file descriptor > > > > util/qemu-sockets.c | 52 > > +++++++++++++++++++++++++++++----------------------- > > 1 file changed, 29 insertions(+), 23 deletions(-) > > Applied, thanks. (My test script expected your pullreq to > say "git://github.com/berrange/qemu tags/pull-2017-11-06-2" > but that might be the script not being as smart as it could be.)
Oh my bad. I forgot to actually push the tag to github before generating the cover letter text, and I guess that caused git to leave off the 'tags/' bit 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 :|