emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* git-annex-related org-attach tests failing on master
@ 2019-09-02 22:26 Kyle Meyer
  2019-09-03 23:42 ` Gustav Wikström
  0 siblings, 1 reply; 6+ messages in thread
From: Kyle Meyer @ 2019-09-02 22:26 UTC (permalink / raw)
  To: Gustav Wikström, Org-mode

Hello,

The following tests fail as of ae9cd4370 (org-attach*, org, org-manual,
org-news, ox-html, testing/*, 2018-11-25):

    test-org-attach-git/get-maybe
    test-org-attach-git/use-annex
    test-org-attach/dired-attach-to-next-best-subtree/1
    test-org-attach/dired-attach-to-next-best-subtree/2

Gustav, could you please take a look?  I suspect you didn't see these
failures when you made those changes because you don't have git-annex
installed.

Here are the backtraces:

--8<---------------cut here---------------start------------->8---
selected tests: org-attach
Running 5 tests (2019-09-02 18:22:55-0400)
(Shell command succeeded with no output)
(Shell command succeeded with no output)
Test test-org-attach-git/get-maybe backtrace:
  (org-attach-annex-get-maybe (expand-file-name "test-file"))
  (let ((org-attach-annex-auto-get t)) (org-attach-annex-get-maybe (ex
  (let ((path (expand-file-name "test-file")) (annex-dup (make-temp-fi
  (let ((default-directory tmpdir) (org-attach-id-dir tmpdir)) (shell-
  (unwind-protect (let ((default-directory tmpdir) (org-attach-id-dir 
  (let ((tmpdir (make-temp-file "org-annex-test" t "/"))) (unwind-prot
  (lambda nil (let ((tmpdir (make-temp-file "org-annex-test" t "/"))) 
  ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
  ert-run-test(#s(ert-test :name test-org-attach-git/get-maybe :docume
  ert-run-or-rerun-test(#s(ert--stats :selector "org-attach" :tests [#
  ert-run-tests("org-attach" #f(compiled-function (event-type &rest ev
  ert-run-tests-batch("org-attach")
  ert-run-tests-batch-and-exit("org-attach")
  (let ((org-id-track-globally t) (org-test-selector (if org-test-sele
  org-test-run-batch-tests("org-attach")
  eval((org-test-run-batch-tests org-test-select-re))
  command-line-1(("--eval" "(setq vc-handled-backends nil org-startup-
  command-line()
  normal-top-level()
Test test-org-attach-git/get-maybe condition:
    (void-function org-attach-annex-get-maybe)
   FAILED  1/5  test-org-attach-git/get-maybe
Test test-org-attach-git/use-annex backtrace:
  org-attach-use-annex()
  apply(org-attach-use-annex nil)
  (setq value-3425 (apply fn-3423 args-3424))
  (unwind-protect (setq value-3425 (apply fn-3423 args-3424)) (setq fo
  (if (unwind-protect (setq value-3425 (apply fn-3423 args-3424)) (set
  (let (form-description-3427) (if (unwind-protect (setq value-3425 (a
  (let ((value-3425 (quote ert-form-evaluation-aborted-3426))) (let (f
  (let* ((fn-3423 (function org-attach-use-annex)) (args-3424 (conditi
  (let ((org-attach-git-annex-cutoff 1)) (let* ((fn-3423 (function org
  (let ((default-directory tmpdir) (org-attach-id-dir tmpdir)) (shell-
  (unwind-protect (let ((default-directory tmpdir) (org-attach-id-dir 
  (let ((tmpdir (make-temp-file "org-annex-test" t "/"))) (unwind-prot
  (lambda nil (let ((tmpdir (make-temp-file "org-annex-test" t "/"))) 
  ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
  ert-run-test(#s(ert-test :name test-org-attach-git/use-annex :docume
  ert-run-or-rerun-test(#s(ert--stats :selector "org-attach" :tests [#
  ert-run-tests("org-attach" #f(compiled-function (event-type &rest ev
  ert-run-tests-batch("org-attach")
  ert-run-tests-batch-and-exit("org-attach")
  (let ((org-id-track-globally t) (org-test-selector (if org-test-sele
  org-test-run-batch-tests("org-attach")
  eval((org-test-run-batch-tests org-test-select-re))
  command-line-1(("--eval" "(setq vc-handled-backends nil org-startup-
  command-line()
  normal-top-level()
Test test-org-attach-git/use-annex condition:
    (void-function org-attach-use-annex)
   FAILED  2/5  test-org-attach-git/use-annex
   passed  3/5  test-org-attach/dir
Test test-org-attach/dired-attach-to-next-best-subtree/1 backtrace:
  org-attach-git-commit("/tmp/tmp-orgtest/data/73/65ffd7-1e1f-40e3-a9a
  run-hook-with-args(org-attach-git-commit "/tmp/tmp-orgtest/data/73/6
  (let* ((attach-dir (org-attach-dir (quote get-create))) (fname (expa
  (let ((basename (file-name-nondirectory file))) (let* ((attach-dir (
  org-attach-attach("/tmp/tmp-orgtest/acrffdU")
  (let ((file (car --dolist-tail--))) (org-attach-attach file) (setq -
  (while --dolist-tail-- (let ((file (car --dolist-tail--))) (org-atta
  (let ((--dolist-tail-- files)) (while --dolist-tail-- (let ((file (c
  (let ((start-win (selected-window)) (other-win (get-window-with-pred
  org-attach-dired-to-subtree(("/tmp/tmp-orgtest/acrffdU"))
  funcall-interactively(org-attach-dired-to-subtree ("/tmp/tmp-orgtest
  call-interactively(org-attach-dired-to-subtree)
  (progn (split-window) (dired temporary-file-directory) (progn (or (e
  (progn (setq buffer (find-file file)) (if (re-search-forward "<point
  (unwind-protect (progn (setq buffer (find-file file)) (if (re-search
  (let ((file (make-temp-file "org-test")) (inside-text (if (stringp "
  (unwind-protect (let ((file (make-temp-file "org-test")) (inside-tex
  (let ((a-filename (make-temp-file "a"))) (unwind-protect (let ((file
  (setq value-3527 (let ((a-filename (make-temp-file "a"))) (unwind-pr
  (unwind-protect (setq value-3527 (let ((a-filename (make-temp-file "
  (if (unwind-protect (setq value-3527 (let ((a-filename (make-temp-fi
  (let (form-description-3528) (if (unwind-protect (setq value-3527 (l
  (let ((value-3527 (gensym "ert-form-evaluation-aborted-"))) (let (fo
  (closure (t) nil (let ((value-3527 (gensym "ert-form-evaluation-abor
  ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
  ert-run-test(#s(ert-test :name test-org-attach/dired-attach-to-next-
  ert-run-or-rerun-test(#s(ert--stats :selector "org-attach" :tests [#
  ert-run-tests("org-attach" #f(compiled-function (event-type &rest ev
  ert-run-tests-batch("org-attach")
  ert-run-tests-batch-and-exit("org-attach")
  (let ((org-id-track-globally t) (org-test-selector (if org-test-sele
  org-test-run-batch-tests("org-attach")
  eval((org-test-run-batch-tests org-test-select-re))
  command-line-1(("--eval" "(setq vc-handled-backends nil org-startup-
  command-line()
  normal-top-level()
Test test-org-attach/dired-attach-to-next-best-subtree/1 condition:
    (wrong-number-of-arguments
     ((t)
      nil "Commit changes to git if `org-attach-id-dir' is properly initialized.
This checks for the existence of a \".git\" directory in that directory."
      (let*
	  ((dir ...)
	   (git-dir ...)
	   (use-annex ...)
	   (changes 0))
	(if
	    (and git-dir ...)
	    (progn ...))))
     1)
   FAILED  4/5  test-org-attach/dired-attach-to-next-best-subtree/1
Directory has changed on disk; type g to update Dired
Test test-org-attach/dired-attach-to-next-best-subtree/2 backtrace:
  org-attach-git-commit("/tmp/tmp-orgtest/data/cb/6acd9b-4618-478e-907
  run-hook-with-args(org-attach-git-commit "/tmp/tmp-orgtest/data/cb/6
  (let* ((attach-dir (org-attach-dir (quote get-create))) (fname (expa
  (let ((basename (file-name-nondirectory file))) (let* ((attach-dir (
  org-attach-attach("/tmp/tmp-orgtest/a69HoUW")
  (let ((file (car --dolist-tail--))) (org-attach-attach file) (setq -
  (while --dolist-tail-- (let ((file (car --dolist-tail--))) (org-atta
  (let ((--dolist-tail-- files)) (while --dolist-tail-- (let ((file (c
  (let ((start-win (selected-window)) (other-win (get-window-with-pred
  org-attach-dired-to-subtree(("/tmp/tmp-orgtest/a69HoUW" "/tmp/tmp-or
  funcall-interactively(org-attach-dired-to-subtree ("/tmp/tmp-orgtest
  call-interactively(org-attach-dired-to-subtree)
  (progn (split-window) (dired temporary-file-directory) (progn (or (e
  (progn (setq buffer (find-file file)) (if (re-search-forward "<point
  (unwind-protect (progn (setq buffer (find-file file)) (if (re-search
  (let ((file (make-temp-file "org-test")) (inside-text (if (stringp "
  (unwind-protect (let ((file (make-temp-file "org-test")) (inside-tex
  (let ((a-filename (make-temp-file "a")) (b-filename (make-temp-file 
  (setq value-3529 (let ((a-filename (make-temp-file "a")) (b-filename
  (unwind-protect (setq value-3529 (let ((a-filename (make-temp-file "
  (if (unwind-protect (setq value-3529 (let ((a-filename (make-temp-fi
  (let (form-description-3530) (if (unwind-protect (setq value-3529 (l
  (let ((value-3529 (gensym "ert-form-evaluation-aborted-"))) (let (fo
  (closure (t) nil (let ((value-3529 (gensym "ert-form-evaluation-abor
  ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
  ert-run-test(#s(ert-test :name test-org-attach/dired-attach-to-next-
  ert-run-or-rerun-test(#s(ert--stats :selector "org-attach" :tests [#
  ert-run-tests("org-attach" #f(compiled-function (event-type &rest ev
  ert-run-tests-batch("org-attach")
  ert-run-tests-batch-and-exit("org-attach")
  (let ((org-id-track-globally t) (org-test-selector (if org-test-sele
  org-test-run-batch-tests("org-attach")
  eval((org-test-run-batch-tests org-test-select-re))
  command-line-1(("--eval" "(setq vc-handled-backends nil org-startup-
  command-line()
  normal-top-level()
Test test-org-attach/dired-attach-to-next-best-subtree/2 condition:
    (wrong-number-of-arguments
     ((t)
      nil "Commit changes to git if `org-attach-id-dir' is properly initialized.
This checks for the existence of a \".git\" directory in that directory."
      (let*
	  ((dir ...)
	   (git-dir ...)
	   (use-annex ...)
	   (changes 0))
	(if
	    (and git-dir ...)
	    (progn ...))))
     1)
   FAILED  5/5  test-org-attach/dired-attach-to-next-best-subtree/2

Ran 5 tests, 1 results as expected, 4 unexpected (2019-09-02 18:22:57-0400)

4 unexpected results:
   FAILED  test-org-attach-git/get-maybe
   FAILED  test-org-attach-git/use-annex
   FAILED  test-org-attach/dired-attach-to-next-best-subtree/1
   FAILED  test-org-attach/dired-attach-to-next-best-subtree/2

mk/targets.mk:104: recipe for target 'test-dirty' failed
make: *** [test-dirty] Error 1
--8<---------------cut here---------------end--------------->8---

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

* RE: git-annex-related org-attach tests failing on master
  2019-09-02 22:26 git-annex-related org-attach tests failing on master Kyle Meyer
@ 2019-09-03 23:42 ` Gustav Wikström
  2019-09-04  3:32   ` Kyle Meyer
  0 siblings, 1 reply; 6+ messages in thread
From: Gustav Wikström @ 2019-09-03 23:42 UTC (permalink / raw)
  To: Kyle Meyer, Org-mode

Hi Kyle,

> -----Original Message-----
> From: Kyle Meyer <kyle@kyleam.com>
> 
> The following tests fail as of ae9cd4370 (org-attach*, org, org-manual,
> org-news, ox-html, testing/*, 2018-11-25):
> 
>     test-org-attach-git/get-maybe
>     test-org-attach-git/use-annex
>     test-org-attach/dired-attach-to-next-best-subtree/1
>     test-org-attach/dired-attach-to-next-best-subtree/2
> 
> Gustav, could you please take a look?  I suspect you didn't see these
> failures when you made those changes because you don't have git-annex
> installed.

Indeed, that was the case. I've taken a look and have fixed the issues on my
local machine. I must say the whole git-annex thing feels misplaced in the
Org mode repository though. I guess, as long as git annex isn't installed 
it's not really an issue. But it surely would fit better as a separate library.

Anyhow - I'll commit the fixes after having taken a more thorough look at how 
the commit message should be formatted, per your request. Stay tuned.

Oh, and the fix will make git-annex work only with id-based attachments 
again, as it was before. I tried to make it more general but realized I don't
have a clear understanding of the use-case to make it function in a more general
fashion.

Thanks for reporting this,

Kind regards
Gustav

> 
> Here are the backtraces:
> 
> --8<---------------cut here---------------start------------->8---
> selected tests: org-attach
> Running 5 tests (2019-09-02 18:22:55-0400) (Shell command succeeded with
> no output) (Shell command succeeded with no output) Test test-org-attach-
> git/get-maybe backtrace:
>   (org-attach-annex-get-maybe (expand-file-name "test-file"))
>   (let ((org-attach-annex-auto-get t)) (org-attach-annex-get-maybe (ex
>   (let ((path (expand-file-name "test-file")) (annex-dup (make-temp-fi
>   (let ((default-directory tmpdir) (org-attach-id-dir tmpdir)) (shell-
>   (unwind-protect (let ((default-directory tmpdir) (org-attach-id-dir
>   (let ((tmpdir (make-temp-file "org-annex-test" t "/"))) (unwind-prot
>   (lambda nil (let ((tmpdir (make-temp-file "org-annex-test" t "/")))
>   ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
>   ert-run-test(#s(ert-test :name test-org-attach-git/get-maybe :docume
>   ert-run-or-rerun-test(#s(ert--stats :selector "org-attach" :tests [#
>   ert-run-tests("org-attach" #f(compiled-function (event-type &rest ev
>   ert-run-tests-batch("org-attach")
>   ert-run-tests-batch-and-exit("org-attach")
>   (let ((org-id-track-globally t) (org-test-selector (if org-test-sele
>   org-test-run-batch-tests("org-attach")
>   eval((org-test-run-batch-tests org-test-select-re))
>   command-line-1(("--eval" "(setq vc-handled-backends nil org-startup-
>   command-line()
>   normal-top-level()
> Test test-org-attach-git/get-maybe condition:
>     (void-function org-attach-annex-get-maybe)
>    FAILED  1/5  test-org-attach-git/get-maybe Test test-org-attach-
> git/use-annex backtrace:
>   org-attach-use-annex()
>   apply(org-attach-use-annex nil)
>   (setq value-3425 (apply fn-3423 args-3424))
>   (unwind-protect (setq value-3425 (apply fn-3423 args-3424)) (setq fo
>   (if (unwind-protect (setq value-3425 (apply fn-3423 args-3424)) (set
>   (let (form-description-3427) (if (unwind-protect (setq value-3425 (a
>   (let ((value-3425 (quote ert-form-evaluation-aborted-3426))) (let (f
>   (let* ((fn-3423 (function org-attach-use-annex)) (args-3424 (conditi
>   (let ((org-attach-git-annex-cutoff 1)) (let* ((fn-3423 (function org
>   (let ((default-directory tmpdir) (org-attach-id-dir tmpdir)) (shell-
>   (unwind-protect (let ((default-directory tmpdir) (org-attach-id-dir
>   (let ((tmpdir (make-temp-file "org-annex-test" t "/"))) (unwind-prot
>   (lambda nil (let ((tmpdir (make-temp-file "org-annex-test" t "/")))
>   ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
>   ert-run-test(#s(ert-test :name test-org-attach-git/use-annex :docume
>   ert-run-or-rerun-test(#s(ert--stats :selector "org-attach" :tests [#
>   ert-run-tests("org-attach" #f(compiled-function (event-type &rest ev
>   ert-run-tests-batch("org-attach")
>   ert-run-tests-batch-and-exit("org-attach")
>   (let ((org-id-track-globally t) (org-test-selector (if org-test-sele
>   org-test-run-batch-tests("org-attach")
>   eval((org-test-run-batch-tests org-test-select-re))
>   command-line-1(("--eval" "(setq vc-handled-backends nil org-startup-
>   command-line()
>   normal-top-level()
> Test test-org-attach-git/use-annex condition:
>     (void-function org-attach-use-annex)
>    FAILED  2/5  test-org-attach-git/use-annex
>    passed  3/5  test-org-attach/dir
> Test test-org-attach/dired-attach-to-next-best-subtree/1 backtrace:
>   org-attach-git-commit("/tmp/tmp-orgtest/data/73/65ffd7-1e1f-40e3-a9a
>   run-hook-with-args(org-attach-git-commit "/tmp/tmp-orgtest/data/73/6
>   (let* ((attach-dir (org-attach-dir (quote get-create))) (fname (expa
>   (let ((basename (file-name-nondirectory file))) (let* ((attach-dir (
>   org-attach-attach("/tmp/tmp-orgtest/acrffdU")
>   (let ((file (car --dolist-tail--))) (org-attach-attach file) (setq -
>   (while --dolist-tail-- (let ((file (car --dolist-tail--))) (org-atta
>   (let ((--dolist-tail-- files)) (while --dolist-tail-- (let ((file (c
>   (let ((start-win (selected-window)) (other-win (get-window-with-pred
>   org-attach-dired-to-subtree(("/tmp/tmp-orgtest/acrffdU"))
>   funcall-interactively(org-attach-dired-to-subtree ("/tmp/tmp-orgtest
>   call-interactively(org-attach-dired-to-subtree)
>   (progn (split-window) (dired temporary-file-directory) (progn (or (e
>   (progn (setq buffer (find-file file)) (if (re-search-forward "<point
>   (unwind-protect (progn (setq buffer (find-file file)) (if (re-search
>   (let ((file (make-temp-file "org-test")) (inside-text (if (stringp "
>   (unwind-protect (let ((file (make-temp-file "org-test")) (inside-tex
>   (let ((a-filename (make-temp-file "a"))) (unwind-protect (let ((file
>   (setq value-3527 (let ((a-filename (make-temp-file "a"))) (unwind-pr
>   (unwind-protect (setq value-3527 (let ((a-filename (make-temp-file "
>   (if (unwind-protect (setq value-3527 (let ((a-filename (make-temp-fi
>   (let (form-description-3528) (if (unwind-protect (setq value-3527 (l
>   (let ((value-3527 (gensym "ert-form-evaluation-aborted-"))) (let (fo
>   (closure (t) nil (let ((value-3527 (gensym "ert-form-evaluation-abor
>   ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
>   ert-run-test(#s(ert-test :name test-org-attach/dired-attach-to-next-
>   ert-run-or-rerun-test(#s(ert--stats :selector "org-attach" :tests [#
>   ert-run-tests("org-attach" #f(compiled-function (event-type &rest ev
>   ert-run-tests-batch("org-attach")
>   ert-run-tests-batch-and-exit("org-attach")
>   (let ((org-id-track-globally t) (org-test-selector (if org-test-sele
>   org-test-run-batch-tests("org-attach")
>   eval((org-test-run-batch-tests org-test-select-re))
>   command-line-1(("--eval" "(setq vc-handled-backends nil org-startup-
>   command-line()
>   normal-top-level()
> Test test-org-attach/dired-attach-to-next-best-subtree/1 condition:
>     (wrong-number-of-arguments
>      ((t)
>       nil "Commit changes to git if `org-attach-id-dir' is properly
> initialized.
> This checks for the existence of a \".git\" directory in that directory."
>       (let*
> 	  ((dir ...)
> 	   (git-dir ...)
> 	   (use-annex ...)
> 	   (changes 0))
> 	(if
> 	    (and git-dir ...)
> 	    (progn ...))))
>      1)
>    FAILED  4/5  test-org-attach/dired-attach-to-next-best-subtree/1
> Directory has changed on disk; type g to update Dired Test test-org-
> attach/dired-attach-to-next-best-subtree/2 backtrace:
>   org-attach-git-commit("/tmp/tmp-orgtest/data/cb/6acd9b-4618-478e-907
>   run-hook-with-args(org-attach-git-commit "/tmp/tmp-orgtest/data/cb/6
>   (let* ((attach-dir (org-attach-dir (quote get-create))) (fname (expa
>   (let ((basename (file-name-nondirectory file))) (let* ((attach-dir (
>   org-attach-attach("/tmp/tmp-orgtest/a69HoUW")
>   (let ((file (car --dolist-tail--))) (org-attach-attach file) (setq -
>   (while --dolist-tail-- (let ((file (car --dolist-tail--))) (org-atta
>   (let ((--dolist-tail-- files)) (while --dolist-tail-- (let ((file (c
>   (let ((start-win (selected-window)) (other-win (get-window-with-pred
>   org-attach-dired-to-subtree(("/tmp/tmp-orgtest/a69HoUW" "/tmp/tmp-or
>   funcall-interactively(org-attach-dired-to-subtree ("/tmp/tmp-orgtest
>   call-interactively(org-attach-dired-to-subtree)
>   (progn (split-window) (dired temporary-file-directory) (progn (or (e
>   (progn (setq buffer (find-file file)) (if (re-search-forward "<point
>   (unwind-protect (progn (setq buffer (find-file file)) (if (re-search
>   (let ((file (make-temp-file "org-test")) (inside-text (if (stringp "
>   (unwind-protect (let ((file (make-temp-file "org-test")) (inside-tex
>   (let ((a-filename (make-temp-file "a")) (b-filename (make-temp-file
>   (setq value-3529 (let ((a-filename (make-temp-file "a")) (b-filename
>   (unwind-protect (setq value-3529 (let ((a-filename (make-temp-file "
>   (if (unwind-protect (setq value-3529 (let ((a-filename (make-temp-fi
>   (let (form-description-3530) (if (unwind-protect (setq value-3529 (l
>   (let ((value-3529 (gensym "ert-form-evaluation-aborted-"))) (let (fo
>   (closure (t) nil (let ((value-3529 (gensym "ert-form-evaluation-abor
>   ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
>   ert-run-test(#s(ert-test :name test-org-attach/dired-attach-to-next-
>   ert-run-or-rerun-test(#s(ert--stats :selector "org-attach" :tests [#
>   ert-run-tests("org-attach" #f(compiled-function (event-type &rest ev
>   ert-run-tests-batch("org-attach")
>   ert-run-tests-batch-and-exit("org-attach")
>   (let ((org-id-track-globally t) (org-test-selector (if org-test-sele
>   org-test-run-batch-tests("org-attach")
>   eval((org-test-run-batch-tests org-test-select-re))
>   command-line-1(("--eval" "(setq vc-handled-backends nil org-startup-
>   command-line()
>   normal-top-level()
> Test test-org-attach/dired-attach-to-next-best-subtree/2 condition:
>     (wrong-number-of-arguments
>      ((t)
>       nil "Commit changes to git if `org-attach-id-dir' is properly
> initialized.
> This checks for the existence of a \".git\" directory in that directory."
>       (let*
> 	  ((dir ...)
> 	   (git-dir ...)
> 	   (use-annex ...)
> 	   (changes 0))
> 	(if
> 	    (and git-dir ...)
> 	    (progn ...))))
>      1)
>    FAILED  5/5  test-org-attach/dired-attach-to-next-best-subtree/2
> 
> Ran 5 tests, 1 results as expected, 4 unexpected (2019-09-02 18:22:57-
> 0400)
> 
> 4 unexpected results:
>    FAILED  test-org-attach-git/get-maybe
>    FAILED  test-org-attach-git/use-annex
>    FAILED  test-org-attach/dired-attach-to-next-best-subtree/1
>    FAILED  test-org-attach/dired-attach-to-next-best-subtree/2
> 
> mk/targets.mk:104: recipe for target 'test-dirty' failed
> make: *** [test-dirty] Error 1
> --8<---------------cut here---------------end--------------->8---

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

* Re: git-annex-related org-attach tests failing on master
  2019-09-03 23:42 ` Gustav Wikström
@ 2019-09-04  3:32   ` Kyle Meyer
  2019-09-08 12:52     ` Gustav Wikström
  0 siblings, 1 reply; 6+ messages in thread
From: Kyle Meyer @ 2019-09-04  3:32 UTC (permalink / raw)
  To: Gustav Wikström, Org-mode

Gustav Wikström <gustav@whil.se> writes:

> I've taken a look and have fixed the issues on my
> local machine.

Thank you!

> I must say the whole git-annex thing feels misplaced in the
> Org mode repository though. I guess, as long as git annex isn't installed 
> it's not really an issue. But it surely would fit better as a separate library.

I'm a heavy git-annex user, but I haven't done more than play with
org-attach (a while back), so I don't have much of an opinion here, but
your general assessment sounds fair.  That being said, the
org-attach/git-annex functionality has been around since 2012, so
provided it's not making other things more difficult, moving it outside
of Org doesn't seem worth the effort or disruption of people's
workflows.

Thanks again for looking into the failures.

-- 
Kyle

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

* Re: git-annex-related org-attach tests failing on master
  2019-09-04  3:32   ` Kyle Meyer
@ 2019-09-08 12:52     ` Gustav Wikström
  2019-09-08 15:46       ` Kyle Meyer
  0 siblings, 1 reply; 6+ messages in thread
From: Gustav Wikström @ 2019-09-08 12:52 UTC (permalink / raw)
  To: Kyle Meyer, Org-mode

Hi again Kyle,

I've pushed a change to master that fixes the failed tests. 
Please verify on your end as well as. 

> -----Original Message-----
> From: Kyle Meyer <kyle@kyleam.com>
> Sent: den 4 september 2019 05:32
> To: Gustav Wikström <gustav@whil.se>; Org-mode <emacs-orgmode@gnu.org>
> Subject: RE: git-annex-related org-attach tests failing on master
> 
> Gustav Wikström <gustav@whil.se> writes:
> 
> > I've taken a look and have fixed the issues on my
> > local machine.
> 
> Thank you!
> 
> > I must say the whole git-annex thing feels misplaced in the
> > Org mode repository though. I guess, as long as git annex isn't installed
> > it's not really an issue. But it surely would fit better as a separate
> library.
> 
> I'm a heavy git-annex user, but I haven't done more than play with
> org-attach (a while back), so I don't have much of an opinion here, but
> your general assessment sounds fair.  That being said, the
> org-attach/git-annex functionality has been around since 2012, so
> provided it's not making other things more difficult, moving it outside
> of Org doesn't seem worth the effort or disruption of people's
> workflows.

No worries, I was just voicing my concern. Issue hopefully fixed now.

> Thanks again for looking into the failures.
> 
> --
> Kyle

Kind regards
Gustav

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

* Re: git-annex-related org-attach tests failing on master
  2019-09-08 12:52     ` Gustav Wikström
@ 2019-09-08 15:46       ` Kyle Meyer
  2019-09-08 16:44         ` Gustav Wikström
  0 siblings, 1 reply; 6+ messages in thread
From: Kyle Meyer @ 2019-09-08 15:46 UTC (permalink / raw)
  To: Gustav Wikström, Org-mode

Gustav Wikström <gustav@whil.se> writes:

> Hi again Kyle,
>
> I've pushed a change to master that fixes the failed tests. 
> Please verify on your end as well as. 

It does.  I've pushed a minor fixup commit to silence the byte-compiler.

Thanks.

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

* Re: git-annex-related org-attach tests failing on master
  2019-09-08 15:46       ` Kyle Meyer
@ 2019-09-08 16:44         ` Gustav Wikström
  0 siblings, 0 replies; 6+ messages in thread
From: Gustav Wikström @ 2019-09-08 16:44 UTC (permalink / raw)
  To: Kyle Meyer, Org-mode

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

Ahh, thanks for that!

/G

Get Outlook for Android<https://aka.ms/ghei36>

________________________________
From: Kyle Meyer <kyle@kyleam.com>
Sent: Sunday, September 8, 2019 5:46:21 PM
To: Gustav Wikström <gustav@whil.se>; Org-mode <emacs-orgmode@gnu.org>
Subject: RE: git-annex-related org-attach tests failing on master

Gustav Wikström <gustav@whil.se> writes:

> Hi again Kyle,
>
> I've pushed a change to master that fixes the failed tests.
> Please verify on your end as well as.

It does.  I've pushed a minor fixup commit to silence the byte-compiler.

Thanks.

[-- Attachment #2: Type: text/html, Size: 1872 bytes --]

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

end of thread, other threads:[~2019-09-08 16:44 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-02 22:26 git-annex-related org-attach tests failing on master Kyle Meyer
2019-09-03 23:42 ` Gustav Wikström
2019-09-04  3:32   ` Kyle Meyer
2019-09-08 12:52     ` Gustav Wikström
2019-09-08 15:46       ` Kyle Meyer
2019-09-08 16:44         ` Gustav Wikström

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

	https://git.savannah.gnu.org/cgit/emacs/org-mode.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).