From mboxrd@z Thu Jan 1 00:00:00 1970 From: ng0 Subject: Re: [PATCH] gnu: Add vim-full. Date: Mon, 17 Oct 2016 21:07:52 +0000 Message-ID: <87bmyi65w7.fsf@we.make.ritual.n0.is> References: <20161015203841.19904-2-ng0@we.make.ritual.n0.is> <20161017163240.15227-1-ng0@we.make.ritual.n0.is> <20161017163240.15227-2-ng0@we.make.ritual.n0.is> <878ttm970g.fsf@duckhunt.i-did-not-set--mail-host-address--so-tickle-me> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:52159) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bwF8x-0002sK-1c for guix-devel@gnu.org; Mon, 17 Oct 2016 17:08:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bwF8t-0007nx-2E for guix-devel@gnu.org; Mon, 17 Oct 2016 17:08:23 -0400 Received: from aibo.runbox.com ([91.220.196.211]:39572) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1bwF8s-0007l5-Ru for guix-devel@gnu.org; Mon, 17 Oct 2016 17:08:18 -0400 In-Reply-To: <878ttm970g.fsf@duckhunt.i-did-not-set--mail-host-address--so-tickle-me> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Marius Bakke , guix-devel@gnu.org Hi, thanks for reviewing. Marius Bakke writes: > ng0 writes: > >> * gnu/packages/vim.scm (vim-full): New variable. >> * gnu/packages/patches/vim-8.0.0003.patch: New file. >> * gnu/packages/patches/vim-8.0.0004.patch: New file. >> * gnu/packages/patches/vim-8.0.0005.patch: New file. >> * gnu/local.mk (dist_patch_DATA): Add patches. >> --- >> gnu/local.mk | 3 ++ >> gnu/packages/patches/vim-8.0.0003.patch | 87 +++++++++++++++++++++++++++++++++ >> gnu/packages/patches/vim-8.0.0004.patch | 60 +++++++++++++++++++++++ >> gnu/packages/patches/vim-8.0.0005.patch | 45 +++++++++++++++++ >> gnu/packages/vim.scm | 81 ++++++++++++++++++++++++++++++ >> 5 files changed, 276 insertions(+) >> create mode 100644 gnu/packages/patches/vim-8.0.0003.patch >> create mode 100644 gnu/packages/patches/vim-8.0.0004.patch >> create mode 100644 gnu/packages/patches/vim-8.0.0005.patch > > [...] > >> + >> +(define-public vim-full >> + (package >> + (inherit vim) >> + (name "vim-full") >> + (version (package-version vim)) >> + (source >> + (origin >> + (method url-fetch) >> + (uri (string-append "ftp://ftp.vim.org/pub/vim/unix/vim-" >> + version ".tar.bz2")) >> + (sha256 >> + (base32 >> + "1s34rf8089klsbdx5l0iw7vjymir0kzfrx8wb30s31wygnq29axc")) >> + ;; Patches need to be applied sequentially. 8.0 is the release of >> + ;; vim version 8.0.0002 so we start at 8.0.0003 >> + (patches (search-patches "vim-8.0.0003.patch" >> + "vim-8.0.0004.patch" >> + "vim-8.0.0005.patch")))) > > Shouldn't these patches be applied on the base vim package? Then > "vim-full" can just inherit the source from there. I also think > "version" should be adjusted to match the patch level. No (for explanation see the other emails I've sent in the earlier thread). I agree on the version, I will update that. > Other than that this package works for me. Perhaps the description could > be updated to mention that this also includes the graphical "gvim" if > anyone is searching for that. Okay, I will include that. > Could you separate this out to two patches, one that updates "vim" and > one that adds "vim-full"? See above. But I think hardly anyone will search for it, so here's the reason: The patches are only needed for what vim-full does and for that only to make the tests succeed. I don't see any rational point in including patches which will make a testsuite succeed which is not run by the package you suggest I apply them to. If this is extended in the future (say vim fixes some serious bug where we need to sequentially apply the patches up to that), feel free to change it to vim. I just need to make the testsuite succeed, which the patches even comment afaik. If it would extend any functionality our standard (minimal) vim has, I would apply them, but as far as I see it this is not the case. Correct me if I'm wrong. > Thanks! >