On Tue, Apr 23, 2013 at 12:23:28PM -0400, Scott Talbert wrote:
> On Tue, 23 Apr 2013, Phil Dibowitz wrote:
> 
> >> +m4_ifdef([AM_PROG_AR], [AM_PROG_AR])
> > What's that?
> 
> It is something I had to add in order to get the latest version of 
> autoconf to work.  I added the ifdef so it will work on older versions 
> without the AM_PROG_AR present.

I did a bunch of reading on this. Looks like one day eventually libtool will
figure this crap out for us, but in the meantime, this works.

It turns out - just to add this for prosperity - it's only needed when using
the Microsoft linker...

> >> +#define sleep(x) Sleep((x) * 1000)
> >> +#define snprintf _snprintf
> >
> > Shouldn't those be guarded with #ifdef WIN23 ?
> 
> They are - it is just further up in the source that you don't see in the 
> diff.

Cool.

Your patch looks good... 

-- 
Phil Dibowitz                             p...@ipom.com
Open Source software and tech docs        Insanity Palace of Metallica
http://www.phildev.net/                   http://www.ipom.com/

"Be who you are and say what you feel, because those who mind don't matter
 and those who matter don't mind."
 - Dr. Seuss

Attachment: signature.asc
Description: Digital signature

------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service 
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_apr
_______________________________________________
concordance-devel mailing list
concordance-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/concordance-devel

Reply via email to