Opened 8 years ago
Closed 8 years ago
#52706 closed update (fixed)
emacs-app @24.5: update to 25.1
| Reported by: | isomarcte (David Strawn) | Owned by: | humem (humem) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | haspatch | Cc: | drkp (Dan Ports) |
| Port: | emacs-app |
Description
I have upgraded the emacs-app port for emacs 25.1. This includes the porting the inline patch for emacs 25.1.
I am a bit confused about why the inline variant is on by default, so in the attached file I disabled it by default. It seems, at least to me, to be surprising for the average user. Normally, I'd expect the default install of a port to be as close as possible to the upstream codebase, with variants adding customizations or extra features. This does seem to be what the macports guide itself recommends, https://guide.macports.org/#development.variants.default .
Anyway, I hope it is helpful, let me know if I can change/fix anything. I have been running the 25.1 build locally for the past week using the given Portfile, off and on with the inline patch, and have thus far had no problems.
Attachments (2)
Change History (8)
Changed 8 years ago by isomarcte (David Strawn)
| Attachment: | patch-inline-25.1-20151017.diff added |
|---|
comment:1 Changed 8 years ago by larryv (Lawrence Velázquez)
| Cc: | hum@… removed |
|---|---|
| Owner: | changed from macports-tickets@… to hum@… |
| Status: | new → assigned |
| Summary: | emacs-app 25.1 → emacs-app @24.5: update to 25.1 |
| Version: | 2.3.4 |
Replying to isomarcte:
I am a bit confused about why the inline variant is on by default, so in the attached file I disabled it by default. It seems, at least to me, to be surprising for the average user. Normally, I'd expect the default install of a port to be as close as possible to the upstream codebase, with variants adding customizations or extra features. This does seem to be what the macports guide itself recommends, https://guide.macports.org/#development.variants.default .
It’s up to the maintainer. We do not recommend or discourage adhering to the “upstream” configuration, although we do discourage overly fussy variants. Many maintainers choose to enable optional features without providing variants at all.
Since you’re updating version, you must remove the revision line.
Changed 8 years ago by isomarcte (David Strawn)
| Attachment: | Portfile-emacs-app.diff added |
|---|
Updated Portfile
comment:3 Changed 8 years ago by isomarcte (David Strawn)
Looks like I also put the wrong date on the inline patch...definitely is not 2015...
I don't see any way to rename a file, just replace them. So I am leaving it as is.
comment:4 Changed 8 years ago by iqgrande
Greetings:
I just opened up #52831 and I think it may be helped by emacs-app getting updated as well.
Kind regards, Anthony
comment:5 Changed 8 years ago by drkp (Dan Ports)
| Cc: | drkp added |
|---|
comment:6 Changed 8 years ago by drkp (Dan Ports)
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
I updated emacs-app in https://github.com/macports/macports-ports/commit/e7869a0e0636b6ab2069d1466f86c2834b739805
Updated inline patch