package/gtest: bump to version 1.11.0

Python is removed as dependency.
gtest uses python for self-tests which are not run by buildroot,
and the remaining scripts are not used by the build, and aren't
maintained or supported.

Special handling for gtest-config and gmock-config is removed aswell,
the CMake Buildsystem now does take care of those.

Installation of the python gmock generator scripts is conditional upon
host-python[3]. Since this is one of the unsupported + unmaintained
scripts, installation probably should be gutted completely.

Signed-off-by: Norbert Lange <nolange79@gmail.com>
---
 package/gtest/gtest.hash |  4 +--
 package/gtest/gtest.mk   | 53 ++++++----------------------------------
 2 files changed, 10 insertions(+), 47 deletions(-)

Hello Norbert,

thank you for updating this package!

I added it some years ago, but do not work with it at the time being and
do not have time to maintain it. Would you mind "adopting" it?

Kind regards

Stephan

On 08/07/2021 16:59, Norbert Lange wrote:
> Python is removed as dependency.
> gtest uses python for self-tests which are not run by buildroot,
> and the remaining scripts are not used by the build, and aren't
> maintained or supported.
> 
> Special handling for gtest-config and gmock-config is removed aswell,
> the CMake Buildsystem now does take care of those.
> 
> Installation of the python gmock generator scripts is conditional upon
> host-python[3].

 No it's not. The location is conditional, but it's always installed. So I kept
that part.

> Since this is one of the unsupported + unmaintained
> scripts, installation probably should be gutted completely.

 This may be true though.


 Applied to master with the above change.

 Regards,
 Arnout

> 
> Signed-off-by: Norbert Lange <nolange79@gmail.com>
> ---
>  package/gtest/gtest.hash |  4 +--
>  package/gtest/gtest.mk   | 53 ++++++----------------------------------
>  2 files changed, 10 insertions(+), 47 deletions(-)
> 
> diff --git a/package/gtest/gtest.hash b/package/gtest/gtest.hash
> index 082c597780..d21fea7142 100644
> --- a/package/gtest/gtest.hash
> +++ b/package/gtest/gtest.hash
> @@ -1,3 +1,3 @@
>  # Locally computed:
> -sha256	9dc9157a9a1551ec7a7e43daea9a694a0bb5fb8bec81235d8a1e6ef64c716dcb	gtest-1.10.0.tar.gz
> -sha256	9702de7e4117a8e2b20dafab11ffda58c198aede066406496bef670d40a22138	googletest/LICENSE
> +sha256  b4870bf121ff7795ba20d20bcdd8627b8e088f2d1dab299a031c1034eddc93d5  gtest-1.11.0.tar.gz
> +sha256  9702de7e4117a8e2b20dafab11ffda58c198aede066406496bef670d40a22138  LICENSE
> diff --git a/package/gtest/gtest.mk b/package/gtest/gtest.mk
> index f8b2339d7e..994f63ec2d 100644
> --- a/package/gtest/gtest.mk
> +++ b/package/gtest/gtest.mk
> @@ -4,7 +4,7 @@
>  #
>  ################################################################################
>  
> -GTEST_VERSION = 1.10.0
> +GTEST_VERSION = 1.11.0
>  GTEST_SITE = $(call github,google,googletest,release-$(GTEST_VERSION))
>  GTEST_INSTALL_STAGING = YES
>  GTEST_INSTALL_TARGET = NO
> @@ -20,15 +20,12 @@ endif
>  HOST_GTEST_LICENSE = Apache-2.0
>  HOST_GTEST_LICENSE_FILES = googlemock/scripts/generator/LICENSE
>  ifeq ($(BR2_PACKAGE_PYTHON3),y)
> -HOST_GTEST_PYTHON_VERSION = $(PYTHON3_VERSION_MAJOR)
> -HOST_GTEST_DEPENDENCIES += host-python3
> -else
> -HOST_GTEST_PYTHON_VERSION = $(PYTHON_VERSION_MAJOR)
> -HOST_GTEST_DEPENDENCIES += host-python
> -endif
> -
>  HOST_GTEST_GMOCK_PYTHONPATH = \
> -	$(HOST_DIR)/lib/python$(HOST_GTEST_PYTHON_VERSION)/site-packages
> +	$(HOST_DIR)/lib/python$(PYTHON3_VERSION_MAJOR)/site-packages
> +else ifeq ($(BR2_PACKAGE_PYTHON),y)
> +HOST_GTEST_GMOCK_PYTHONPATH = \
> +	$(HOST_DIR)/lib/python$(PYTHON_VERSION_MAJOR)/site-packages
> +endif
>  
>  # While it is possible to build gtest as shared library, using this gtest shared
>  # library requires to set some special configure option in the project using
> @@ -48,54 +45,20 @@ endif
>  
>  GTEST_CONF_OPTS += -DBUILD_GTEST=ON
>  
> -# Generate the gtest-config script manually, since the CMake build system is
> -# not doing it.
> -define GTEST_INSTALL_GTEST_CONFIG
> -	sed 's%@PACKAGE_TARNAME@%gtest%;\
> -		s%@PACKAGE_VERSION@%$(GTEST_VERSION)%;\
> -		s%@prefix@%$(STAGING_DIR)/usr%;\
> -		s%@exec_prefix@%$(STAGING_DIR)/usr%;\
> -		s%@libdir@%$(STAGING_DIR)/usr/lib%;\
> -		s%@includedir@%$(STAGING_DIR)/usr/include%;\
> -		s%@bindir@%$(STAGING_DIR)/usr/bin%;\
> -		s%@PTHREAD_CFLAGS@%%;\
> -		s%@PTHREAD_LIBS@%-lpthread%;' \
> -		$(@D)/googletest/scripts/gtest-config.in \
> -		> $(STAGING_DIR)/usr/bin/gtest-config
> -	chmod +x $(STAGING_DIR)/usr/bin/gtest-config
> -endef
> -GTEST_POST_INSTALL_STAGING_HOOKS = GTEST_INSTALL_GTEST_CONFIG
> -
>  ifeq ($(BR2_PACKAGE_GTEST_GMOCK),y)
>  GTEST_CONF_OPTS += -DBUILD_GMOCK=ON
> -
> -# Generate the gmock-config script manually, since the CMake build system is
> -# not doing it.
> -define GTEST_INSTALL_GMOCK_CONFIG
> -	sed 's%@PACKAGE_TARNAME@%gmock%;\
> -		s%@PACKAGE_VERSION@%$(GTEST_VERSION)%;\
> -		s%@prefix@%$(STAGING_DIR)/usr%;\
> -		s%@exec_prefix@%$(STAGING_DIR)/usr%;\
> -		s%@libdir@%$(STAGING_DIR)/usr/lib%;\
> -		s%@includedir@%$(STAGING_DIR)/usr/include%;\
> -		s%@bindir@%$(STAGING_DIR)/usr/bin%;\
> -		s%@PTHREAD_CFLAGS@%%;\
> -		s%@PTHREAD_LIBS@%-lpthread%;' \
> -		$(@D)/googlemock/scripts/gmock-config.in \
> -		> $(STAGING_DIR)/usr/bin/gmock-config
> -	chmod +x $(STAGING_DIR)/usr/bin/gmock-config
> -endef
> -GTEST_POST_INSTALL_STAGING_HOOKS += GTEST_INSTALL_GMOCK_CONFIG
>  else
>  GTEST_CONF_OPTS += -DBUILD_GMOCK=OFF
>  endif
>  
> +ifneq ($(HOST_GTEST_GMOCK_PYTHONPATH),)
>  define HOST_GTEST_INSTALL_CMDS
>  	$(INSTALL) -D -m 0755 $(@D)/googlemock/scripts/generator/gmock_gen.py \
>  		$(HOST_DIR)/bin/gmock_gen
>  	cp -rp $(@D)/googlemock/scripts/generator/cpp \
>  		$(HOST_GTEST_GMOCK_PYTHONPATH)
>  endef
> +endif
>  
>  $(eval $(cmake-package))
>  # The host package does not build anything, just installs gmock_gen stuff, so
>

