Conversation
|
Note: I have not built the package, I have low spec hardware. I just wanted to bump the update quickly, so I just changed the name and got the hash from running:
Let me know if there is some mistake or some major changes, as to take more steps, rather than simple name change, so I can close this PR. |
For |
|
Also, don't forget to add the label |
|
Thank you for the clarification! |
But only on unstable, right? |
Should I change that? Then there is no point for
|
The problem is that the version bump should apply to both I would do these changes in a separate PR, pertaining only to But I'm a NixOS novice, so let's hear also to more experienced people. Thanks for the PR, BTW! |
|
Please don't change the main attribute in this PR. Let's just get the bump merged first and then do a follow-up changing the main attribute. That requires release notes and more thorough testing. |
|
I'm newbie at this matter lol, I just opened pr to notify and just change the hash. Glad to contribute to nixos, and it is such a wonderful place. Somebody, or you guys can continue with other steps/PR. |
|
Emacs overlay auto-updated a couple of hours ago and we've been running hydra jobsets for 29.1 on the nix-community hydra in https://hydra.nix-community.org/eval/400298?filter=-unstable&compare=399847&full= with no visible regressions. |
@idlip While I appreciate your enthusiasm you need to apply a bit of testing before opening a PR. While not all checkboxes in the PR template are mandatory you should at least check some of them. I am going to close this one and open my own bump PR as you already state that you are unable to build anything. |
|
Sure, no worries. I understand, and thank you for taking your time on this!
Sorry for the pings and getting your attention.
Next time you see, hopefully I will be upgraded to next version 😅.
…On Sun, Jul 30, 2023, 8:22 PM adisbladis ***@***.***> wrote:
Closed #246169 <#246169>.
—
Reply to this email directly, view it on GitHub
<#246169 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/A34ZJ7OMTUKLQJE3GVOSUXTXSZYLPANCNFSM6AAAAAA25C2QDQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
TIL: The output of To sum up, there is no alternative to a) bump the version and b) run Again, I'll leave up to more experienced people to comment on this. |
Well, you can just build the package without having to eval the entire nixos. cd $WHEREEVER_YOU_CLONED_NIXPKGS
git checkout -b my_local_version upstream/nixos-unstable # less rebuilding when working against unstable compared to master
# now bump the version, and then
nix build -f . emacs-29 # build whatever thing you are working on - it will show you the proper hash hereMuch faster than running |
|
I did not know that, Thank you for the tip.
TIL.
…On Mon, Jul 31, 2023, 9:08 AM Peter Hoeg ***@***.***> wrote:
To sum up, there is no alternative to a) bump the version and b) run
nixos-rebuild test or switch and obtain the expected hash from there.
Well, you can just build the package without having to eval the entire
nixos.
cd $WHEREEVER_YOU_CLONED_NIXPKGS
git checkout -b my_local_version upstream/nixos-unstable # less rebuilding when working against unstable compared to master# now bump the version, and then
nix build -f . emacs-29 # build whatever thing you are working on - it will show you the proper hash here
Much faster than running nixos-rebuild.
—
Reply to this email directly, view it on GitHub
<#246169 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A34ZJ7K4P3LF44KTYXMQRHTXS4SDNANCNFSM6AAAAAA25C2QDQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
I will write a tutorial about this. |
Description of changes
Changelog: https://git.savannah.gnu.org/cgit/emacs.git/tree/etc/NEWS?h=emacs-29
Release Notes: https://lists.gnu.org/archive/html/emacs-devel/2023-07/msg00879.html
Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)