straight not building magit-version.el

[CeleritasCelery]

What's wrong

as described in magit/magit#2070, the package manager should be building magit-version.el but straight is not. This is leading to the error

Cannot determine Magit’s version (error "c:/Users/tjhinckl/AppData/Roaming/.emacs.d/straight/build/magit/magit.el" repo static elpa 

Directions to reproduce

run (magit-version) after loading the package with straight.

Version information

• Emacs version: 26.3
• Operating system: Windows

[progfolio]

related: #72

[raxod502]

That doesn't sound correct to me. I installed Magit using package.el and I don't see any magit-version.el file.

[raxod502]

Also, #72 isn't supported by package.el either. So that's definitely not it.

[progfolio]

Perhaps I was mistaken. I didn't get to look into it too deeply, but I did notice that the file only exists as part of the make process and is removed if it still exists afterward by the magit-version function:

https://github.com/magit/magit/blob/16dd29206f678c79c02674d346ff727eb968ccc2/lisp/magit.el#L411-L423

Apologies if this is just noise.

[CeleritasCelery]

Maybe @tarsius can explain how the package manager is supposed to build magit-version.el. Especially if package.el does not build it.

[tarsius]

Duplicate of #480.

That is labeled as external bug, which I don't really agree with.

In the ideal case we can determine the exact version in use:

$ git describe --tags --dirty 
v2.90.1-971-g9a388a61d

We cannot do that when not running Magit from its Git repository.

When we create a release tarball then we bundle a magit-version.el with that.
When we install using make in the Git repository then we also install a magit-version.el.

In a shallow clone Git isn't of any help:

$ git clone --depth 1 git@github.com:magit/magit.git
$ cd magit
$ git describe --tags --dirty 
fatal: No names found, cannot describe anything.

When installing using package.el then we cannot make it remember the precise version that it installs. So we just ask package.el what version string it has assigned to the version it has installed. That will be less precise than what we get with the methods above but its better than nothing.

It appears we have to do the same for straight.el (because it uses a shallow clone?). Please propose a patch to magit-version that gets this information from straight.el when that's actually what is being used.

[CeleritasCelery]

It appears we have to do the same for straight.el (because it uses a shallow clone?)

Straight does not normally use a shallow clone. I think the issue is that the load path for magit points to the build directory which is not a git repo and only contains the byte-compiled code. Therefore getting the version from the tag doesn’t work.

If we could point the magit-version function to the the magit directory in straight/repos that would work.

[raxod502]

Okay, I just tested and with the following init-file

(defvar bootstrap-version)
(let ((bootstrap-file
       (expand-file-name "straight/repos/straight.el/bootstrap.el" user-emacs-directory))
      (bootstrap-version 5))
  (unless (file-exists-p bootstrap-file)
    (with-current-buffer
        (url-retrieve-synchronously
         "https://github.com/raxod502/straight.el/develop/install.el"
         'silent 'inhibit-cookies)
      (goto-char (point-max))
      (eval-print-last-sexp)))
  (load bootstrap-file nil 'nomessage))

(straight-use-package 'magit)

the command M-x magit-version works just fine, with the following output:

Magit v2.90.1-972-g26da7efb, Git 2.26.1, Emacs 28.0.50, gnu/linux

So I suspect there is something else going on in your configuration @CeleritasCelery.

Now for shallow clones, I agree there's not much can be done on the side of Magit. However, I'm not quite sure @tarsius why the lack of version information should produce a warning at startup/byte-compilation. Why not just report "version info unavailable" or generate a warning at the time that magit-version is run?

I think the reason I marked #480 as external bug was because Magit was producing a warning while being byte-compiled, which I considered to be a bug. Not because it was failing to magically produce a version number out of nothing :)

[CeleritasCelery]

the command M-x magit-version works just fine, with the following output:

Your minimal setup works on linux, but not on windows. I still get

Cannot determine Magit’s version (error "c:/Users/cel/AppData/Roaming/.emacs.d/straight/build/magit/magit.el" repo static elpa dirname)

[dinkonin]

Had the same problem with magit and (setq straight-vc-git-default-clone-depth 1) . Fixed it by specifying :straight (:depth full) in my use-package magit daclaration.

Same problem (and solution) with org, without a full clone I kept getting:

Error running timer ‘require’: (error "Invalid version syntax: ‘fatal: No names found, cannot describe anything.’ (must start with a number)")