[0/4] kernel-shark: CMake changes

Message

Michal Sojka March 17, 2021, 4:40 p.m. UTC

This patch series is a follow up to my previous patch for the older
kernel-shark repo. The first two patches will make life easier for
distributions like NixOS, the last two patches replace my previous
patch.

Note that I didn't add kshark-su-record to a separate component, as
suggested by Yordan, because it is not necessary. The generated polkit
action will refer to the correct path and I think it is preferable to
have all kernelshark binaries in the same directory.

Michal Sojka (4):
  kernel-shark: Allow specifying PKG_CONFIG_DIR on cmake command line
  kernel-shark: Allow specifying TT_FONT_FILE on cmake command line
  kernel-shark: Allow installing polkit policy separately
  kernel-shark: Do not use sudo in install_gui.sh + update README

 CMakeLists.txt       | 12 ++++++++++--
 README               |  2 +-
 build/install_gui.sh | 10 +++++++++-
 src/CMakeLists.txt   | 10 ++++++----
 4 files changed, 26 insertions(+), 8 deletions(-)

Comments

Yordan Karadzhov March 23, 2021, 1:45 p.m. UTC | #1

On 17.03.21 г. 18:40, Michal Sojka wrote:
> This patch series is a follow up to my previous patch for the older
> kernel-shark repo. The first two patches will make life easier for
> distributions like NixOS, the last two patches replace my previous
> patch.
> 
> Note that I didn't add kshark-su-record to a separate component, as
> suggested by Yordan, because it is not necessary. The generated polkit
> action will refer to the correct path and I think it is preferable to
> have all kernelshark binaries in the same directory.
> 
> Michal Sojka (4):
>    kernel-shark: Allow specifying PKG_CONFIG_DIR on cmake command line
>    kernel-shark: Allow specifying TT_FONT_FILE on cmake command line
>    kernel-shark: Allow installing polkit policy separately
>    kernel-shark: Do not use sudo in install_gui.sh + update README

Hi Michal,

Thank you very much for helping us improving KernelShark!
I'm applying all 4 patches.

Best,
Yordan


> 
>   CMakeLists.txt       | 12 ++++++++++--
>   README               |  2 +-
>   build/install_gui.sh | 10 +++++++++-
>   src/CMakeLists.txt   | 10 ++++++----
>   4 files changed, 26 insertions(+), 8 deletions(-)
>