[MPlayer-dev-eng] configuration patch to allow setting of ar from the commandline

[Marc Hoffman]

On 8/16/07, Marc Hoffman <mmhoffm at gmail.com> wrote:
> On 8/16/07, Diego Biurrun <diego at biurrun.de> wrote:
> > On Thu, Aug 16, 2007 at 09:04:56AM -0400, Marc Hoffman wrote:
> > > On 8/15/07, Diego Biurrun <diego at biurrun.de> wrote:
> > > > On Wed, Aug 15, 2007 at 08:50:52PM -0400, Marc Hoffman wrote:
> > > > > On 8/15/07, Diego Biurrun <diego at biurrun.de> wrote:
> > > > > > On Wed, Aug 15, 2007 at 11:34:11AM -0400, mmh wrote:
> > > > > > >
> > > > > > > patch to allow toplevel configuration change of the librarian. Needed
> > > > > > > for cross compilation on some platforms where ar has a different
> > > > > > > implementation e.g. macosx->linux.
> > > > > >
> > > > > > Apply.
> > > > >
> > > > > I need to do the same with ranlib, can I bring that out as well?
> > > >
> > > > Yes.
> > >
> > > Done and commited.
> >
> > You (and I during review) missed adding the new option to the help
> > output.
>
> I will fix it. good catch.
>
> >
> > > I was thinking it might be real nice to configure
> > > those automatically somehow.  I'm not sure how one might go about
> > > this.  Here is a stab at it just so you get the idea.  Most configure
> > > environments support this type of target option and default tools
> > > setup.
> > >
> > >
> > > .e.g. given on the commandline --target=xxx-yyy-zzz then:
> > > _cc=${_target}-gcc
> > > ...
> > >
> > > You really do need to run the commandline twice or, make it have this
> > > type of semantics when its the first ac_option.
> >
> > I don't see why it needs to be run twice ..
> >
> So you can override it.  if we do it at the end of the the loop i.e.
> just check if target is set and then setup cc, as, ... then you can't
> set them by hand. (which someone might want to do)  If I put it in the
> loop i.e. where target is currently caught then its commandline
> position dependent. (which might be ok)
>

I never got a definitive answer on this Diego.

Thanks
Marc