unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: vim: Update to 8.0.0047.
@ 2016-10-27 13:29 Marius Bakke
  2016-10-27 20:20 ` ng0
  2016-10-28  6:02 ` Ricardo Wurmus
  0 siblings, 2 replies; 7+ messages in thread
From: Marius Bakke @ 2016-10-27 13:29 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 351 bytes --]


The official vim mirrors only carries major versions and individual
patches, no up-to-date releases. This patch changes the source uri to
the tagged github releases instead of downloading all 40+ (so far)
patches individually.

I'm not very happy about changing to a third-party source, but IMO it
beats keeping track of the frequent patches. WDYT?


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-vim-Update-to-8.0.0047.patch --]
[-- Type: text/x-patch, Size: 1306 bytes --]

From 786dc474126431c8d2b9b97644dd629f5f7aed37 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Thu, 27 Oct 2016 14:14:40 +0100
Subject: [PATCH] gnu: vim: Update to 8.0.0047.

* gnu/packages/vim.scm (vim): Update to 8.0.0047.
[source]: Change to github release archive.
---
 gnu/packages/vim.scm | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/vim.scm b/gnu/packages/vim.scm
index 4ee8c7d..32f9e62 100644
--- a/gnu/packages/vim.scm
+++ b/gnu/packages/vim.scm
@@ -48,14 +48,15 @@
 (define-public vim
   (package
     (name "vim")
-    (version "8.0")
+    (version "8.0.0047")
     (source (origin
              (method url-fetch)
-             (uri (string-append "ftp://ftp.vim.org/pub/vim/unix/vim-"
-                                 version ".tar.bz2"))
+             (uri (string-append "https://github.com/vim/vim/archive/v"
+                                 version ".tar.gz"))
+             (file-name (string-append name "-" version ".tar.gz"))
              (sha256
               (base32
-               "1s34rf8089klsbdx5l0iw7vjymir0kzfrx8wb30s31wygnq29axc"))))
+               "1l24pbc06970crk513a98vlr77sp6wq6d0ylviwk9nhvfz4p0cff"))))
     (build-system gnu-build-system)
     (arguments
      `(#:test-target "test"
-- 
2.10.1


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] gnu: vim: Update to 8.0.0047.
  2016-10-27 13:29 [PATCH] gnu: vim: Update to 8.0.0047 Marius Bakke
@ 2016-10-27 20:20 ` ng0
  2016-10-28  6:02 ` Ricardo Wurmus
  1 sibling, 0 replies; 7+ messages in thread
From: ng0 @ 2016-10-27 20:20 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

Marius Bakke <mbakke@fastmail.com> writes:

> The official vim mirrors only carries major versions and individual
> patches, no up-to-date releases. This patch changes the source uri to
> the tagged github releases instead of downloading all 40+ (so far)
> patches individually.
>
> I'm not very happy about changing to a third-party source, but IMO it
> beats keeping track of the frequent patches. WDYT?

Someone (you?) previously told me that this is not third-party
but encouraged source by upstream itself. My other system
(Gentoo) pulls vim from github aswell so this seems okay. I guess
more systems do it this way.

> From 786dc474126431c8d2b9b97644dd629f5f7aed37 Mon Sep 17 00:00:00 2001
> From: Marius Bakke <mbakke@fastmail.com>
> Date: Thu, 27 Oct 2016 14:14:40 +0100
> Subject: [PATCH] gnu: vim: Update to 8.0.0047.
>
> * gnu/packages/vim.scm (vim): Update to 8.0.0047.
> [source]: Change to github release archive.
> ---
>  gnu/packages/vim.scm | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/gnu/packages/vim.scm b/gnu/packages/vim.scm
> index 4ee8c7d..32f9e62 100644
> --- a/gnu/packages/vim.scm
> +++ b/gnu/packages/vim.scm
> @@ -48,14 +48,15 @@
>  (define-public vim
>    (package
>      (name "vim")
> -    (version "8.0")
> +    (version "8.0.0047")
>      (source (origin
>               (method url-fetch)
> -             (uri (string-append "ftp://ftp.vim.org/pub/vim/unix/vim-"
> -                                 version ".tar.bz2"))
> +             (uri (string-append "https://github.com/vim/vim/archive/v"
> +                                 version ".tar.gz"))
> +             (file-name (string-append name "-" version ".tar.gz"))
>               (sha256
>                (base32
> -               "1s34rf8089klsbdx5l0iw7vjymir0kzfrx8wb30s31wygnq29axc"))))
> +               "1l24pbc06970crk513a98vlr77sp6wq6d0ylviwk9nhvfz4p0cff"))))
>      (build-system gnu-build-system)
>      (arguments
>       `(#:test-target "test"

-- 
♥Ⓐ  ng0  | ng0.chaosnet.org

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] gnu: vim: Update to 8.0.0047.
  2016-10-27 13:29 [PATCH] gnu: vim: Update to 8.0.0047 Marius Bakke
  2016-10-27 20:20 ` ng0
@ 2016-10-28  6:02 ` Ricardo Wurmus
  2016-10-28  7:19   ` ng0
  2016-10-28 16:55   ` Marius Bakke
  1 sibling, 2 replies; 7+ messages in thread
From: Ricardo Wurmus @ 2016-10-28  6:02 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel


Marius Bakke <mbakke@fastmail.com> writes:

> The official vim mirrors only carries major versions and individual
> patches, no up-to-date releases. This patch changes the source uri to
> the tagged github releases instead of downloading all 40+ (so far)
> patches individually.
>
> I'm not very happy about changing to a third-party source, but IMO it
> beats keeping track of the frequent patches. WDYT?

I’m also not happy about using a third-party mirror for vim.  Can we be
sure that this is updated consistently and in time?  (Is this done
automatically?)

Guix can handle downloading patches, so there’s really no need for
switching in my opinion.  “gnu/packages/bash.scm” could be used as a
reference for how to deal with a large number of patches.

~~ Ricardo

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] gnu: vim: Update to 8.0.0047.
  2016-10-28  6:02 ` Ricardo Wurmus
@ 2016-10-28  7:19   ` ng0
  2016-10-28 16:55   ` Marius Bakke
  1 sibling, 0 replies; 7+ messages in thread
From: ng0 @ 2016-10-28  7:19 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

Ricardo Wurmus <rekado@elephly.net> writes:

> Marius Bakke <mbakke@fastmail.com> writes:
>
>> The official vim mirrors only carries major versions and individual
>> patches, no up-to-date releases. This patch changes the source uri to
>> the tagged github releases instead of downloading all 40+ (so far)
>> patches individually.
>>
>> I'm not very happy about changing to a third-party source, but IMO it
>> beats keeping track of the frequent patches. WDYT?
>
> I’m also not happy about using a third-party mirror for vim.  Can we be
> sure that this is updated consistently and in time?  (Is this done
> automatically?)
>
> Guix can handle downloading patches, so there’s really no need for
> switching in my opinion.  “gnu/packages/bash.scm” could be used as a
> reference for how to deal with a large number of patches.
>
> ~~ Ricardo

Last time I looked at the method used in bash.scm it was not
obvious how the method should be used, and from my impression in
irc it wasn't obvious to others when they tried to update bash or
what it was.

If vim gets an equal method, can we please document how the
patches are downloaded (in other words: how you execute the
download of scripts to store) in the module(s) or somewhere else?

In addition to my first message: I think we should not rely too
much on github.com, when it goes down again we have no fallback
solution.
-- 
♥Ⓐ  ng0  | ng0.chaosnet.org

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] gnu: vim: Update to 8.0.0047.
  2016-10-28  6:02 ` Ricardo Wurmus
  2016-10-28  7:19   ` ng0
@ 2016-10-28 16:55   ` Marius Bakke
  2016-10-28 17:12     ` Ricardo Wurmus
  1 sibling, 1 reply; 7+ messages in thread
From: Marius Bakke @ 2016-10-28 16:55 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

[-- Attachment #1: Type: text/plain, Size: 1651 bytes --]

Ricardo Wurmus <rekado@elephly.net> writes:

> Marius Bakke <mbakke@fastmail.com> writes:
>
>> The official vim mirrors only carries major versions and individual
>> patches, no up-to-date releases. This patch changes the source uri to
>> the tagged github releases instead of downloading all 40+ (so far)
>> patches individually.
>>
>> I'm not very happy about changing to a third-party source, but IMO it
>> beats keeping track of the frequent patches. WDYT?
>
> I’m also not happy about using a third-party mirror for vim.  Can we be
> sure that this is updated consistently and in time?  (Is this done
> automatically?)

When I sent this, the 0047 release was about an hour old and the
corresponding patch was not yet available on ftp.vim.org. So it seems
to be the other way around. The "vim" organization on Github is endorsed
on vim.org and maintained by Bram Molenaar himself.

> Guix can handle downloading patches, so there’s really no need for
> switching in my opinion.  “gnu/packages/bash.scm” could be used as a
> reference for how to deal with a large number of patches.

vim-7.4 ended at 2367 patches[0]. 8.0 is currently at 51 (four since
yesterday!), whereas bash has 42, so they are not really comparable. I
think vim would rarely be updated, if it required downloading and
creating potentially hundreds of patch references at once.

They are also not signed, though we could rely on the MD5SUMS file.

With this information, do you still think a custom patch importer is
better? I don't really mind either way, but someone needs to make it :)

0: http://ftp.vim.org/pub/vim/patches/7.4/

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 454 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] gnu: vim: Update to 8.0.0047.
  2016-10-28 16:55   ` Marius Bakke
