Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

Change cmake install command. #158

Merged
merged 1 commit into from
Jun 3, 2021
Merged

Change cmake install command. #158

merged 1 commit into from
Jun 3, 2021

Conversation

heiner
Copy link
Contributor

@heiner heiner commented Jun 3, 2021

It seems a recent version of cmake (happened to me for cmake 3.20.0)
no longer installs nethackdir when called with
cmake --build . --target install
Starting from cmake 3.15, the command
cmake --install .
works instead (https://stackoverflow.com/a/56457739/1136208).

Fixes issues along the lines of
$ pip install -e .
$ python -m nle.scripts.play
FileNotFoundError: Couldn't find NetHack installation at '/Users/hnr/miniconda3/envs/extdev/lib/python3.8/site-packages/nle/nethackdir

It seems a recent version of cmake (happened to me for cmake 3.20.0)
no longer installs nethackdir when called with
  cmake --build . --target install
Starting from cmake 3.15, the command
  cmake --install .
works instead (https://stackoverflow.com/a/56457739/1136208).

Fixes issues along the lines of
$ pip install -e .
$ python -m nle.scripts.play
FileNotFoundError: Couldn't find NetHack installation at '/Users/hnr/miniconda3/envs/extdev/lib/python3.8/site-packages/nle/nethackdir
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 3, 2021
@heiner heiner requested a review from cdmatters June 3, 2021 13:52
Copy link
Contributor

@cdmatters cdmatters left a comment

Choose a reason for hiding this comment

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

LGTM

@heiner heiner merged commit 8f6e075 into master Jun 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants