Skip to content

Commit

Permalink
vim: 8.0.1150 -> 8.0.1245
Browse files Browse the repository at this point in the history
This update contains a lot of fixes that are too much to be summarized
here, so here is the upstream changelog (basically "git log"):

https://github.com/vim/vim/commits/v8.0.1250

The main reason for this bump is that I got annoyed by a bug that was
fixed in upstream version 8.0.1194, which caused a race condition during
vim startup when it's trying to retrieve background colors from the
terminal.

Sometimes it could happen that random commands are executed at Vim
startup (typically pasting the "" buffer) and after bisecting I've found
out that version 8.0.1194 indeed fixed this problem.

The reason why I'm updating to version 8.0.1250 is that when looking
through the Git log it contains a whole lot of fixes but no new
features, so I'd assume it's safe to upgrade.

I've tested all packages that depend on Vim and they still succeed
building. In addition to that I've used the new version for a couple of
hours without any issue.

Signed-off-by: aszlig <aszlig@nix.build>
Cc: @lovek323, @LnL7, @vaibhavsagar
  • Loading branch information
aszlig committed Nov 3, 2017
1 parent 9a15344 commit 74260a4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkgs/applications/editors/vim/common.nix
@@ -1,12 +1,12 @@
{ lib, fetchFromGitHub }:
rec {
version = "8.0.1150";
version = "8.0.1245";

src = fetchFromGitHub {
owner = "vim";
repo = "vim";
rev = "v${version}";
sha256 = "1k1qkmb2jbymqikrp99q1yjagdf508xzabrw7b08dlh926b2v23j";
sha256 = "10nh8b2jgfbq4xvha0d2qashvdyqzvq96w8k6rl6793d8y7f3z8s";
};

enableParallelBuilding = true;
Expand Down

4 comments on commit 74260a4

@LnL7
Copy link
Member

@LnL7 LnL7 commented on 74260a4 Nov 4, 2017

Choose a reason for hiding this comment

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

@aszlig Looks fine to me, perhaps we should also backport this?

@aszlig
Copy link
Member Author

@aszlig aszlig commented on 74260a4 Nov 4, 2017

Choose a reason for hiding this comment

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

@LnL7: Yep, but I'd give it a few days of testing in master to be 100% sure :-)

@globin
Copy link
Member

@globin globin commented on 74260a4 Nov 4, 2017

Choose a reason for hiding this comment

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

πŸ‘

@aszlig
Copy link
Member Author

@aszlig aszlig commented on 74260a4 Nov 15, 2017

Choose a reason for hiding this comment

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

@LnL7: Backported in 33f482e and ccb2cc7.

Please sign in to comment.