Syllo / nvtop

GPU & Accelerator process monitoring for AMD, Apple, Huawei, Intel, NVIDIA and Qualcomm
Other
8.06k stars 292 forks source link

Fix format-security warnings #160

Closed felixonmars closed 2 years ago

felixonmars commented 2 years ago

nvtop 2.0.3 fails to build on Arch Linux with -Werror=format-security enabled. Let's fix it.

Syllo commented 2 years ago

LGTM. Is that blocking for Arch or is that a local cflag?

felixonmars commented 2 years ago

It's a globally enabled CFLAG for Arch. I have added this patch already, for now :)

Syllo commented 2 years ago

All right, thanks for the patch and the heads up. I've added the flag to the debug build to catch similar issues in the future.

felixonmars commented 2 years ago

That's awesome. Thanks!