[MPlayer-dev-eng] [PATCH] --target=alpha_ev5-linux

Diego Biurrun diego at biurrun.de
Mon Oct 22 18:00:20 CEST 2007


On Mon, Oct 08, 2007 at 02:39:12PM +0200, Diego Biurrun wrote:
> On Sat, Oct 06, 2007 at 04:49:35PM +0200, A Mennucc wrote:
> > Dominik 'Rathann' Mierzejewski ha scritto:
> > > On Saturday, 06 October 2007 at 15:16, A Mennucc wrote:
> > > [...]
> > >>>>  int main() {
> > >>>>      unsigned long ver, mask;
> > >>>> @@ -2142,9 +2149,8 @@
> > >>> Cosmetics.
> > >> yes; just reindented to represent code block
> > >>
> > >> the "real code" is just  3 lines
> > > [...]
> > > 
> > > I still think that the cosmetic indentation fix below
> > > should be in a separate patch. Without it, the whole hunk
> > > would consist of only adding "fi".
> > 
> > here is the code-only patch
> 
> This patch contains two unrelated changes.
> 
> > --- configure	(revisione 24712)
> > +++ configure	(copia locale)
> > @@ -146,7 +146,7 @@
> >  
> >  alpha() {
> >    case "$host_arch" in
> > -    alpha) return 0;;
> > +    alpha*) return 0;;
> 
> Why?
> 
> > @@ -1257,7 +1257,7 @@
> >          fi
> >        ;;
> >        macppc|ppc|ppc64) host_arch=ppc ;;
> > -      alpha) host_arch=alpha ;;
> > +      alpha*) host_arch=alpha ;;
> >        sparc) host_arch=sparc ;;
> >        sparc64) host_arch=sparc64 ;;
> >        parisc*|hppa*|9000*) host_arch=hppa ;;
> > @@ -2126,13 +2126,19 @@
> >  
> >      ;;
> >  
> > -  alpha)
> > +  alpha*)
> 
> This is redundant after the change above.

Andrea, will you send in an updated patch please?

Diego



More information about the MPlayer-dev-eng mailing list