[FFmpeg-devel] [PATCH] configure: allow overriding ranlib

Michael Niedermayer 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
> then
> > > 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

patch applied

thanks

[...]
-- 
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
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140425/833d6b18/attachment.asc>


More information about the ffmpeg-devel mailing list