in the interest of stimulating debate : IMHO the iotivity code base has no business dealing with libs like gtest, tinythis-or-that, etc. these are just dependencies, no different in principle than dependencies on libc, glib, etc. if you want to do iotivity it's your responsibility to set up the environment. don't expect the iotivity sdk to do it for you.
Doing otherwise results in the fragility that is now on display. in the case of gtest, consider what happens when we move to cross-compilation. fwiw, I've got cross-compilation working (sort of; WIP alert) in iochibity. I've completely removed gtest because I might want to use gtest for multiple projects, and I do not want 16 copies of the source. so I arrange things so that I have one copy of the gtest source and my build logic puts the libs in the right place - different dirs for different targets (this depends on env vars set by running 'source' on a config file). it's pretty simple, although it does require the extra step of building and installing gtest. But that's normal, imo. I'll provide details later but for now I just want to put this on the radar. gregg On Sep 1, 2016 1:48 PM, "Nash, George" <george.nash at intel.com> wrote: > I am trying Phil Coval's patch. > > https://gerrit.iotivity.org/gerrit/#/c/11119/ > > Phil's patch is basically is pointing to a copy of the zip file found in > the fedoraproject. I think the fix is good to get things running again > right away but we probably want to change it to use the code from github in > the future. I think someone mentioned moving to 1.8.0 which may not be a > bad idea. > > George > > -----Original Message----- > From: iotivity-dev-bounces at lists.iotivity.org [mailto: > iotivity-dev-bounces at lists.iotivity.org] On Behalf Of Nash, George > Sent: Thursday, September 1, 2016 11:33 AM > To: Mats Wichmann <mats at osg.samsung.com>; iotivity-dev at > lists.iotivity.org > Subject: Re: [dev] Unable to build iotivity on ubuntu > > Issue 1: Google has shut down the googlecode link and moved google test > to github. > Solution: > In extlibs/gtest/SConscript change > - gtest_url = 'https://googletest.googlecode.com/files/gtest-1. > 7.0.zip' > + gtest_url = 'https://github.com/google/ > googletest/archive/release-1.7.0.zip' > > Issue #2: When the file is unzipped the name of folder has changed from > gtest-1.7.0 to googletest-release-1.7.0 Two possible solutions: > 1. change script to use new name > 2. rename folder after it is unzipped > > I think we should do option 1. I looked at the issues and found someone > asking about the name change and the googletest developers responded saying > the name change was due to the move to github and there was nothing they > could do to change it. So this name change is here to stay for now. > > Issue #3: The release no longer contains the configure script in the zip > file > Solution: > Get SCons to run autoconf to generate the configure script before > calling configure > > Issue #4: (this may be an issue with just my system) > When ./configure is run the --disable-shared option is not > recognized causing the build to fail saying 'configure error: cannot find > install-sh, or shtool in build-aux "."/build-aux' > > Still looking for solution to option #4 > > For issue #2 I have change the name in the gtest/SConsript but the path is > referenced in a few other scripts that need to also be fixed. > > Has anyone created a JIRA ticket for this issue yet? > > George > > -----Original Message----- > From: iotivity-dev-bounces at lists.iotivity.org [mailto: > iotivity-dev-bounces at lists.iotivity.org] On Behalf Of Mats Wichmann > Sent: Wednesday, August 31, 2016 6:18 PM > To: iotivity-dev at lists.iotivity.org > Subject: Re: [dev] Unable to build iotivity on ubuntu > > On 08/31/2016 07:11 PM, Thiago Macieira wrote: > > Em quarta-feira, 31 de agosto de 2016, ?s 16:40:06 PDT, Trace Log > escreveu: > >> Download > >> /home/kunal/proj/custom/iotivity-1.1.1/extlibs/gtest/gtest-1.7.0.zip > >> from https://googletest.googlecode.com/files/gtest-1.7.0.zip > Downloading ... > >> scons: *** HTTP Error 404: Not Found > >> [https://googletest.googlecode.com/files/gtest-1.7.0.zip] Stop. > >> > >> > >> the url itself > >> https://googletest.googlecode.com/files/gtest-1.7.0.zip > >> results in 404. > >> > >> Can you please suggest fix or alternative. > > > > The fix is to find the correct URL for GTest version 1.7 and change > > the file that contains the URL. Once you do that, you should submit > > the fix to IoTivity so other may benefit from it. > > > > I believe you can find the URL here: > > https://github.com/google/googletest/releases > > that is the official place (google code has been on the way out for a long > time), but apparently there's some issue with the 1.7.0 zipfile at github > (or so I was told). I know Philippe Coval was already working on this one. > > > _______________________________________________ > iotivity-dev mailing list > iotivity-dev at lists.iotivity.org > https://lists.iotivity.org/mailman/listinfo/iotivity-dev > _______________________________________________ > iotivity-dev mailing list > iotivity-dev at lists.iotivity.org > https://lists.iotivity.org/mailman/listinfo/iotivity-dev > _______________________________________________ > iotivity-dev mailing list > iotivity-dev at lists.iotivity.org > https://lists.iotivity.org/mailman/listinfo/iotivity-dev > -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.iotivity.org/pipermail/iotivity-dev/attachments/20160901/2018de5f/attachment.html>
