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

Tag 1.4.1 fails to build from source tarball #7355

Open
3 tasks done
bassmadrigal opened this issue Oct 30, 2024 · 0 comments · May be fixed by #7356
Open
3 tasks done

Tag 1.4.1 fails to build from source tarball #7355

bassmadrigal opened this issue Oct 30, 2024 · 0 comments · May be fixed by #7356

Comments

@bassmadrigal
Copy link

Before posting

Please follow the steps below and check the boxes with [x] once you did the step.

  • I checked the issue tracker for similar issues
  • I checked the changelog if the issue is already resolved
  • I tried the latest Clementine build from here

System information

Please provide information about your system and the version of Clementine used.

  • Operating System: Slackware64 15.0
  • Clementine version: 1.4.1 tag ( a809096)

Expected behaviour / actual behaviour

Expected: Tarballs tagged as a release version and downloaded via github tags should build without issue.

Actual: CMake configure fails when using the tagged 1.4.1 "release" due to forcing detecting a git revision in cmake/Version.cmake, tracing the breakage to commmit 5e5b888.

I could not find a build flag that allowed me to bypass the git revision detection (but I'm not super strong with cmake).

Steps to reproduce the problem (only for bugs)

Download the 1.4.1 tag source tarball
wget --content-disposition https://github.com/clementine-player/Clementine/archive/refs/tags/1.4.1/Clementine-1.4.1.tar.gz
Extract source
tar xvf Clementine-1.4.1.tar.gz
cd into source, create and cd to build/ dir and run cmake
cd Clementine-1.4.1; mkdir build; cd build; cmake ..

Build will fail with

fatal: not a git repository (or any of the parent directories): .git
CMake Error at cmake/Version.cmake:136 (message):
  git describe failed with code 128:
Call Stack (most recent call first):
  CMakeLists.txt:10 (include)


CMake Error at cmake/Version.cmake:141 (string):
  string sub-command REGEX, mode REPLACE needs at least 6 arguments total to
  command.
Call Stack (most recent call first):
  CMakeLists.txt:10 (include)


CMake Error at cmake/Version.cmake:145 (message):
  Failed to parse git revision string ''
Call Stack (most recent call first):
  CMakeLists.txt:10 (include)


-- Configuring incomplete, errors occurred!
See also "/tmp/SBo/Clementine-1.4.1/build/CMakeFiles/CMakeOutput.log".

I have an incoming PR that provides a way to bypass this, however, it requires changing the INCLUDE_GIT_REVISION variable within that file to OFF, which I was doing via a sed command in my build script.


Also, unrelated to this issue, the 1.4.1 tagged tarball (and the master branch) still have cmake/Version.cmake with versioned with 1.4.0rc2.

bassmadrigal added a commit to bassmadrigal/Clementine that referenced this issue Oct 30, 2024
@bassmadrigal bassmadrigal linked a pull request Oct 31, 2024 that will close this issue
bassmadrigal added a commit to bassmadrigal/Clementine that referenced this issue Oct 31, 2024
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 a pull request may close this issue.

1 participant