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

CMake 3.17 or higher is required. The repository asks for version 3.13.4 #144

Closed
Rubiel1 opened this issue Feb 13, 2024 · 1 comment · Fixed by #145
Closed

CMake 3.17 or higher is required. The repository asks for version 3.13.4 #144

Rubiel1 opened this issue Feb 13, 2024 · 1 comment · Fixed by #145
Labels
documentation Improvements or additions to documentation

Comments

@Rubiel1
Copy link

Rubiel1 commented Feb 13, 2024

Prerequisites

Before submitting your issue, please ensure the following:

  • [ X] I am running the latest version of PowerInfer. Development is rapid, and as of now, there are no tagged versions.
  • [ X] I have carefully read and followed the instructions in the README.md.
  • [ X] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).

Expected Behavior

Build PowerInfer

Please provide a detailed written description of what you were trying to do, and what you expected PowerInfer to do.

1.- I run the line """PowerInfer]$ cmake -S . -B build -DLLAMA_CUBLAS=ON""" and I was expected PowerInfer to be build.

Current Behavior

Please provide a detailed written description of what PowerInfer did, instead.

I received the output: """CMake Error at CMakeLists.txt:252 (cmake_minimum_required):
CMake 3.17 or higher is required. You are running version 3.13.4
"""

But the github repository asks for CMake 3.13+

Failure Information (for bugs)

Please help provide information about the failure / bug.

It asks me for CMake 3.17 when the github repository asks for CMake 3.13.

Steps to Reproduce

Please provide detailed steps for reproducing the issue. We are not sitting in front of your screen, so the more detail the better.

  1. step 1 Open CMakeList.txt
  2. step 2 Go to line 252
  3. step 3 "if (LLAMA_CUBLAS)
    cmake_minimum_required(VERSION 3.17)"
@Rubiel1 Rubiel1 added the bug-unconfirmed Unconfirmed bugs label Feb 13, 2024
@hodlen hodlen added documentation Improvements or additions to documentation and removed bug-unconfirmed Unconfirmed bugs labels Feb 18, 2024
@hodlen
Copy link
Collaborator

hodlen commented Feb 18, 2024

Thank you for bringing this discrepancy to our attention. We will update our documentation to match the correct CMake version requirement ASAP.

@hodlen hodlen linked a pull request Feb 18, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants