all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Brian Leung <bkleung89@gmail.com>
To: Oleg Pyhalov <go.wigust@gmail.com>
Cc: 37259@debbugs.gnu.org
Subject: [bug#37259] [PATCH] Update emacs-org-ql to 0.2.
Date: Mon, 2 Sep 2019 02:40:19 +0200	[thread overview]
Message-ID: <CAAc=MEzH2XFaef-pNPcfSdUrm9pAGHx5_d113++m_UKSz7rKcg@mail.gmail.com> (raw)
In-Reply-To: <87mufnu85r.fsf@majordomo.ru>


[-- Attachment #1.1: Type: text/plain, Size: 878 bytes --]

Hi Oleg,

I've modified the patches. Three tests from ts.el are failing right now; I
don't think it's a serious problem though (see the patch for the
explanation).

On Sun, Sep 1, 2019 at 9:30 PM Oleg Pyhalov <go.wigust@gmail.com> wrote:

> Hello Brian,
>
> Brian Leung <bkleung89@gmail.com> writes:
>
> > I changed the commit message on the last patch.
> > From 4f75a46c67e840eccc01f21cfd62768fc8e5ffdf Mon Sep 17 00:00:00 2001
> > From: Brian Leung <bkleung89@gmail.com>
> > Date: Sun, 1 Sep 2019 19:46:27 +0200
> > Subject: [PATCH 1/3] gnu: Add emacs-ts.
> >
> > * gnu/packages/emacs-xyz.scm (emacs-ts): New variable.
> > ---
> >  gnu/packages/emacs-xyz.scm | 25 +++++++++++++++++++++++++
> >  1 file changed, 25 insertions(+)
>
> Could you take a look on enabling test suite, please?
>
> Everything else LGTM.
>
> […]
>
> Thanks,
> Oleg.
>

[-- Attachment #1.2: Type: text/html, Size: 1392 bytes --]

[-- Attachment #2: 0001-gnu-Add-emacs-ts.patch --]
[-- Type: text/x-patch, Size: 2692 bytes --]

From 92ab81c3468b3bd84a455af2fc85a4d64c830503 Mon Sep 17 00:00:00 2001
From: Brian Leung <bkleung89@gmail.com>
Date: Sun, 1 Sep 2019 19:46:27 +0200
Subject: [PATCH 1/3] gnu: Add emacs-ts.

* gnu/packages/emacs-xyz.scm (emacs-ts): New variable.
---
 gnu/packages/emacs-xyz.scm | 43 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 43 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 4bfc09d0aa..815a68dc63 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -11113,6 +11113,49 @@ into sections while preserving the structure imposed by any timestamps.")
 tables of contents.")
     (license license:gpl3+)))
 
+(define-public emacs-ts
+  (let ((commit "93c074f2895a204e003e8c7f3033c37d6486fac8")
+        (revision "1"))
+    (package
+      (name "emacs-ts")
+      (version (git-version "0.1" revision commit))
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://github.com/alphapapa/ts.el")
+                      (commit commit)))
+                (sha256
+                 (base32
+                  "0lpyv78k04vbp9glnv14dawcfgi3m49847wlgwfmkdq5cr3fn735"))
+                (file-name (git-file-name name version))))
+      (build-system emacs-build-system)
+      (propagated-inputs
+       `(("emacs-s" ,emacs-s)
+         ("emacs-dash" ,emacs-dash)))
+      (arguments
+       ;; XXX: Three tests are failing because of a timezone-related issue
+       ;; with how they're written.  On my machine, all the failing test
+       ;; results are 18000 seconds (5 hours) off.
+
+       ;; The ts-parse-org function accepts a string without any timezone
+       ;; info, not assumed to be in Unix time, and converts it to a so-called
+       ;; ts struct.  The ts-unix function (accessor) accepts a ts struct,
+       ;; then seems to assume the struct's corresponding time is in terms of
+       ;; the user's current time zone, before returning a Unix time in
+       ;; seconds.
+
+       ;; The failing tests all have similar problems, but nothing else about
+       ;; the library seems particularly off.
+       `(#:tests? #f
+         #:test-command '("emacs" "--batch"
+                          "-l" "test/test.el"
+                          "-f" "ert-run-tests-batch-and-exit")))
+      (home-page "https://github.com/alphapapa/ts.el")
+      (synopsis "Timestamp and date/time library")
+      (description "This package facilitates manipulating dates, times, and
+timestamps by providing a @code{ts} struct.")
+      (license license:gpl3+))))
+
 (define-public emacs-org-ql
   (package
     (name "emacs-org-ql")
-- 
2.23.0


[-- Attachment #3: 0002-gnu-emacs-org-sidebar-Update-to-0.1-1.ed951d1.patch --]
[-- Type: text/x-patch, Size: 1351 bytes --]

From a5d8f1a36b1891595051e2d555f2bd4c4bb07a0b Mon Sep 17 00:00:00 2001
From: Brian Leung <bkleung89@gmail.com>
Date: Sun, 1 Sep 2019 19:50:18 +0200
Subject: [PATCH 2/3] gnu: emacs-org-sidebar: Update to 0.1-1.ed951d1.

* gnu/packages/emacs-xyz.scm (emacs-org-sidebar): Update to 0.1-1.ed951d1.
---
 gnu/packages/emacs-xyz.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 815a68dc63..b1169cfd34 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -4257,7 +4257,7 @@ started with 20 minutes.  All values are customizable.")
     (license license:gpl3+)))
 
 (define-public emacs-org-sidebar
-  (let ((commit "74ca98b9920f3de3f13d49866581435e1ec63ec5")
+  (let ((commit "ed951d1e0d8b7e65ed35797403fd3e8c88f507f5")
         (revision "1"))
     (package
       (name "emacs-org-sidebar")
@@ -4270,7 +4270,7 @@ started with 20 minutes.  All values are customizable.")
                (commit commit)))
          (file-name (git-file-name name version))
          (sha256
-          (base32 "03p1ndyw2qp2skib5hszc4xyh84w7p2mhkd4a9dy6qv8q47xpsqn"))))
+          (base32 "01sf8v53pjsy80fzwmj2n8rp2z5gsnpyld0fm6j3bdv213clp69y"))))
       (build-system emacs-build-system)
       (propagated-inputs
        `(("emacs-dash" ,emacs-dash)
-- 
2.23.0


[-- Attachment #4: 0003-gnu-emacs-org-ql-Update-to-0.2.patch --]
[-- Type: text/x-patch, Size: 1511 bytes --]

From 82ee36c5fe6a4f0f27c448bfda2aad5a90714835 Mon Sep 17 00:00:00 2001
From: Brian Leung <bkleung89@gmail.com>
Date: Sun, 1 Sep 2019 19:51:05 +0200
Subject: [PATCH 3/3] gnu: emacs-org-ql: Update to 0.2.

* gnu/packages/emacs-xyz.scm (emacs-org-ql): Update to 0.2.
[inputs]: Add emacs-ts and emacs-org.
---
 gnu/packages/emacs-xyz.scm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index b1169cfd34..5a084ea002 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -11159,7 +11159,7 @@ timestamps by providing a @code{ts} struct.")
 (define-public emacs-org-ql
   (package
     (name "emacs-org-ql")
-    (version "0.1")
+    (version "0.2")
     (source (origin
               (method git-fetch)
               (uri (git-reference
@@ -11167,11 +11167,13 @@ timestamps by providing a @code{ts} struct.")
                     (commit version)))
               (sha256
                (base32
-                "1nvzirn1lmgmgl7irbsc1n391a2cw8gmvwm3pa228l2c1gcx8kd8"))
+                "0mq0aj0a3a5gi9nz0ncpzsh731d92n86b0iinvx1m45dcal06h9y"))
               (file-name (git-file-name name version))))
     (build-system emacs-build-system)
     (propagated-inputs
      `(("emacs-s" ,emacs-s)
+       ("emacs-ts" ,emacs-ts)
+       ("emacs-org" ,emacs-org)
        ("emacs-dash" ,emacs-dash)))
     (home-page "https://github.com/alphapapa/org-ql/")
     (synopsis "Query language for Org buffers")
-- 
2.23.0


  reply	other threads:[~2019-09-02  0:42 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-01 17:55 [bug#37259] [PATCH] Update emacs-org-ql to 0.2 Brian Leung
2019-09-01 18:09 ` Brian Leung
2019-09-01 19:29   ` Oleg Pyhalov
2019-09-02  0:40     ` Brian Leung [this message]
2019-09-02 17:31       ` Oleg Pyhalov
2019-09-02 19:10         ` Brian Leung
2019-09-03  7:00           ` Oleg Pyhalov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAAc=MEzH2XFaef-pNPcfSdUrm9pAGHx5_d113++m_UKSz7rKcg@mail.gmail.com' \
    --to=bkleung89@gmail.com \
    --cc=37259@debbugs.gnu.org \
    --cc=go.wigust@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.