Skip to content

Fork update#1

Merged
krzikalla merged 4 commits intokrzikalla:masterfrom
google:master
Aug 25, 2020
Merged

Fork update#1
krzikalla merged 4 commits intokrzikalla:masterfrom
google:master

Conversation

@krzikalla
Copy link
Copy Markdown
Owner

No description provided.

dominichamon and others added 4 commits August 18, 2020 08:53
First attempt at a non-travis/non appveyor CI solution
…#997)

As noted in #995, this causes issues when the command line flag already
starts with "benchmark_", which they all do.

Not caught by tests as the test flags didn't start with "benchmark".

Fixes #995
Build instructions needlessly referred to make when CMake offers
a command-line interface to abstract away from the specific build
system.

Furthermore, CMake offers command-line "tool mode" which performs basic
filesystem operations. While the syntax is a bit more verbose than
Linux commands it is platform-independent. Now the commands can be
copy-pasted on both Linux and Windows and will just work.

Finally, the Release build type is included in initial commands. A natural flow
for a new-comer is to read and execute the commands and only then learn
that one has to go back and redo them again this time with proper parameters.
Now instead the parameters are only explained later but present already in the
initial commands.
* ctest is now working

* Update README

* remove commented out lines

* Tweaked docs

Added note to use parallel and cleaned build config notes

* Response to comments

* revert all but the readme

* make error message clearer

* drop --parallel
@krzikalla krzikalla merged commit 12b8cbe into krzikalla:master Aug 25, 2020
krzikalla pushed a commit that referenced this pull request Sep 15, 2023
Support for `getloadavg` was added in API level 29.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants