[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add vim-full.
From: |
Marius Bakke |
Subject: |
Re: [PATCH] gnu: Add vim-full. |
Date: |
Mon, 17 Oct 2016 19:15:27 +0100 |
ng0 <address@hidden> 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.
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.
Could you separate this out to two patches, one that updates "vim" and
one that adds "vim-full"?
Thanks!
- [PATCH] Add vim-full (this time with the patches included), ng0, 2016/10/15
- [PATCH] gnu: Add vim-full., ng0, 2016/10/15
- vim-full v2, ng0, 2016/10/17
- [PATCH] gnu: Add vim-full., ng0, 2016/10/17
- Re: [PATCH] gnu: Add vim-full.,
Marius Bakke <=
- Re: [PATCH] gnu: Add vim-full., ng0, 2016/10/17
- Re: [PATCH] gnu: Add vim-full., Marius Bakke, 2016/10/17
- Re: [PATCH] gnu: Add vim-full., ng0, 2016/10/17
- Re: [PATCH] gnu: Add vim-full., Marius Bakke, 2016/10/18
- Re: [PATCH] gnu: Add vim-full., Kei Kebreau, 2016/10/17
- Re: [PATCH] gnu: Add vim-full., ng0, 2016/10/17