Your message dated Tue, 10 Apr 2018 07:55:27 +0000 with message-id <e1f5o7j-0001md...@fasolo.debian.org> and subject line Bug#789267: fixed in googletest 1.8.0-7 has caused the Debian Bug report #789267, regarding libgtest-dev: inconsistency with GTEST_HAS_PTHREAD in library interface leads to crashes on non-linux to be marked as done.
This means that you claim that the problem has been dealt with. If this is not the case it is now your responsibility to reopen the Bug report if necessary, and/or fix the problem forthwith. (NB: If you are a system administrator and have no idea what this message is talking about, this may indicate a serious mail system misconfiguration somewhere. Please contact ow...@bugs.debian.org immediately.) -- 789267: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=789267 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems
--- Begin Message ---Package: libgtest-dev Version: 1.7.0-3 Severity: normal Dear Maintainer, Current libgtest-dev has two different means to determine if GTEST_HAS_PTHREAD is defined: one in CMake rules [1] and second one in the header [2]. When they don't match [3], the static library and the client code might be compiled with different "value" of this define. This, in turn, leads to the crashes because the definition of ThreadLocal class are different depending on the value of GTEST_HAS_PTHREAD. A possible solution would be to always declare CMake-determined defines as a public interface of gtest target, so everything which links gtest will get those defines as well. For this case, something along the lines "target_compile_definitions(gtest PUBLIC -DGTEST_HAS_PTHREAD)" If you prefer, I can try to produce a patch. [1] /usr/src/gtest/cmake/internal_utils.cmake:108 [2] /usr/include/gtest/internal/gtest-port.h:(473-481) [3] e.g. on our non-linux architectures -- System Information: Debian Release: 8.0 APT prefers unstable APT policy: (500, 'unstable'), (500, 'testing'), (500, 'stable'), (1, 'experimental') Architecture: amd64 (x86_64) Kernel: Linux 3.16.0-4-amd64 (SMP w/2 CPU cores) Locale: LANG=fi_FI.UTF-8, LC_CTYPE=fi_FI.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Init: sysvinit (via /sbin/init) -- no debconf information
--- End Message ---
--- Begin Message ---Source: googletest Source-Version: 1.8.0-7 We believe that the bug you reported is fixed in the latest version of googletest, which is due to be installed in the Debian FTP archive. A summary of the changes between this version and the previous one is attached. Thank you for reporting the bug, which will now be closed. If you have further comments please address them to 789...@bugs.debian.org, and the maintainer will reopen the bug report if appropriate. Debian distribution maintenance software pp. Steve M. Robbins <s...@debian.org> (supplier of updated googletest package) (This message was generated automatically at their request; if you believe that there is a problem with it please contact the archive administrators by mailing ftpmas...@ftp-master.debian.org) -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 Format: 1.8 Date: Mon, 09 Apr 2018 23:39:24 -0500 Source: googletest Binary: googletest libgtest-dev google-mock Architecture: source Version: 1.8.0-7 Distribution: unstable Urgency: medium Maintainer: Steve M. Robbins <s...@debian.org> Changed-By: Steve M. Robbins <s...@debian.org> Description: google-mock - Google's framework for writing and using C++ mock classes googletest - Google's C++ test framework libgtest-dev - Google's framework for writing C++ tests [source code] Closes: 789267 856915 866837 868234 Changes: googletest (1.8.0-7) unstable; urgency=medium . [ Steven Robbins ] * [8275985] Change to debhelper compat=11. * [d6465bc] Remove now-obsolete --parallel. . * [11261ca] 0006-Use-GNU-Install-Dirs-for-multiarch.patch: New. Ensure libraries installed into usr/lib/<triplet>. . * [37e59b2] 0005-inconsistency-with-GTEST_HAS_PTHREAD-in-library-inte.patch. Helps to ensure tests compiled against gtest use same defines as gtest. Closes: #789267. . * [34a076a] Annotate python build dependency with :native. Closes: #856915. . * [5100a5a] Update VCS to salsa. . * [19cfdd3] Remove Giuseppe Iuculano <iucul...@debian.org> at request of MIA Team. Closes: #866837. . * [527c04e] priority extra has been replaced by priority optional . * [49faee7] Provide compiled libraries for both googltest and googlemock. Closes: #868234 Checksums-Sha1: 634302c762fdcbf8bd3a32f093884cc55ecdc121 2035 googletest_1.8.0-7.dsc 97f0303890572bc551ef7d4855a8184c213448fa 10032 googletest_1.8.0-7.debian.tar.xz 7909cf2849f22adf0f9d52a0075e0388f249ad0b 7030 googletest_1.8.0-7_source.buildinfo Checksums-Sha256: 2c8429c26058a85e9766aaf4fe694a648746cb030e4b49ab2b6cba203e51641d 2035 googletest_1.8.0-7.dsc 37765172d9efdfb614378d95dd1b35a575003d90bf27cfdfe46648374d215473 10032 googletest_1.8.0-7.debian.tar.xz ba7250964af3f5c5c21204ab3a36189c82fad8d8bdbdae7e16a2ec4a9751111c 7030 googletest_1.8.0-7_source.buildinfo Files: 9e3506e07e7700340d3ce63282829a8e 2035 devel optional googletest_1.8.0-7.dsc fa4a80ee1f7f2b4316e919e6ad9fa06b 10032 devel optional googletest_1.8.0-7.debian.tar.xz 32621b9093cbe514c9899f55c15eaf02 7030 devel optional googletest_1.8.0-7_source.buildinfo -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEy89k8fa3rclNjyokyeVeL63I9LkFAlrMQaYACgkQyeVeL63I 9LnpBA/+LEWgZFXNNL2V2ZmEEDxcAjp/dq9rtYKsDq7rMZ//I6rmeZVtz0BBJdiS Dx9MPOMOqw5Qkp+Vzjd+oX/82gz4LTuXxc3CuJcq61buO+SEaBTT2r5/nFgBcBWJ PX8KOpoS9KXt5FLsZ+S0kZtaYBiMeehW/z6I9Rb5np4O0oN735qv2ME5CcvRwzAq fGu3j3hm1h3MjozVV1qU9rwPWhMcaWrf03Tv6K1EIEdrCW94Xwbg/3qlnpJtAJDc Vj8uvA/smCtsRQpZRff8pFeogbN+TNh0ZV8E5mPXgxYkXlRTBuUHKfmmnL6TD5rA kLeGAjt7JBf6vsvt9o0I/DY38wjcX5bD7rEP3L3X3kmCL6Jy+pLKUl4LnmUuOEOh xmiZOUp7P42h/0rWyP984V3PY2/nZX5FcGRBMtMT42A4AP/FtkvagY0P1Of5bBE9 L+COm1gEPHGThahu+rop7Skt0g+/Q9TyrTIdh3eLMmXEhVhWlq1Ga26dORiC2tg7 tJ2ImjkunZRsFvz5bUGuPeqCEqaJF6QaGnLFzKmAiRdAYcuBTBFYwKDcEyTAV4s/ x7Lsj+WDKgU9o10FyD4cwniY62UAkvHesaqm6JpTo0Igq1WxQq9lYx8K8yKMw6GY Ri66qBSY6h+tvDnGT7lKi2rPc6vg65QX9UK4xS3RpBta25TvWP0= =GW46 -----END PGP SIGNATURE-----
--- End Message ---
