Skip to content
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

Improvements in the package installation feature. #504

Merged
merged 1 commit into from
Aug 3, 2016

Conversation

hernando
Copy link

@hernando hernando commented Aug 3, 2016

Support for yum and avoid caching of INSTALL_PACKAGES in CMakeCache.txt.
This changes are motivated by some issues reported by a user trying
to compile Equalizer in RHEL.

@hernando
Copy link
Author

hernando commented Aug 3, 2016

Wait a moment. This code is leaving __pkg_mng set and it appears in the ccmake menu.

find_program(__pkg_mng apt-get)
if(__pkg_mng)
set(__pkg_type DEB)
elseif(__pkg_mng)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

else()

@hernando hernando force-pushed the master branch 2 times, most recently from b003f78 to 1057744 Compare August 3, 2016 09:14
find_program(__pkg_mng apt-get)
if(__pkg_mng)
set(__pkg_type DEB)
else(__pkg_mng)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please don't repeat the condition in the else()

@rdumusc
Copy link

rdumusc commented Aug 3, 2016

looks good otherwise

@hernando hernando force-pushed the master branch 2 times, most recently from 03f69ec to 786de86 Compare August 3, 2016 13:02
# "port install" the known system packages. Will be unset after installation.
# - INSTALL_PACKAGES: command line cache variable which will "apt-get", "yum" or
# "port install" the known system packages. This variable is unset from
# common_find_package_post.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing doc update

Support for yum.
Avoid caching of INSTALL_PACKAGES in CMakeCache.txt (this behaviour was
documented but not implemented).
Improved documentation.

This changes are motivated by some issues reported by a user trying
to compile Equalizer in RHEL.
@rdumusc rdumusc merged commit 4e6d37a into Eyescale:master Aug 3, 2016
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