From: Alexander Turenko <alexander.turenko@tarantool.org> To: HustonMmmavr <huston.mavr@gmail.com> Cc: tarantool-patches@dev.tarantool.org, yaroslav.dynnikov@gmail.com Subject: Re: [Tarantool-patches] [PATCH v2] build: refactor static build process Date: Tue, 25 Aug 2020 17:51:03 +0300 [thread overview] Message-ID: <20200825145103.hiqmjkimuk3al3rt@tkn_work_nb> (raw) In-Reply-To: <20200825140108.52090-1-huston.mavr@gmail.com> I looked very briefly (not thoroughly at all) on this iteration. There is nothing that confuses me (except few tiny comments below). I hope Igor will do thorough review. WBR, Alexander Turenko. > +if (APPLE) > + find_program(C_COMPILER clang) > + find_program(CXX_COMPILER clang++) > +else() > + find_program(C_COMPILER gcc) > + find_program(CXX_COMPILER g++) > +endif() Can we just leave it default? In offline discussion Alexandr B. said that tarantool builds with gcc, but icu with clang that gives some problem. Possible solution is to pass ${CMAKE_C_COMPILER} (and CXX too where necessary) to a subproject as we do for c-ares and curl. It seems it is already done, so maybe it worth to re-check whether it solves the problem. Anyway, if we really need to set a compiler here explicitly, I don't mind. Just noted that this way is somewhat unusual as I see. > diff --git a/static-build/test/static-build/box.lua b/static-build/test/static-build/box.lua > new file mode 100755 > index 000000000..bad8a9055 > --- /dev/null > +++ b/static-build/test/static-build/box.lua > @@ -0,0 +1,3 @@ > +#!/usr/bin/env tarantool > + > +require('console').listen(os.getenv('ADMIN')) Is looks redundant, see the comment below. > diff --git a/static-build/test/static-build/suite.ini b/static-build/test/static-build/suite.ini > new file mode 100644 > index 000000000..92e349466 > --- /dev/null > +++ b/static-build/test/static-build/suite.ini > @@ -0,0 +1,5 @@ > +[default] > +core = app > +description = Static build tests > +script = box.lua > +is_parallel = True 'script' does not have sense for 'core = app' test, it is for 'core = tarantool' tests.
next prev parent reply other threads:[~2020-08-25 14:51 UTC|newest] Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-08-25 14:01 HustonMmmavr 2020-08-25 14:51 ` Alexander Turenko [this message] 2020-08-27 21:43 ` Alexandr Barulev 2020-08-31 17:37 ` Alexandr Barulev 2020-09-04 17:41 ` Igor Munkin 2020-09-09 14:49 ` Alexandr Barulev
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20200825145103.hiqmjkimuk3al3rt@tkn_work_nb \ --to=alexander.turenko@tarantool.org \ --cc=huston.mavr@gmail.com \ --cc=tarantool-patches@dev.tarantool.org \ --cc=yaroslav.dynnikov@gmail.com \ --subject='Re: [Tarantool-patches] [PATCH v2] build: refactor static build process' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox