all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#48291] [PATCH] - updated lisp library unix-opts.
@ 2021-05-08 17:29 cage
  2021-05-08 21:28 ` Maxime Devos
  0 siblings, 1 reply; 3+ messages in thread
From: cage @ 2021-05-08 17:29 UTC (permalink / raw)
  To: 48291

Hi!

I found this library outdated, this patch makes guix uses a commit with
new features added.

Bye!
C.

---
 gnu/packages/lisp-xyz.scm | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/lisp-xyz.scm b/gnu/packages/lisp-xyz.scm
index 0824fa1d82..275517654f 100644
--- a/gnu/packages/lisp-xyz.scm
+++ b/gnu/packages/lisp-xyz.scm
@@ -3276,19 +3276,21 @@ settings ensure a very simple mode of operation, similar to that provided by
   (sbcl-package->ecl-package sbcl-cl-json))
 
 (define-public sbcl-unix-opts
+  (let ((commit "0e61f34b2ecf62288437810d4abb31e572048b04")
+        (revision "1"))
   (package
     (name "sbcl-unix-opts")
-    (version "0.1.7")
+    (version (git-version "0.1.7" revision commit))
     (source
      (origin
        (method git-fetch)
        (uri (git-reference
              (url "https://github.com/libre-man/unix-opts")
-             (commit version)))
+             (commit commit)))
        (file-name (git-file-name "unix-opts" version))
        (sha256
         (base32
-         "08djdi1ard09fijb7w9bdmhmwd98b1hzmcnjw9fqjiqa0g3b44rr"))))
+         "16mcqpzwrz808p9n3wwl99ckg3hg7yihw08y1i4l7c92aldbkasq"))))
     (build-system asdf-build-system/sbcl)
     (home-page "https://github.com/hankhero/cl-json")
     (synopsis "Unix-style command line options parser")
-- 
2.31.1





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

* [bug#48291] [PATCH] - updated lisp library unix-opts.
  2021-05-08 17:29 [bug#48291] [PATCH] - updated lisp library unix-opts cage
@ 2021-05-08 21:28 ` Maxime Devos
  2021-05-09 10:10   ` cage
  0 siblings, 1 reply; 3+ messages in thread
From: Maxime Devos @ 2021-05-08 21:28 UTC (permalink / raw)
  To: cage, 48291

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

cage schreef op za 08-05-2021 om 19:29 [+0200]:
> Hi!
> 
> I found this library outdated, this patch makes guix uses a commit with
> new features added.

There is a strong preference for only using versions that are ‘official releases’
‘blessed’ by upstream, and not some random commit, unless there are compelling
reasons to do otherwise.

I see the 0.1.7 commit dates from 2017.  If there are new features and/or bug fixes,
maybe ask upstream to make a new release?  If there are important bug fixes and
upstream doesn't feel ready for a new release, we can cherry-pick the bug fixes
via the 'patches' mechanism.

Or maybe we could just use the commit, I dunno. But then a comment should be added
explaining why a particular commit is used instead of an official release.

Greetings,
Maxime.


[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

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

* [bug#48291] [PATCH] - updated lisp library unix-opts.
  2021-05-08 21:28 ` Maxime Devos
@ 2021-05-09 10:10   ` cage
  0 siblings, 0 replies; 3+ messages in thread
From: cage @ 2021-05-09 10:10 UTC (permalink / raw)
  To: Maxime Devos; +Cc: cage, 48291

Hi!

[...]

>
> There is a strong preference for only using versions that are ‘official releases’
> ‘blessed’ by upstream, and not some random commit, unless there are compelling
> reasons to do otherwise.

I  can totally  see the  point here,  i have  sent this  patch because
another program  i have packaged (tinmop)  uses a new version  of this
library and simply crash if called with any command line switch. :(

So i have asked upstream to tag the current commit (or any commit they
choose,  of course)  as new  release,

https://github.com/libre-man/unix-opts/issues/35

so i  suppose we  just have  to wait. :)

Thanks for your reply!
C.




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

end of thread, other threads:[~2021-05-09 10:11 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-08 17:29 [bug#48291] [PATCH] - updated lisp library unix-opts cage
2021-05-08 21:28 ` Maxime Devos
2021-05-09 10:10   ` cage

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.