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

use execute_process instead of exec_program in FindAPACHE.cmake #4948

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@jisakiel

jisakiel commented Jul 1, 2014

With this patch the debian bug 707109 can be avoided; as apxs2 returns noise in debian wheezy, using exec_program the variable gets assigned the STDERR instead of the proper value.

@jisakiel

This comment has been minimized.

Show comment
Hide comment

jisakiel commented Jul 1, 2014

tbonfort added a commit that referenced this pull request Aug 26, 2014

use execute_process instead of exec_program in FindAPACHE.cmake (#4948)
also remove trailing whitespaces as they break builds
@tbonfort

This comment has been minimized.

Show comment
Hide comment
@tbonfort

tbonfort Aug 26, 2014

Member

squashed and cherry-picked to the stable branch in 644d750

Member

tbonfort commented Aug 26, 2014

squashed and cherry-picked to the stable branch in 644d750

@tbonfort tbonfort closed this Aug 26, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment