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

Slight improvement to magic snippet #1064

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

Conversation

dabrahams
Copy link

Should work on Emacs >= 23.1
I don't think this needs a changelog, but please LMK if you disagree.

Should work on Emacs  >= 23.1
Copy link
Member

@raxod502 raxod502 left a comment

Choose a reason for hiding this comment

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

The minimum supported version of Emacs for straight.el is Emacs 25.1. In particular, Emacs versions 24.3 and below were never supported, even in 2017 when the project was first begun. Does it change anything to add backwards compatibility here?

@raxod502
Copy link
Member

Oh, I see. You're not adding backwards compatibility, you're just using a more readable function instead of expand-file-name.

Copy link
Member

@raxod502 raxod502 left a comment

Choose a reason for hiding this comment

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

Ok, this looks good, but can you update also the other instances of the bootstrap snippet with the same change, and also bump bootstrap-version from 6 to 7?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants