google/benchmark

master is broken on Android

Open

#731 aperta il 26 nov 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C++ (1539 fork)batch import
bughelp wanted

Metriche repository

Star
 (7968 star)
Metriche merge PR
 (Merge medio 4g 2h) (19 PR mergiate in 30 g)

Descrizione

master version (I tested revision c9311a44e1280853632fe2472345dd04514a2f74) fails to build on Android, with the error:

/Users/marat/benchmark/src/sysinfo.cc:586:21: error: use of undeclared identifier 'getloadavg'
  const int nelem = getloadavg(res.data(), kMaxSamples);
                    ^
1 error generated.

Repro:

git clone https://github.com/google/benchmark.git
cd benchmark
mkdir build && cd build
cmake -DCMAKE_TOOLCHAIN_FILE=/path/to/android/ndk/build/cmake/android.toolchain.cmake -DBENCHMARK_ENABLE_TESTING=OFF ..
make

Guida contributor

Direzione di ricerca
Indaga la disponibilità di getloadavg su Android e implementa una condizionale di piattaforma per utilizzare un'alternativa o disabilitare la funzionalità se non supportata.
Tech stack
cpp
Dominio
testing
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Android NDKCMakeGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65