* [Buildroot] [PATCH 6/6] gtest: retrieve package from github @ 2016-08-31 16:24 Fabrice Fontaine 2016-09-06 13:58 ` Carlos Santos 0 siblings, 1 reply; 10+ messages in thread From: Fabrice Fontaine @ 2016-08-31 16:24 UTC (permalink / raw) To: buildroot https://googletest.googlecode.com/files is no more available, so update gtest.mk and gtest.hash to retrieve googletest source code as a tar.gz (and not a zip) from github as https://googletest.googlecode.com is now a redirection to https://github.com/google/googletest. Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com> --- package/gtest/gtest.hash | 4 ++-- package/gtest/gtest.mk | 9 ++------- 2 files changed, 4 insertions(+), 9 deletions(-) diff --git a/package/gtest/gtest.hash b/package/gtest/gtest.hash index 8ff79cb..79e31e4 100644 --- a/package/gtest/gtest.hash +++ b/package/gtest/gtest.hash @@ -1,2 +1,2 @@ -# From http://code.google.com/p/googletest/downloads/detail?name=gtest-1.7.0.zip&can=2&q= -sha1 f85f6d2481e2c6c4a18539e391aa4ea8ab0394af gtest-1.7.0.zip +# Locally computed: +sha256 f73a6546fdf9fce9ff93a5015e0333a8af3062a152a9ad6bcb772c96687016cc gtest-release-1.7.0.tar.gz diff --git a/package/gtest/gtest.mk b/package/gtest/gtest.mk index 11aa7e9..eb30905 100644 --- a/package/gtest/gtest.mk +++ b/package/gtest/gtest.mk @@ -5,9 +5,8 @@ ################################################################################ # Make sure this remains the same version as the gmock one -GTEST_VERSION = 1.7.0 -GTEST_SOURCE = gtest-$(GTEST_VERSION).zip -GTEST_SITE = http://googletest.googlecode.com/files +GTEST_VERSION = release-1.7.0 +GTEST_SITE = $(call github,google,googletest,$(GTEST_VERSION)) GTEST_INSTALL_STAGING = YES GTEST_INSTALL_TARGET = NO GTEST_LICENSE = BSD-3c @@ -22,10 +21,6 @@ GTEST_LICENSE_FILES = LICENSE # the gtest sources. GTEST_CONF_OPTS = -DBUILD_SHARED_LIBS=OFF -define GTEST_EXTRACT_CMDS - $(UNZIP) $(DL_DIR)/$(GTEST_SOURCE) -d $(BUILD_DIR) -endef - define GTEST_INSTALL_STAGING_CMDS $(INSTALL) -D -m 0755 $(@D)/libgtest.a $(STAGING_DIR)/usr/lib/libgtest.a $(INSTALL) -D -m 0755 $(@D)/libgtest_main.a $(STAGING_DIR)/usr/lib/libgtest_main.a -- 2.7.4 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 6/6] gtest: retrieve package from github 2016-08-31 16:24 [Buildroot] [PATCH 6/6] gtest: retrieve package from github Fabrice Fontaine @ 2016-09-06 13:58 ` Carlos Santos 2016-09-06 20:40 ` Arnout Vandecappelle 0 siblings, 1 reply; 10+ messages in thread From: Carlos Santos @ 2016-09-06 13:58 UTC (permalink / raw) To: buildroot > From: "Fabrice Fontaine" <fontaine.fabrice@gmail.com> > To: buildroot at buildroot.org > Cc: "Fabrice Fontaine" <fabrice.fontaine@orange.com> > Sent: Wednesday, August 31, 2016 1:24:56 PM > Subject: [Buildroot] [PATCH 6/6] gtest: retrieve package from github > https://googletest.googlecode.com/files is no more available, so update > gtest.mk and gtest.hash to retrieve googletest source code as a tar.gz > (and not a zip) from github as https://googletest.googlecode.com is now > a redirection to https://github.com/google/googletest. > > Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com> > --- > package/gtest/gtest.hash | 4 ++-- > package/gtest/gtest.mk | 9 ++------- > 2 files changed, 4 insertions(+), 9 deletions(-) > > diff --git a/package/gtest/gtest.hash b/package/gtest/gtest.hash > index 8ff79cb..79e31e4 100644 > --- a/package/gtest/gtest.hash > +++ b/package/gtest/gtest.hash > @@ -1,2 +1,2 @@ > -# From > http://code.google.com/p/googletest/downloads/detail?name=gtest-1.7.0.zip&can=2&q= > -sha1 f85f6d2481e2c6c4a18539e391aa4ea8ab0394af gtest-1.7.0.zip > +# Locally computed: > +sha256 f73a6546fdf9fce9ff93a5015e0333a8af3062a152a9ad6bcb772c96687016cc > gtest-release-1.7.0.tar.gz > diff --git a/package/gtest/gtest.mk b/package/gtest/gtest.mk > index 11aa7e9..eb30905 100644 > --- a/package/gtest/gtest.mk > +++ b/package/gtest/gtest.mk > @@ -5,9 +5,8 @@ > ################################################################################ > > # Make sure this remains the same version as the gmock one > -GTEST_VERSION = 1.7.0 > -GTEST_SOURCE = gtest-$(GTEST_VERSION).zip > -GTEST_SITE = http://googletest.googlecode.com/files > +GTEST_VERSION = release-1.7.0 > +GTEST_SITE = $(call github,google,googletest,$(GTEST_VERSION)) > GTEST_INSTALL_STAGING = YES > GTEST_INSTALL_TARGET = NO > GTEST_LICENSE = BSD-3c > @@ -22,10 +21,6 @@ GTEST_LICENSE_FILES = LICENSE > # the gtest sources. > GTEST_CONF_OPTS = -DBUILD_SHARED_LIBS=OFF > > -define GTEST_EXTRACT_CMDS > - $(UNZIP) $(DL_DIR)/$(GTEST_SOURCE) -d $(BUILD_DIR) > -endef > - > define GTEST_INSTALL_STAGING_CMDS > $(INSTALL) -D -m 0755 $(@D)/libgtest.a $(STAGING_DIR)/usr/lib/libgtest.a > $(INSTALL) -D -m 0755 $(@D)/libgtest_main.a > $(STAGING_DIR)/usr/lib/libgtest_main.a > -- > 2.7.4 GMock needs to be fixed too. I attempted to do this before (http://patchwork.ozlabs.org/patch/586130/) but it was refused, unfortunately. Carlos Santos (Casantos) DATACOM, P&D ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 6/6] gtest: retrieve package from github 2016-09-06 13:58 ` Carlos Santos @ 2016-09-06 20:40 ` Arnout Vandecappelle [not found] ` <29481_1473259225_57D026D9_29481_7433_1_EEB3FED4859B6C488DDDEC4B2D3DE92B0FCC280F@OPEXCLILM23.corporate.adroot.infra.ftgroup> 2016-09-07 23:22 ` Carlos Santos 0 siblings, 2 replies; 10+ messages in thread From: Arnout Vandecappelle @ 2016-09-06 20:40 UTC (permalink / raw) To: buildroot On 06-09-16 15:58, Carlos Santos wrote: >> From: "Fabrice Fontaine" <fontaine.fabrice@gmail.com> >> To: buildroot at buildroot.org >> Cc: "Fabrice Fontaine" <fabrice.fontaine@orange.com> >> Sent: Wednesday, August 31, 2016 1:24:56 PM >> Subject: [Buildroot] [PATCH 6/6] gtest: retrieve package from github > >> https://googletest.googlecode.com/files is no more available, so update >> gtest.mk and gtest.hash to retrieve googletest source code as a tar.gz >> (and not a zip) from github as https://googletest.googlecode.com is now >> a redirection to https://github.com/google/googletest. >> >> Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com> [snip] > GMock needs to be fixed too. I attempted to do this before (http://patchwork.ozlabs.org/patch/586130/) but it was refused, unfortunately. Quoting Yann from that patchwork entry: > As I understand it, gtest and gmock are being phased out as separate > packages, and are to be replaced with the googletest package. > > So why don't we directly add "googletest" and get rid of gtest and > gmock, rather than fix them to eventually get rid of them? In that contet, I think it's a sane idea to have this commit as a first step, and do the bump to v1.8.0 (which implies merging gtest and gmock) in a second step. Fabrice, care to work on that? If you do, here are some tips: - GMOCK should become a suboption of GTEST: BR2_PACKAGE_GTEST_GMOCK. - A legacy entry should be added for it. - gmock now has a usable cmake and builds gtest implicitly. - host-gmock is tricky - but perhaps it's no longer needed. Regards, Arnout -- Arnout Vandecappelle arnout at mind be Senior Embedded Software Architect +32-16-286500 Essensium/Mind http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF ^ permalink raw reply [flat|nested] 10+ messages in thread
[parent not found: <29481_1473259225_57D026D9_29481_7433_1_EEB3FED4859B6C488DDDEC4B2D3DE92B0FCC280F@OPEXCLILM23.corporate.adroot.infra.ftgroup>]
* [Buildroot] [PATCH 6/6] gtest: retrieve package from github [not found] ` <29481_1473259225_57D026D9_29481_7433_1_EEB3FED4859B6C488DDDEC4B2D3DE92B0FCC280F@OPEXCLILM23.corporate.adroot.infra.ftgroup> @ 2016-09-07 23:03 ` Carlos Santos 0 siblings, 0 replies; 10+ messages in thread From: Carlos Santos @ 2016-09-07 23:03 UTC (permalink / raw) To: buildroot > From: "fabrice fontaine" <fabrice.fontaine@orange.com> > To: "Arnout Vandecappelle" <arnout@mind.be>, "Carlos Santos" <casantos@datacom.ind.br>, "Fabrice Fontaine" > <fontaine.fabrice@gmail.com> > Cc: buildroot at buildroot.org > Sent: Wednesday, September 7, 2016 11:40:24 AM > Subject: RE: [Buildroot] [PATCH 6/6] gtest: retrieve package from github > Dear Arnout, > > I sent a new patch merging gmock inside gtest. > However, I did not quite understand what is a "legacy entry", did you mean that > we should keep a package/gmock/Config.in with a BR2_PACKAGE_GMOCK that should > enable BR2_PACKAGE_GTEST_GMOCK? A legacy entry would be an entry in Config.in.legacy, Read the description at the beginning of that file. Carlos Santos (Casantos) DATACOM, P&D ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH 6/6] gtest: retrieve package from github 2016-09-06 20:40 ` Arnout Vandecappelle [not found] ` <29481_1473259225_57D026D9_29481_7433_1_EEB3FED4859B6C488DDDEC4B2D3DE92B0FCC280F@OPEXCLILM23.corporate.adroot.infra.ftgroup> @ 2016-09-07 23:22 ` Carlos Santos 2016-09-08 16:58 ` [Buildroot] [PATCH] gmock: download from github and convert to a cmake-package Carlos Santos 1 sibling, 1 reply; 10+ messages in thread From: Carlos Santos @ 2016-09-07 23:22 UTC (permalink / raw) To: buildroot > From: "Arnout Vandecappelle" <arnout@mind.be> > To: "Carlos Santos" <casantos@datacom.ind.br>, "Fabrice Fontaine" <fontaine.fabrice@gmail.com> > Cc: "Fabrice Fontaine" <fabrice.fontaine@orange.com>, buildroot at buildroot.org > Sent: Tuesday, September 6, 2016 5:40:58 PM > Subject: Re: [Buildroot] [PATCH 6/6] gtest: retrieve package from github > On 06-09-16 15:58, Carlos Santos wrote: >>> From: "Fabrice Fontaine" <fontaine.fabrice@gmail.com> >>> To: buildroot at buildroot.org >>> Cc: "Fabrice Fontaine" <fabrice.fontaine@orange.com> >>> Sent: Wednesday, August 31, 2016 1:24:56 PM >>> Subject: [Buildroot] [PATCH 6/6] gtest: retrieve package from github >> >>> https://googletest.googlecode.com/files is no more available, so update >>> gtest.mk and gtest.hash to retrieve googletest source code as a tar.gz >>> (and not a zip) from github as https://googletest.googlecode.com is now >>> a redirection to https://github.com/google/googletest. >>> >>> Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com> > [snip] >> GMock needs to be fixed too. I attempted to do this before >> (http://patchwork.ozlabs.org/patch/586130/) but it was refused, unfortunately. > > Quoting Yann from that patchwork entry: >> As I understand it, gtest and gmock are being phased out as separate >> packages, and are to be replaced with the googletest package. >> >> So why don't we directly add "googletest" and get rid of gtest and >> gmock, rather than fix them to eventually get rid of them? > > In that contet, I think it's a sane idea to have this commit as a first step, > and do the bump to v1.8.0 (which implies merging gtest and gmock) in a second > step. It can be done, even though I don't think it's really necessary to do this. > Fabrice, care to work on that? If you do, here are some tips: > > - GMOCK should become a suboption of GTEST: BR2_PACKAGE_GTEST_GMOCK. > - A legacy entry should be added for it. > - gmock now has a usable cmake and builds gtest implicitly. > - host-gmock is tricky - but perhaps it's no longer needed. I just sent an updated patch with a proposed solution for the gmock host-gmock packages using virtual packages, so no legacy entry is necessary. Carlos Santos (Casantos) DATACOM, P&D ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH] gmock: download from github and convert to a cmake-package 2016-09-07 23:22 ` Carlos Santos @ 2016-09-08 16:58 ` Carlos Santos 2016-09-08 19:20 ` Thomas Petazzoni 0 siblings, 1 reply; 10+ messages in thread From: Carlos Santos @ 2016-09-08 16:58 UTC (permalink / raw) To: buildroot https://googlemock.googlecode.com/files is no more available, so update gmock.mk and gmock.hash to retrieve googlemock source code as a tar.gz (and not a zip) from github as https://googlemock.googlecode.com is now a redirection to https://github.com/google/googlemock. The new tarball does not contain autotools stuff, so gmock was converted to a cmake package, using a trick to find the gtest build directory. It would be possible to extract the gtest source in the gmock directory or somewhere under $(STAGING_DIR) but this trick would be even uglier. Based on the work by Fabrice Fontaine on gtest and on comments made by Thomas Petazzoni and Arnout Vandecappelle. Signed-off-by: Carlos Santos <casantos@datacom.ind.br> --- package/gmock/0001-force-use-python2.patch | 4 ++-- package/gmock/0002-fix-gtest-path.patch | 15 ++++++++++++ package/gmock/Config.in | 2 +- package/gmock/gmock.hash | 2 +- package/gmock/gmock.mk | 37 +++++++++++------------------- 5 files changed, 32 insertions(+), 28 deletions(-) create mode 100644 package/gmock/0002-fix-gtest-path.patch diff --git a/package/gmock/0001-force-use-python2.patch b/package/gmock/0001-force-use-python2.patch index 5dcb231..12f7c23 100644 --- a/package/gmock/0001-force-use-python2.patch +++ b/package/gmock/0001-force-use-python2.patch @@ -2,8 +2,8 @@ Force use of Python 2 even when Python 3 is the default Python interpreter. Signed-off-by: Carlos Santos <casantos@datacom.ind.br> ---- ./gtest/scripts/fuse_gtest_files.py.orig 2013-09-18 14:48:30.000000000 -0300 -+++ ./gtest/scripts/fuse_gtest_files.py 2015-07-22 15:42:53.291591205 -0300 +--- ./scripts/fuse_gmock_files.py.orig 2013-09-18 14:48:30.000000000 -0300 ++++ ./scripts/fuse_gmock_files.py 2015-07-22 15:42:53.291591205 -0300 @@ -1,4 +1,4 @@ -#!/usr/bin/env python +#!/usr/bin/env python2 diff --git a/package/gmock/0002-fix-gtest-path.patch b/package/gmock/0002-fix-gtest-path.patch new file mode 100644 index 0000000..4314270 --- /dev/null +++ b/package/gmock/0002-fix-gtest-path.patch @@ -0,0 +1,15 @@ +Help the GMock build to locate the GTest source code + +Signed-off-by: Carlos Santos <casantos@datacom.ind.br> + +--- ./CMakeLists.txt.orig 2013-09-18 21:32:04.000000000 -0300 ++++ ./CMakeLists.txt 2016-09-08 11:10:51.519780433 -0300 +@@ -15,7 +15,7 @@ + if (EXISTS "${CMAKE_CURRENT_SOURCE_DIR}/gtest/CMakeLists.txt") + set(gtest_dir gtest) + else() +- set(gtest_dir ../gtest) ++ set(gtest_dir ../gtest-${GTEST_VERSION}) + endif() + + # Defines pre_project_set_up_hermetic_build() and set_up_hermetic_build(). diff --git a/package/gmock/Config.in b/package/gmock/Config.in index ec3eb92..895fbc9 100644 --- a/package/gmock/Config.in +++ b/package/gmock/Config.in @@ -20,7 +20,7 @@ config BR2_PACKAGE_GMOCK * works on Linux, Mac OS X, Windows, Windows Mobile, minGW, and Symbian. - http://code.google.com/p/googlemock/ + https://github.com/google/googlemock/ There are both host and target packages. The target one has include files required to compile the tests and the static libraries required diff --git a/package/gmock/gmock.hash b/package/gmock/gmock.hash index 2b71739..5f9befc 100644 --- a/package/gmock/gmock.hash +++ b/package/gmock/gmock.hash @@ -1,2 +1,2 @@ # Locally computed -sha256 26fcbb5925b74ad5fc8c26b0495dfc96353f4d553492eb97e85a8a6d2f43095b gmock-1.7.0.zip +sha256 3f20b6acb37e5a98e8c4518165711e3e35d47deb6cdb5a4dd4566563b5efd232 gmock-release-1.7.0.tar.gz diff --git a/package/gmock/gmock.mk b/package/gmock/gmock.mk index 4f04422..e317e91 100644 --- a/package/gmock/gmock.mk +++ b/package/gmock/gmock.mk @@ -5,9 +5,8 @@ ################################################################################ # Make sure this remains the same version as the gtest one -GMOCK_VERSION = 1.7.0 -GMOCK_SOURCE = gmock-$(GMOCK_VERSION).zip -GMOCK_SITE = http://googlemock.googlecode.com/files +GMOCK_VERSION = release-1.7.0 +GMOCK_SITE = $(call github,google,googlemock,$(GMOCK_VERSION)) GMOCK_INSTALL_STAGING = YES GMOCK_INSTALL_TARGET = NO GMOCK_LICENSE = BSD-3c @@ -24,41 +23,31 @@ HOST_GMOCK_PYTHONPATH=$(HOST_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR)/site-pac # "Google Mock can be used as a DLL, but the same DLL must contain Google # Test as well. See Google Test's README file for instructions on how to # set up necessary compiler settings". -GMOCK_CONF_OPTS = --enable-static --disable-shared +GMOCK_CONF_OPTS = -DBUILD_SHARED_LIBS=OFF -define GMOCK_EXTRACT_CMDS - $(UNZIP) $(DL_DIR)/$(GMOCK_SOURCE) -d $(BUILD_DIR) -endef +# This is used to help the GMock build to find the GTest source code. Passing +# GTEST_VERSION covers the case in which GTEST_OVERRIDE_SRCDIR is set, because +# GTEST_VERSION will be "custom" instead of "release-1.7.0". +GMOCK_CONF_OPTS += -DGTEST_VERSION=$(GTEST_VERSION) -# We can't use the default rule for autotools-package staging because it fails -# because it tries to rebuild/install gtest stuff and fails after this error: -# "'make install' is dangerous and not supported. Instead, see README for -# how to integrate Google Test into your build system." +# We can't use the default install rule for cmake-package because there is no +# 'install' target in the CMake-based build. define GMOCK_INSTALL_STAGING_CMDS - $(INSTALL) -D -m 0755 $(@D)/lib/.libs/libgmock.a $(STAGING_DIR)/usr/lib/libgmock.a - $(INSTALL) -D -m 0755 $(@D)/lib/.libs/libgmock_main.a $(STAGING_DIR)/usr/lib/libgmock_main.a + $(INSTALL) -D -m 0755 $(@D)/libgmock.a $(STAGING_DIR)/usr/lib/libgmock.a + $(INSTALL) -D -m 0755 $(@D)/libgmock_main.a $(STAGING_DIR)/usr/lib/libgmock_main.a $(INSTALL) -d -m 0755 $(STAGING_DIR)/usr/include/gmock/ cp -rp $(@D)/include/gmock/* $(STAGING_DIR)/usr/include/gmock/ $(INSTALL) -D -m 0755 package/gmock/gmock.pc \ $(STAGING_DIR)/usr/lib/pkgconfig/gmock.pc endef -# Unzipping inside $(@D) and moving everything from the created subdirectory is -# required because unzipping directly in $(BUILD_DIR) would cause host-gmock to -# overwrite the gmock subdir instead of unzipping in a host-gmock subdir. -define HOST_GMOCK_EXTRACT_CMDS - $(UNZIP) $(DL_DIR)/$(GMOCK_SOURCE) -d $(@D) - mv $(@D)/gmock-$(GMOCK_VERSION)/* $(@D) - rmdir $(@D)/gmock-$(GMOCK_VERSION) -endef - define HOST_GMOCK_INSTALL_CMDS $(INSTALL) -D -m 0755 $(@D)/scripts/generator/gmock_gen.py $(HOST_DIR)/usr/bin/gmock_gen.py ln -sf gmock_gen.py $(HOST_DIR)/usr/bin/gmock_gen cp -rp $(@D)/scripts/generator/cpp $(HOST_GMOCK_PYTHONPATH) endef -$(eval $(autotools-package)) +$(eval $(cmake-package)) # The host package does not build anything, just installs gmock_gen stuff, so -# it does not need to be a host-autotools-package. +# it does not need to be a host-cmake-package. $(eval $(host-generic-package)) -- 2.7.4 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH] gmock: download from github and convert to a cmake-package 2016-09-08 16:58 ` [Buildroot] [PATCH] gmock: download from github and convert to a cmake-package Carlos Santos @ 2016-09-08 19:20 ` Thomas Petazzoni 2016-09-08 20:11 ` Carlos Santos 0 siblings, 1 reply; 10+ messages in thread From: Thomas Petazzoni @ 2016-09-08 19:20 UTC (permalink / raw) To: buildroot Hello, On Thu, 8 Sep 2016 13:58:03 -0300, Carlos Santos wrote: > +--- ./CMakeLists.txt.orig 2013-09-18 21:32:04.000000000 -0300 > ++++ ./CMakeLists.txt 2016-09-08 11:10:51.519780433 -0300 > +@@ -15,7 +15,7 @@ > + if (EXISTS "${CMAKE_CURRENT_SOURCE_DIR}/gtest/CMakeLists.txt") > + set(gtest_dir gtest) > + else() > +- set(gtest_dir ../gtest) > ++ set(gtest_dir ../gtest-${GTEST_VERSION}) So you make the assumption that you have the following code organization: output/build/gmock-1.7.0 output/build/gtest-1.7.0 i.e, during its build process, gmock needs to access the gtest source code? Why is it the case? As far as I'm aware, it's not currently the case with the existing packaging. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH] gmock: download from github and convert to a cmake-package 2016-09-08 19:20 ` Thomas Petazzoni @ 2016-09-08 20:11 ` Carlos Santos 2016-09-08 20:14 ` Thomas Petazzoni 0 siblings, 1 reply; 10+ messages in thread From: Carlos Santos @ 2016-09-08 20:11 UTC (permalink / raw) To: buildroot > From: "Thomas Petazzoni" <thomas.petazzoni@free-electrons.com> > To: "Carlos Santos" <casantos@datacom.ind.br> > Cc: buildroot at buildroot.org, "fabrice fontaine" <fabrice.fontaine@orange.com>, "fontaine fabrice" > <fontaine.fabrice@gmail.com> > Sent: Thursday, September 8, 2016 4:20:33 PM > Subject: Re: [Buildroot] [PATCH] gmock: download from github and convert to a cmake-package > Hello, > > On Thu, 8 Sep 2016 13:58:03 -0300, Carlos Santos wrote: > >> +--- ./CMakeLists.txt.orig 2013-09-18 21:32:04.000000000 -0300 >> ++++ ./CMakeLists.txt 2016-09-08 11:10:51.519780433 -0300 >> +@@ -15,7 +15,7 @@ >> + if (EXISTS "${CMAKE_CURRENT_SOURCE_DIR}/gtest/CMakeLists.txt") >> + set(gtest_dir gtest) >> + else() >> +- set(gtest_dir ../gtest) >> ++ set(gtest_dir ../gtest-${GTEST_VERSION}) > > So you make the assumption that you have the following code > organization: > > output/build/gmock-1.7.0 > output/build/gtest-1.7.0 > > i.e, during its build process, gmock needs to access the gtest source > code? Right. > Why is it the case? As far as I'm aware, it's not currently the case > with the existing packaging. Because the old gmock distribution (gmock-1.7.0.zip) contained the gtest code, while the new one (gmock-release-1.7.0.tar.gz) does not. Carlos Santos (Casantos) DATACOM, P&D ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH] gmock: download from github and convert to a cmake-package 2016-09-08 20:11 ` Carlos Santos @ 2016-09-08 20:14 ` Thomas Petazzoni 2016-09-08 20:34 ` Carlos Santos 0 siblings, 1 reply; 10+ messages in thread From: Thomas Petazzoni @ 2016-09-08 20:14 UTC (permalink / raw) To: buildroot Hello, On Thu, 8 Sep 2016 17:11:48 -0300 (BRT), Carlos Santos wrote: > > Why is it the case? As far as I'm aware, it's not currently the case > > with the existing packaging. > > Because the old gmock distribution (gmock-1.7.0.zip) contained the gtest > code, while the new one (gmock-release-1.7.0.tar.gz) does not. If the old gmock distribution contained the gtest code, why do we have two separate packages for gmock and gtest ? In any case, I guess this is a transitional solution, since the final goal is to have a single package, right ? Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH] gmock: download from github and convert to a cmake-package 2016-09-08 20:14 ` Thomas Petazzoni @ 2016-09-08 20:34 ` Carlos Santos 0 siblings, 0 replies; 10+ messages in thread From: Carlos Santos @ 2016-09-08 20:34 UTC (permalink / raw) To: buildroot > From: "Thomas Petazzoni" <thomas.petazzoni@free-electrons.com> > To: "Carlos Santos" <casantos@datacom.ind.br> > Cc: "fabrice fontaine" <fabrice.fontaine@orange.com>, "fontaine fabrice" <fontaine.fabrice@gmail.com>, > buildroot at buildroot.org > Sent: Thursday, September 8, 2016 5:14:32 PM > Subject: Re: [Buildroot] [PATCH] gmock: download from github and convert to a cmake-package > Hello, > > On Thu, 8 Sep 2016 17:11:48 -0300 (BRT), Carlos Santos wrote: > >> > Why is it the case? As far as I'm aware, it's not currently the case >> > with the existing packaging. >> >> Because the old gmock distribution (gmock-1.7.0.zip) contained the gtest >> code, while the new one (gmock-release-1.7.0.tar.gz) does not. > > If the old gmock distribution contained the gtest code, why do we have > two separate packages for gmock and gtest ? In the beginning there was gtest, only. We (DATACOM) added gmock last year, due to the need to support more than a hundred packages in which we use it to run unit tests. > In any case, I guess this is a transitional solution, since the final > goal is to have a single package, right ? Yes, I think that starting in gtest v. 1.8.0 it will be possible to get rid of the separated packages. Carlos Santos (Casantos) DATACOM, P&D ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2016-09-08 20:34 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-31 16:24 [Buildroot] [PATCH 6/6] gtest: retrieve package from github Fabrice Fontaine
2016-09-06 13:58 ` Carlos Santos
2016-09-06 20:40 ` Arnout Vandecappelle
[not found] ` <29481_1473259225_57D026D9_29481_7433_1_EEB3FED4859B6C488DDDEC4B2D3DE92B0FCC280F@OPEXCLILM23.corporate.adroot.infra.ftgroup>
2016-09-07 23:03 ` Carlos Santos
2016-09-07 23:22 ` Carlos Santos
2016-09-08 16:58 ` [Buildroot] [PATCH] gmock: download from github and convert to a cmake-package Carlos Santos
2016-09-08 19:20 ` Thomas Petazzoni
2016-09-08 20:11 ` Carlos Santos
2016-09-08 20:14 ` Thomas Petazzoni
2016-09-08 20:34 ` Carlos Santos
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.