@ 2016-10-28 17:12     ` Ricardo Wurmus
  2016-10-28 19:57       ` Marius Bakke
  0 siblings, 1 reply; 7+ messages in thread
From: Ricardo Wurmus @ 2016-10-28 17:12 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel


Marius Bakke <mbakke@fastmail.com> writes:

> Ricardo Wurmus <rekado@elephly.net> writes:
>
>> Marius Bakke <mbakke@fastmail.com> writes:
>>
>>> The official vim mirrors only carries major versions and individual
>>> patches, no up-to-date releases. This patch changes the source uri to
>>> the tagged github releases instead of downloading all 40+ (so far)
>>> patches individually.
>>>
>>> I'm not very happy about changing to a third-party source, but IMO it
>>> beats keeping track of the frequent patches. WDYT?
>>
>> I’m also not happy about using a third-party mirror for vim.  Can we be
>> sure that this is updated consistently and in time?  (Is this done
>> automatically?)
>
> When I sent this, the 0047 release was about an hour old and the
> corresponding patch was not yet available on ftp.vim.org. So it seems
> to be the other way around. The "vim" organization on Github is endorsed
> on vim.org and maintained by Bram Molenaar himself.

Ah, cool.  This makes all the difference, IMO.

>> Guix can handle downloading patches, so there’s really no need for
>> switching in my opinion.  “gnu/packages/bash.scm” could be used as a
>> reference for how to deal with a large number of patches.
>
> vim-7.4 ended at 2367 patches[0]. 8.0 is currently at 51 (four since
> yesterday!), whereas bash has 42, so they are not really comparable. I
> think vim would rarely be updated, if it required downloading and
> creating potentially hundreds of patch references at once.
>
> They are also not signed, though we could rely on the MD5SUMS file.
>
> With this information, do you still think a custom patch importer is
> better? I don't really mind either way, but someone needs to make it :)

