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

fix: 'doom sync' generates autoload files for symbolic link files #7826

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kilesduli
Copy link

  • lisp/lib/autoloads.el(doom-autoloads--scan) Remove invoke file-truename of file, keeping symbolic from being converted to a real path.
  • lisp/doom-modules.el(doom-module-from-path) Replace file-in-directory-p with string-match to determine the module to which the file belongs.

Fix: #7821


  • I searched the issue tracker and this hasn't been PRed before.
  • My commits conform to the git conventions.
  • Any relevant issues or PRs have been linked to.

@kilesduli kilesduli requested a review from a team as a code owner April 16, 2024 06:58
@hlissner
Copy link
Member

hlissner commented Apr 16, 2024

I vaguely recall there being a good reason why I didn't opt for string comparison in doom-module-from-path (besides the paths in the autoloads file, I mean). I'll dig through my notes in the morning and get back to this soon.

@hlissner hlissner added is:bug Something isn't working as intended module:core Relevant to Doom core labels Apr 16, 2024
@hlissner hlissner added this to the modules v24.05 milestone Apr 16, 2024
* lisp/lib/autoloads.el(doom-autoloads--scan) Remove invoke
`file-truename` of file, keeping symbolic from being converted to a real
path.
* lisp/doom-modules.el(doom-module-from-path) Replace
`file-in-directory-p` with `string-match` to determine the module to
which the file belongs.

Fix: doomemacs#7821
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is:bug Something isn't working as intended module:core Relevant to Doom core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

doom-module-from-path couldn't handle symbolic links correctly.
2 participants