Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

uninstall: use execute_process() instead of exec_program() #5285

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
Loading
from

Conversation

Rtoax
Copy link
Contributor

@Rtoax Rtoax commented Apr 16, 2025

exec_program() is deprecated since 3.28, see: 'cmake --help-policy CMP0153', and cmake support execute_process() since CMake 2.6.0.

$ sudo make uninstall
....
-- Uninstalling /usr/share/bcc/examples/networking/distributed_bridge/tunnel.py
CMake Warning (dev) at /home/rongtao/Git/bcc/build/CmakeUninstall.cmake:14 (exec_program):
  Policy CMP0153 is not set: The exec_program command should not be called.
  Run "cmake --help-policy CMP0153" for policy details.  Use the cmake_policy
  command to set the policy and suppress this warning.

  Use execute_process() instead.
Call Stack (most recent call first):
  /home/rongtao/Git/bcc/build/CmakeUninstall.cmake:28 (UninstallManifest)
This warning is for project developers.  Use -Wno-dev to suppress it.

@Rtoax Rtoax force-pushed the patch-92-cmake-uninstall branch 2 times, most recently from a89c229 to 4f26e42 Compare May 1, 2025 00:25
@Rtoax
Copy link
Contributor Author

Rtoax commented May 1, 2025

Rebase to master (commit 5043759)

@ekyooo
Copy link
Contributor

ekyooo commented May 4, 2025

LGTM

@Rtoax Rtoax force-pushed the patch-92-cmake-uninstall branch from 4f26e42 to 8761399 Compare May 12, 2025 06:51
@Rtoax
Copy link
Contributor Author

Rtoax commented May 12, 2025

rebase to master.

exec_program() is deprecated since 3.28, see: 'cmake --help-policy CMP0153',
and cmake support execute_process() since CMake 2.6.0.

    $ sudo make uninstall
    ....
    -- Uninstalling /usr/share/bcc/examples/networking/distributed_bridge/tunnel.py
    CMake Warning (dev) at /home/rongtao/Git/bcc/build/CmakeUninstall.cmake:14 (exec_program):
      Policy CMP0153 is not set: The exec_program command should not be called.
      Run "cmake --help-policy CMP0153" for policy details.  Use the cmake_policy
      command to set the policy and suppress this warning.

      Use execute_process() instead.
    Call Stack (most recent call first):
      /home/rongtao/Git/bcc/build/CmakeUninstall.cmake:28 (UninstallManifest)
    This warning is for project developers.  Use -Wno-dev to suppress it.

Link: https://cmake.org/cmake/help/v3.12/command/execute_process.html
Signed-off-by: Rong Tao <rongtao@cestc.cn>
@Rtoax Rtoax force-pushed the patch-92-cmake-uninstall branch from 8761399 to 884cb29 Compare May 15, 2025 00:42
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.

2 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.