On Sat, Sep 06, 2008 at 11:52:03AM +0200, Søren Hauberg wrote: > lør, 06 09 2008 kl. 11:40 +0200, skrev Thomas Weber: > > Could someone please comment on the patch? > > Back when I was learning how to program, my teacher told me that global > variables are bad, and that I should avoid them at any cost. So, I agree > with your motivation. I don't use the audio package so I was hoping > someone else would comment, but if it works, then I'd say commit it to > SVN.
The global variable makes it more difficult to fix this in a simple way; but that is the package's maintainer decision. > > I plan to include it into Deban, if noone objects. > > I've never been a fan of distribution specific patches. I understand > that they sometimes are needed, but I think bugs should be fixed > upstream when possible. Is this problem really important enough to > justify applying the patch to Debian (isn't it mostly cosmetic?) ? The global variable stuff is uninteresting, the package is simply not working on Linux right now: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=496964 octave:1> demo sound sh: ofsndplay: command not found Thomas ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Octave-dev mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/octave-dev
