[alsa-devel] [alsa-lib PATCH] Document strdup allocation in snd_card_get_name family

Takashi Iwai tiwai at suse.de
Thu Jan 19 20:41:16 CET 2012


At Thu, 19 Jan 2012 08:48:45 -0600,
Andrew Eikum wrote:
> 
> 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 at 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>

Did you really create it via git-format-patch?  It won't create any
attachment usually but the patch must be embedded.
If it's not manageable, try git-send-email instead.


Takashi


More information about the Alsa-devel mailing list