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
nano: 2.9.8 -> 3.0 #46444
nano: 2.9.8 -> 3.0 #46444
Conversation
Changelog: http://lists.gnu.org/archive/html/info-nano/2018-09/msg00000.html Also updated the nix highlighting for nano
Success on x86_64-linux (full log) Attempted: nano Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: nano Partial log (click to expand)
|
I'd say that this is worth backporting to 18.09 as well, since it's not released yet 🙂 |
Thanks! Why do you think this should be backported? |
@timokau Well, normally it wouldn't. But since 18.09 isn't out yet it may be fine? And I see one big reason, eventual big fixes for eventual problems with nano in the future will probably be made on the 3.x release series. This is a guess though since I'm not involved with that project. |
Well the reason we have a feature-freeze ~one month before release is to make sure everything that will be released is thouroughly tested. Regressions should be avoided, which is why I think we usually only include bug fixes and minor upgrades from that point. That is very important for libraries, however I think for end-user applications we can be a bit more lenient. Your argument is a good one. @samueldr what do you think? |
Looking at (an elided version of) ofborg's diff:
CVS has This looks like it won't be an issue backporting. Additionally, security and minor releases probably won't happen for versions earlier. |
Alright, thanks for having a look at this. For reference: backported in 8468a4b. |
Motivation for this change
Changelog: http://lists.gnu.org/archive/html/info-nano/2018-09/msg00000.html
Also updated the nix highlighting for nano
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)cc @jgeerds @joachifm