* 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).