Conversation
|
amadaluzia
left a comment
There was a problem hiding this comment.
Diff LGTM. I will approve these changes once the port used is stable.
3e7f57f to
0867c3b
Compare
|
Fixed the older packages so they can still build. |
8781db3 to
d624c3c
Compare
|
Both of these package appear to be broken for |
|
I was able to reproduce the failures on my machine using |
25dee4a to
b019b72
Compare
amadaluzia
left a comment
There was a problem hiding this comment.
Diff looks once again OK to me, but I will still choose to wait for jdtsmith to bring his fork out of experimental for approval.
|
9044e6d to
549313d
Compare
549313d to
62a0896
Compare
There was a problem hiding this comment.
Was this abandoned, or it is just dormant?
There was a problem hiding this comment.
To my knowledge, it is only dormant. There doesn't appear to be a branch for Emacs 30, so I swapped sources to a fork until something stable comes out of the woodwork.
Reflecting the removal of Linux packages in cd0ef8b
Packages in pkgs.darwin.apple_sdk.frameworks are not needed when using pkgs.darwin.apple-sdk (note the dash).
The macport has optional features up to macOS 14, but apple-sdk defaults to 11.3, so none of them will compile. Upgrading the SDK will not impact functionality on older versions of macOS as there are also runtime checks.
62a0896 to
e69459d
Compare
|
For what it's worth, Mitsuharu Yamamoto did a release of 29.4 earlier this week (https://bitbucket.org/mituharu/emacs-mac/commits/55768fe4781855e8cd4af721c8bf0610ed952e97), so the project isn't dead yet. I don't know what that means for when 30.1 might come along though, as he hasn't merged that into his |
|
If I'm not mistaken, emacs-macport package doesn't expose |
I don’t think this is entirely accurate. I use |
|
Can you kindly merge the latest nixpkgs so we can enable native compilation? |
|
I think it would be better to keep both 29.4 and 30.1 until the latter is considered stable. |
| acl, | ||
| alsa-lib, | ||
| apple-sdk, | ||
| apple-sdk_14, |
There was a problem hiding this comment.
@juuyokka is apple-sdk_14 the minimum sdk version that is needed for this?
There was a problem hiding this comment.
It has additional functionality on newer SDKs. There is no harm in using a newer SDK for a correctly-written application that does runtime availability checks before using newer APIs, and this appears to do so. However, macOS 14 will be the baseline for 25.11 anyway, so it may not be worth explicitly specifying until they implement optional functionality for newer versions.
| emacs30-macport = import ./make-emacs.nix (mkArgs { | ||
| pname = "emacs-mac"; | ||
| version = "29.1"; | ||
| version = "30.1.50"; |
There was a problem hiding this comment.
@juuyokka I think latest version is now at "30.2.50". See jdtsmith/emacs-mac@34f3540
|
|
|
Btw, I've been using a (slightly modified for unstable) variant of this patch for many weeks now, and all has been working well. I also keep it up to date with jdtsmith's latest Eamcs updates. |
|
It has been working well for me also. |
|
Mitsuharu's Emacs fork hasn't been updated for the latest release yet, but there's an experimental fork that rebases his work off of Emacs 30.1. While it's not ready for packaging, I wanted to get started on it in preparation for a stable Mac port. https://github.com/jdtsmith/emacs-mac
On Linux,
emacs28andemacs29have already been removed as of cd0ef8b. Updatingemacs-macportto 30 would allowemacs28-macportandemacs29-macportto be removed as well on Darwin.Since the addition of the new Darwin SDK framework (
apple-sdk) in b904748, there are many extraneous inputs fromapple_sdk.frameworksthat are no longer needed.Thank you @jasonjckn for bringing this fork to my attention and for your help.
Closes #392960
Closes #351042
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.