[FFmpeg-devel] [PATCH] configure: allow overriding ranlib
michaelni at gmx.at
Fri Apr 25 05:14:04 CEST 2014
On Tue, Apr 22, 2014 at 07:48:48PM -0700, Timothy Gu wrote:
> On Apr 22, 2014 6:34 PM, "Michael Niedermayer" <michaelni at gmx.at> wrote:
> > On Tue, Apr 22, 2014 at 05:22:48PM -0700, Timothy Gu wrote:
> > > On Apr 22, 2014 1:41 PM, "Michael Niedermayer" <michaelni at gmx.at> wrote:
> > > >
> > > > No usecase known, i just implemented this as i thought i need it but
> > > realized i had the cross prefix
> > > > not set correctly
> > > >
> > > > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > > > ---
> > > > configure | 7 ++++---
> > > > 1 file changed, 4 insertions(+), 3 deletions(-)
> > >
> > > Didn't check: is --ranlib= documented in ./configure --help? If not then
> > > you should add it here.
> > probably not, i can of course add it, but the first question is
> > do we want/ need this --ranlib option and patch ?
> Just letting you know that you need to add doc.
> Patch fine for me, but my words probably don't count...
they do count, and we always need more people reviewing code
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel