[MPlayer-cvslog] CVS: main configure,1.1055,1.1056

[Rich Felker]

On Sun, Sep 11, 2005 at 02:58:10PM +0300, Ivan Kalvachev wrote:
> 2005/9/10, Nicolas Plourde CVS <syncmail at mplayerhq.hu>:
> > CVS change done by Nicolas Plourde CVS
> > 
> > Update of /cvsroot/mplayer/main
> > In directory mail:/var2/tmp/cvs-serv4309
> > 
> > Modified Files:
> >         configure
> > Log Message:
> > perl check for macosx
> > 
> > Index: configure
> > ===================================================================
> > RCS file: /cvsroot/mplayer/main/configure,v
> > retrieving revision 1.1055
> > retrieving revision 1.1056
> > diff -u -r1.1055 -r1.1056
> > --- configure   10 Sep 2005 18:38:21 -0000      1.1055
> > +++ configure   10 Sep 2005 18:44:17 -0000      1.1056
> > @@ -45,12 +45,23 @@
> >    return "$TMP"
> >  }
> > 
> > +perl_check() {
> > +  echo >> "$TMPLOG"
> ...
> 
> I very much dislike this .
> 1. It introduce perl as dependency.  Could we assume that it is
> available on all Darwin machines? What would be next? using java
> checks for solaris?
> 
> 2. The new perl check would service only one reason: to skip the
> program that checks for MacOSX API.
> I see no sense.
> 
> 3. I doubt this check could be done only in perl.
> 
> P.S.
> Sorry for not noticing the toppic in dev-end.

Agree, I'm strongly against using perl to detect things too.

Rich