* A recent mobile org change causing sync issues @ 2015-10-18 22:43 Steve Harman 2015-10-19 6:48 ` Nicolas Goaziou 0 siblings, 1 reply; 4+ messages in thread From: Steve Harman @ 2015-10-18 22:43 UTC (permalink / raw) To: emacs-orgmode [-- Attachment #1: Type: text/plain, Size: 867 bytes --] Folks, I believe a recent change in org-mobile.el is causing a sync problem on the Android app. There is some info here: https://github.com/matburt/mobileorg-android/issues/472 The info for the commit is at the bottom. Respectfully, Steve author Sergey Gordienko <gordienkos@gmail.com> Fri, 26 Jun 2015 14:44:20 +0000 (17:44 +0300) committer Nicolas Goaziou <mail@nicolasgoaziou.fr> Sun, 28 Jun 2015 07:06:00 +0000 (09:06 +0200) commit c15db80e32d22dace184203f6f03a0b586bb4b50 tree 31d72afce3cc5adfb0548fdd88680543a2d82e5a tree | snapshot parent 5f3d8037a5099ca92db7edbf95ced22f0adefbef org-mobile: Tiny fix in creating index.org file * lisp/org-mobile.el (org-mobile-create-index-file): Eliminate cloning of `#+READONLY' property in the `index.org' file. A sublists of `org-todo-keywords' don't need to contain the `|'. TINYCHANGE [-- Attachment #2: Type: text/html, Size: 1322 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: A recent mobile org change causing sync issues 2015-10-18 22:43 A recent mobile org change causing sync issues Steve Harman @ 2015-10-19 6:48 ` Nicolas Goaziou 2015-10-19 14:52 ` Steve Harman 0 siblings, 1 reply; 4+ messages in thread From: Nicolas Goaziou @ 2015-10-19 6:48 UTC (permalink / raw) To: Steve Harman; +Cc: emacs-orgmode Hello, Steve Harman <stvharman@gmail.com> writes: > I believe a recent change in org-mobile.el is causing a sync problem on the > Android app. There is some info here: > https://github.com/matburt/mobileorg-android/issues/472 > > The info for the commit is at the bottom. Thank you for the report. This is a known issue, but it is not clear as why it failing. Could you should the differences between "index.org" created with or without this commit ? Regards, -- Nicolas Goaziou ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: A recent mobile org change causing sync issues 2015-10-19 6:48 ` Nicolas Goaziou @ 2015-10-19 14:52 ` Steve Harman 2015-10-24 9:29 ` Nicolas Goaziou 0 siblings, 1 reply; 4+ messages in thread From: Steve Harman @ 2015-10-19 14:52 UTC (permalink / raw) To: Steve Harman, emacs-orgmode [-- Attachment #1: Type: text/plain, Size: 1260 bytes --] Thanks Nicolas, I am not able to post the diff due to the personal nature of the data. However, I can say that with the commit, the tags in the #+TAGS line is missing. Also, a link to one of the org files is missing. Would it be possible to revert this change? This is such a critical bug and it really disrupted many of ours personal workflows. It also embarrassed us towards our colleagues, families, and friends who held the view that proprieatary solutions are better. In general, it is preferable to have a test suite which should pass before releasing these commits. Yes, there is MELPA stable but not every package is on MELPA. thanks Stv On Mon, Oct 19, 2015 at 2:48 AM, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote: > Hello, > > Steve Harman <stvharman@gmail.com> writes: > > > I believe a recent change in org-mobile.el is causing a sync problem on > the > > Android app. There is some info here: > > https://github.com/matburt/mobileorg-android/issues/472 > > > > The info for the commit is at the bottom. > > Thank you for the report. > > This is a known issue, but it is not clear as why it failing. > > Could you should the differences between "index.org" created with or > without this commit ? > > Regards, > > -- > Nicolas Goaziou > [-- Attachment #2: Type: text/html, Size: 2046 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: A recent mobile org change causing sync issues 2015-10-19 14:52 ` Steve Harman @ 2015-10-24 9:29 ` Nicolas Goaziou 0 siblings, 0 replies; 4+ messages in thread From: Nicolas Goaziou @ 2015-10-24 9:29 UTC (permalink / raw) To: Steve Harman; +Cc: emacs-orgmode [-- Attachment #1: Type: text/plain, Size: 1496 bytes --] Steve Harman <stvharman@gmail.com> writes: > Thanks Nicolas, > > I am not able to post the diff due to the personal nature of the data. > However, I can say that with the commit, the tags in the #+TAGS line is > missing. Also, a link to one of the org files is missing. Note that the commit alters neither TAGS lines nor links. > Would it be possible to revert this change? This change was introduced to fix a bug. Reverting it would break "org-mobile.el" in another way. The problem was reported but no one, so far, was able to investigate and tell what is wrong in the patch. > This is such a critical bug and it really disrupted many of ours > personal workflows. I sympathize. Unfortunately, it seems that no Org developer is actually using "org-mobile.el" or knows its internal. Apparently, mobileOrg expects a specific format for index.org, but it is not documented > It also embarrassed us towards our colleagues, > families, and friends who held the view that proprieatary solutions > are better. I wouldn't bother about people thinking that proprietary solutions are better. > In general, it is preferable to have a test suite which should pass before > releasing these commits. Yes, there is MELPA stable but not every package > is on MELPA. What do you want to test? We don't even know what output is expected. In any case, if you implement some tests, feel free to submit them to this ML. Meanwhile, would the following patch fix the issue? Regards, -- Nicolas Goaziou [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: 0001-org-mobile-Fix-c15db80e32d22dace184203f6f03a0b586bb4.patch --] [-- Type: text/x-diff, Size: 914 bytes --] From 9a88fa0859075aae10a6cd90fb03d533acee1f6b Mon Sep 17 00:00:00 2001 From: Nicolas Goaziou <mail@nicolasgoaziou.fr> Date: Sat, 24 Oct 2015 11:15:31 +0200 Subject: [PATCH] org-mobile: Fix c15db80e32d22dace184203f6f03a0b586bb4b50 * lisp/org-mobile.el (org-mobile-create-index-file): Fix c15db80e32d22dace184203f6f03a0b586bb4b50. --- lisp/org-mobile.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lisp/org-mobile.el b/lisp/org-mobile.el index 6c7c8d0..17600bd 100644 --- a/lisp/org-mobile.el +++ b/lisp/org-mobile.el @@ -439,8 +439,8 @@ agenda view showing the flagged items." (if org-mobile-use-encryption org-mobile-encryption-tempfile target-file) - (insert "#+READONLY\n") (while (setq entry (pop def-todo)) + (insert "#+READONLY\n") (setq kwds (mapcar (lambda (x) (if (string-match "(" x) (substring x 0 (match-beginning 0)) x)) -- 2.6.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-10-24 9:28 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-10-18 22:43 A recent mobile org change causing sync issues Steve Harman 2015-10-19 6:48 ` Nicolas Goaziou 2015-10-19 14:52 ` Steve Harman 2015-10-24 9:29 ` Nicolas Goaziou
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).