<p>On Jul 28, 2011 12:43 PM, "Rogério Brito" <<a href="mailto:rbrito@ime.usp.br">rbrito@ime.usp.br</a>> wrote:<br>
><br>
> Hi, Andres<br>
><br>
> On Thu, Jul 28, 2011 at 13:26, Andres Mejia <<a href="mailto:mcitadel@gmail.com">mcitadel@gmail.com</a>> wrote:<br>
> > On Jul 28, 2011 3:15 AM, "Rogério Brito" <<a href="mailto:rbrito@ime.usp.br">rbrito@ime.usp.br</a>> wrote:<br>
> >> Can you please send "git format-patch/git send-email" patches against my<br>
> >> github tree (branch origin), with one topic per patch?<br>
> >><br>
> >> Your patches came foo-bared and they are changing at least two things (the<br>
> >> GTK stuff and the debian subdir).<br>
><br>
> Just for the record, I don't know if I gave the one above proper<br>
> emphasis, but this way, I can give you proper attribution also (I hope<br>
> that this is possible with CVS).<br>
><br>
> >> Please see the attached patch to kill portableio.c if you are using<br>
> >> libsndfile (which we should be in debian, anyway).<br>
><br>
> Just for the record, did you receive the attachment that I sent? I'm<br>
> asking because I didn't. I have to fix some of my e-mail filtering of<br>
> attachments here...</p>
<p>Yes, I got the attachment.</p>
<p>> > Please note my earlier email about sndfile. Enabling sndfile for lame breaks<br>
> > k3b and any app using kio slaves.<br>
><br>
> Oh, darn... Does that mean that Debian's lame will not be compiled<br>
> with sndfile?</p>
<p>Yes, lame is compiled using it's internal io functions.</p>
<p>One question, though: what does k3b and the kio slaves<br>
> use lame for? For decoding MP3s? Shouldn't they be using a specialized<br>
> software for decoding files?</p>
<p>K3b is a cd authoring app that also rips cds to mp3. Lame is used for mp3 encoding.</p>
<p>> > If you look around, the only fix is to use lame's internal io functions when<br>
> > this breakage happens.<br>
><br>
> Well, the functions in the "portableio" (they aren't portable) are all<br>
> easy enough to be written... All they effectively do is write a given<br>
> short/int/long in little or big endian form to a C's FILE... I can<br>
> come up with another patch that reimplement that, but that's so silly<br>
> to have to rewrite... :-(</p>
<p>I have a patch already submitted via the bug tracker. This was simple enough to write, debug, and verify it works correctly.</p>
<p>> Regards,<br>
><br>
> --<br>
> Rogério Brito : rbrito@{<a href="http://ime.usp.br">ime.usp.br</a>,<a href="http://gmail.com">gmail.com</a>} : GPG key 4096R/BCFCAAAA<br>
> <a href="http://rb.doesntexist.org">http://rb.doesntexist.org</a> : Packages for LaTeX : <a href="http://algorithms.berlios.de">algorithms.berlios.de</a><br>
> DebianQA: <a href="http://qa.debian.org/developer.php?login=rbrito%40ime.usp.br">http://qa.debian.org/developer.php?login=rbrito%40ime.usp.br</a><br>
</p>