alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Eikum <aeikum@codeweavers.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel <alsa-devel@alsa-project.org>
Subject: Re: [alsa-lib PATCH] Document strdup allocation in snd_card_get_name family
Date: Thu, 19 Jan 2012 08:48:45 -0600	[thread overview]
Message-ID: <20120119144845.GT24575@foghorn.codeweavers.com> (raw)
In-Reply-To: <s5h4nvr9awq.wl%tiwai@suse.de>

On Thu, Jan 19, 2012 at 12:41:57PM +0100, Takashi Iwai wrote:
> At Wed, 18 Jan 2012 08:58:08 -0600,
> Andrew Eikum wrote:
> > 
> > This was spotted by Jörg Höhle on the wine-patches ML.
> > 
> > Signed-off-by: Andrew Eikum <aeikum@codeweavers.com>
> > ---
> > 
> > I noticed my documentation patches haven't been making their way into
> > alsa-lib.git. Am I submitting these wrong? I added the SOB line above,
> > although I notice not all patches that get into ALSA have it. On the
> > other hand, alsa-lib.git hasn't been changed since November so perhaps
> > my patches are sitting in someone's tree waiting to be pushed :)
> 
> Sorry, I simply overlooked your patches since they have been posted
> during my vacation.  Now all patches are committed and pushed out.
> 

Great, thank you!

> > Please let me know if I can help get these in.
> 
> Please add maintainers to Cc.  Also, at the next time, try to create
> patches directly applicable via git-am.  I had to apply and fix them
> manually.
> 

Hm, that's strange.  I use format-patch to generate the mail files
which I then send out with mutt, and they work fine on my end. I
pulled the mail file for the strdup patch as delivered by alsa-devel
off our mail server and imported it with git-am without any issue at
all. What do the mail files look like when they reach you?

Here's the mail as it arrived in my Maildir, which imports fine with
git-am 1.7.8.3:
<http://pastebin.com/C4R08Xj3>

Andrew
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

  reply	other threads:[~2012-01-19 14:48 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-18 14:58 [alsa-lib PATCH] Document strdup allocation in snd_card_get_name family Andrew Eikum
2012-01-19 11:41 ` Takashi Iwai
2012-01-19 14:48   ` Andrew Eikum [this message]
2012-01-19 19:41     ` Takashi Iwai

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120119144845.GT24575@foghorn.codeweavers.com \
    --to=aeikum@codeweavers.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).