No, you convinced me :)  Thanks for your patience!

~~ Ricardo

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] gnu: vim: Update to 8.0.0047.
  2016-10-28 17:12     ` Ricardo Wurmus
@ 2016-10-28 19:57       ` Marius Bakke
  0 siblings, 0 replies; 7+ messages in thread
From: Marius Bakke @ 2016-10-28 19:57 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

[-- Attachment #1: Type: text/plain, Size: 949 bytes --]

Ricardo Wurmus <rekado@elephly.net> writes:

>>> Guix can handle downloading patches, so there’s really no need for
>>> switching in my opinion.  “gnu/packages/bash.scm” could be used as a
>>> reference for how to deal with a large number of patches.
>>
>> vim-7.4 ended at 2367 patches[0]. 8.0 is currently at 51 (four since
>> yesterday!), whereas bash has 42, so they are not really comparable. I
>> think vim would rarely be updated, if it required downloading and
>> creating potentially hundreds of patch references at once.
>>
>> They are also not signed, though we could rely on the MD5SUMS file.
>>
>> With this information, do you still think a custom patch importer is
>> better? I don't really mind either way, but someone needs to make it :)
>
> No, you convinced me :)  Thanks for your patience!

Thanks for questioning my oft-rushed conclusions! :)
I updated vim in a9afb956b3232ccfb7deaf3feed02b60a12d65bf.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 454 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2016-10-28 19:57 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-27 13:29 [PATCH] gnu: vim: Update to 8.0.0047 Marius Bakke
2016-10-27 20:20 ` ng0
2016-10-28  6:02 ` Ricardo Wurmus
2016-10-28  7:19   ` ng0
2016-10-28 16:55   ` Marius Bakke
2016-10-28 17:12     ` Ricardo Wurmus
2016-10-28 19:57       ` Marius Bakke

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).