Am Sa., 24. Juli 2021 um 22:42 Uhr schrieb Arnout Vandecappelle
<arnout@mind.be>:
>
>
>
> On 08/07/2021 16:59, Norbert Lange wrote:
> > Python is removed as dependency.
> > gtest uses python for self-tests which are not run by buildroot,
> > and the remaining scripts are not used by the build, and aren't
> > maintained or supported.
> >
> > Special handling for gtest-config and gmock-config is removed aswell,
> > the CMake Buildsystem now does take care of those.
> >
> > Installation of the python gmock generator scripts is conditional upon
> > host-python[3].
>
>  No it's not. The location is conditional, but it's always installed. So I kept
> that part.

yeah... unless you dont build host-python[3], then things fall apart:

/usr/bin/install -D -m 0755
/tmp/buildroot/build/host-gtest-1.11.0/googlemock/scripts/generator/gmock_gen.py
/tmp/buildroot/host/bin/gmock_gen
cp -rp /tmp/buildroot/build/host-gtest-1.11.0/googlemock/scripts/generator/cpp
/tmp/buildroot/host/lib/python2.7/site-packages
cp: cannot create directory
'/tmp/buildroot/host/lib/python2.7/site-packages': No such file or
directory
make: *** [package/pkg-generic.mk:280:
/tmp/buildroot/build/host-gtest-1.11.0/.stamp_host_installed] Error 1

>
> > Since this is one of the unsupported + unmaintained
> > scripts, installation probably should be gutted completely.
>
>  This may be true though.

I am not sure what they are used for, seems they were necessary
for gmock some time ago?

>
>
>  Applied to master with the above change.
>
>  Regards,
>  Arnout

buildbot failures incoming?

Norbert