From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp48.i.mail.ru (smtp48.i.mail.ru [94.100.177.108]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 42DFE46971A for ; Fri, 6 Dec 2019 14:23:52 +0300 (MSK) Date: Fri, 6 Dec 2019 14:23:51 +0300 From: Kirill Yukhin Message-ID: <20191206112351.sg7zj4blzr65fgmz@tarantool.org> References: <720acf3e6dbd775132b35c73e12e124e0067a5ec.1575011179.git.avtikhon@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <720acf3e6dbd775132b35c73e12e124e0067a5ec.1575011179.git.avtikhon@tarantool.org> Subject: Re: [Tarantool-patches] [PATCH v2] build: fix unit tests build with lrt List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Alexander V. Tikhonov" Cc: tarantool-patches@dev.tarantool.org Hello, On 29 ноя 10:06, Alexander V. Tikhonov wrote: > After the commit 77fa45bd05f8cdd4c0f9bad85185ef5b61528d49 > ('lua: add fiber.top() listing fiber cpu consumption') > the unit tests builds failed like: > > /opt/rh/devtoolset-6/root/usr/libexec/gcc/x86_64-redhat-linux/6.3.1/ld: > ../../src/lib/core/libcore.a(fiber.c.o): undefined reference to symbol > 'clock_gettime@@GLIBC_2.2.5' > //lib64/librt.so.1: error adding symbols: DSO missing from command line > collect2: error: ld returned 1 exit status > test/unit/CMakeFiles/cbus.test.dir/build.make:108: recipe for target > 'test/unit/cbus.test' failed > make[2]: *** [test/unit/cbus.test] Error 1 > > Found that fiber.cc is using now clock_gettime(), which requires -lrt > with glibc. To fix it added librt dependency for core library for glibc. > Due to glibc requires for -lrt for clock_gettime() only for some > versions, check 'man clock_gettime.2': > 'Link with -lrt (only for glibc versions before 2.17).' > the check whether is able to use clock_gettime() w/o librt library is > added. > > Close #4639 > --- > > Github: https://github.com/tarantool/tarantool/tree/avtikhon/gh-4639-lrt-suggested-full-ci > Issue: https://github.com/tarantool/tarantool/issues/4639 I've checked your patch into master. -- Regards, Kirill Yukhin