Review Request: Fix cmake based build of nss based ssl support on ubuntu

2013-04-05 Thread Gordon Sim
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10299/ --- Review request for qpid, Andrew Stitcher, Alan Conway, Steve Huston, and Rob

Re: Review Request: Fix cmake based build of nss based ssl support on ubuntu

2013-04-05 Thread Alan Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10299/#review18697 --- There is no flag-ordering code in the diff, I think it's from a

Re: Review Request: Fix cmake based build of nss based ssl support on ubuntu

2013-04-05 Thread Gordon Sim
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). No, the libraries were there (and were on the link command, just that they were before the .o files). The ordering change here comes

Re: Review Request: Fix cmake based build of nss based ssl support on ubuntu

2013-04-05 Thread Alan Conway
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

Re: Review Request: Fix cmake based build of nss based ssl support on ubuntu

2013-04-05 Thread Andrew Stitcher
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

Re: Review Request: Fix cmake based build of nss based ssl support on ubuntu

2013-04-05 Thread Andrew Stitcher
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