Closed dimacurrentai closed 9 months ago
@mzhurovich, added support for dynamic libraries too, and tested on Linux, WSL, and macOS!
Probably best to review per commit, but, to get a feel, cmake/run-cmake-test.sh
is self-contained and self-explanatory!
@dimacurrentai I have one question - why q/q/q
? =)
It needs to not find current/ in ../ and ../../, and maybe in ../../..
Where did you find it thought? That's my internal local run script!
On Wed, Jan 3, 2024 at 00:04 Maxim Zhurovich @.***> wrote:
@dimacurrentai https://github.com/dimacurrentai I have one question - why q/q/q? =)
— Reply to this email directly, view it on GitHub https://github.com/C5T/Current/pull/956#issuecomment-1874218686, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVDEKLXBQOUCJTD2NY6FYRDYMQVYLAVCNFSM6AAAAABBJPCE4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZUGIYTQNRYGY . You are receiving this because you were mentioned.Message ID: @.***>
Hi @mzhurovich!
This was easier than I thought!
README.
Clean test:
This test is also run by a Github Action, both on Linux and on macOS.
What this means now is that using Current, with fast incremental builds, and with a non-header-only Googletest, is just a matter of:
src/*.cc
(as well assrc/lib_*.cc
andsrc/test_*.cc
),Makefile
fromC5T/Current/cmake
, andmake
, ormake {debug/release/debug_test/release_test/fmt/clean}
.Thanks, Dima (from Windows)