DigitalInBlue / Celero

C++ Benchmark Authoring Library/Framework
Other
824 stars 95 forks source link

Requires curses.h #134

Closed dtpike closed 5 years ago

dtpike commented 5 years ago

Bug Report:

I tried to cross-compile celero for ARMv8 and realized I needed curses.h to be installed. This is a bit of pain when cross-building since I'm given a sample root file system to use and it's not easy for me to install packages on it with a package manager.

After looking through Console.cpp a bit further I didn't see any reason why curses.h was required. I'm wondering if this is left-over from code that was previously in Console.cpp or put here for another reason. I can submit a PR to remove curses.h. Let me know if this is here for a specific reason.

Line in question: https://github.com/DigitalInBlue/Celero/blob/master/src/Console.cpp#L27

Thanks!