Re: [Patch-gnuradio] Patch for usrp/host/apps to include omnithread

[Eric Blossom]

On Sat, Dec 22, 2007 at 07:09:01PM -0500, Michael Dickens wrote:
>
> I think the correct solution is to add a switch in Makefile.common (top 
> level) to right after
>
> USRP_LA = $(top_builddir)/usrp/host/lib/legacy/libusrp.la
>
> that checks the host OS (I choose "FUSB_TECH_darwin"); if it's darwin, then 
> append $(OMNITHREAD_LA) to the $(USRP_LA) list, and likewise for the 
> $(USRP_INCLUDES).  That way, -any- application or whatever that wants to 
> compile using libusrp.la will also include omnithread.la , and thus will 
> create a working executable (ditto for the includes).  And, since the 
> switch is for darwin only, all the other OSs will just do the same thing 
> they were doing before.
>
> Thoughts?  Comments?  Let me know if you want me to do these changes on an 
> SVN branch to be merged. - MLD

Michael, after a bit more thought, I think the fix needs to go in
usrp/host/lib/legacy/Makefile.am.  You should specify the conditional
addition to the library in libusrp_la_LIBADD so it'll end up in the
.la file.

Eric