GNU bug report logs - #48126
[PATCH] Add package for vim-solarized

Previous Next

Package: guix-patches;

Reported by: Tissevert <tissevert+guix <at> marvid.fr>

Date: Fri, 30 Apr 2021 18:04:02 UTC

Severity: normal

Tags: patch

Done: Tobias Geerinckx-Rice <me <at> tobias.gr>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Tobias Geerinckx-Rice <me <at> tobias.gr>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#48126: closed ([PATCH] Add package for vim-solarized)
Date: Sat, 15 May 2021 13:05:02 +0000
[Message part 1 (text/plain, inline)]
Your message dated Sat, 15 May 2021 15:04:11 +0200
with message-id <87eee8du7o.fsf <at> nckx>
and subject line Re: [PATCH] Add package for vim-solarized
has caused the debbugs.gnu.org bug report #48126,
regarding [PATCH] Add package for vim-solarized
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
48126: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=48126
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Tissevert <tissevert+guix <at> marvid.fr>
To: guix-patches <at> gnu.org
Subject: [PATCH] Add package for vim-solarized
Date: Fri, 30 Apr 2021 19:42:35 +0200
[Message part 3 (text/plain, inline)]
Hi Guix !

I here's my first attempt at packaging something, I noticed that vim-solarized
was missing so I tried to write a package for it.

The only warning I got was from the guix lint which told me it couldn't find
upstream releases. I believe this is due to vim-solarized particular structure:
it's a light «clone» of the main repository
https://github.com/altercation/solarized containing only the vim-files. It
doesn't have any tags/releases but the full repository does. I used the latest
commit on the master branch in git-reference for guix to retrieve it and the
corresponding version on the full repository as package version. Does that seem
right ? I found another copy in project vim-scripts
(https://github.com/vim-scripts/Solarized) which has made a tag (the same
version name that I chose) so we could be using either the full repository
(more unneeded data retrieved during the build) or this second copy (they do
provide a tag but isn't it better to remain as close as possible to the
original source ?).

I hope my patch is correct and thank you everyone for being such a welcoming
community and having made my experience with Guix so far absolutely great !

Tissevert
[0001-Add-package-for-vim-solarized.patch (text/x-patch, attachment)]
[Message part 5 (message/rfc822, inline)]
From: Tobias Geerinckx-Rice <me <at> tobias.gr>
To: 48126-done <at> debbugs.gnu.org
Subject: Re: [PATCH] Add package for vim-solarized
Date: Sat, 15 May 2021 15:04:11 +0200
[Message part 6 (text/plain, inline)]
Tissevert,

I

- added a commit message following our particular flavour of the 
 ‘GNU ChangeLog format’,
- changed the URL to /solarized (the commit already referred to 
 that repository),
- explained that this package installs only the Vim files from 
 there,

and pushed the thing as 992aa56e7c5d10e7d8d260fe3b2d0ce15b00c17b.

Phew!  And thanks,

T G-R
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 4 years and 65 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.