> On April 5, 2013, 1:51 p.m., Alan Conway wrote: > > There is no flag-ordering code in the diff, I think it's from a different > > issue (missing nss libraries). > > Gordon Sim wrote: > No, the libraries were there (and were on the link command, just that > they were before the .o files). The ordering change here comes from the way > in which the libraries are set. Rather than having them added through the > LINKFLAGS, they are explictly added as target_link_libraries. > > Now, this may very well not be the right fix. It was driven by trial and > error rather than in-depth understanding of cmake. So if there are better > solutions, I'd be delighted to see them. > > Alan Conway wrote: > I get it now. I think that is the right solution: tell cmake about the > dependency and let it figure out how to order the link flags - which might be > different on different platformsShi > > Ship it!
I haven't looked at this in enough depth yet, but I'm surprised at the circumlocution needed considering that what you are doing is the basic use case for the nss3 detection logic! So I'd expect that there would be some variable it directly outputs that avoid you needing the foreach. - Andrew ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10299/#review18697 ----------------------------------------------------------- On April 5, 2013, 1:14 p.m., Gordon Sim wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10299/ > ----------------------------------------------------------- > > (Updated April 5, 2013, 1:14 p.m.) > > > Review request for qpid, Andrew Stitcher, Alan Conway, Steve Huston, and Rob > Godfrey. > > > Description > ------- > > The problem is that when the --as-needed flag is used with the linker (the > default for recent versions of ubuntu), libraries must follow any .o file > that needs them or they get ignored as not needed. At present the libraries > are included before the .o files. This is an update to the initial patch that > avoids the duplicated libraries on the link command line. > > > This addresses bug QPID-4702. > https://issues.apache.org/jira/browse/QPID-4702 > > > Diffs > ----- > > /trunk/qpid/cpp/src/ssl.cmake 1464148 > > Diff: https://reviews.apache.org/r/10299/diff/ > > > Testing > ------- > > make test > > > Thanks, > > Gordon Sim > >
