emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* org-mobile-push and MobileOrg compatibility issue
@ 2015-09-04 22:28 Vamsi Vytla
  2015-09-05  8:45 ` Nicolas Goaziou
  0 siblings, 1 reply; 6+ messages in thread
From: Vamsi Vytla @ 2015-09-04 22:28 UTC (permalink / raw)
  To: emacs-orgmode, gordienkos

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

M-x 'org-mobile-push', with the latest org-mode leaves the files
incompatible with MobileOrg Android application. There have been no changes
in MobileOrg for over a year.

I bisected the latest org-mobile related changes and noticed that reverting
this one line below "fixes" the issue (at least for me). Since there aren't
any tests, it's hard to validate changes and understand things easily.

If anybody can point me in the right direction, I would love to look into
it further.

Cheers!

diff --git a/lisp/org-mobile.el b/lisp/org-mobile.el
index 6c7c8d0..7e1127c 100644
--- a/lisp/org-mobile.el
+++ b/lisp/org-mobile.el
@@ -446,7 +446,7 @@ agenda view showing the flagged items."
                                         x))
                           (cdr entry)))
        (insert "#+TODO: " (mapconcat 'identity kwds " ") "\n")
-       (setq dwds (or (member "|" kwds) (last kwds))
+       (setq dwds (member "|" kwds)
              twds (org-delete-all dwds kwds)
              todo-kwds (org-delete-all twds todo-kwds)
              done-kwds (org-delete-all dwds done-kwds)))

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

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

* Re: org-mobile-push and MobileOrg compatibility issue
  2015-09-04 22:28 org-mobile-push and MobileOrg compatibility issue Vamsi Vytla
@ 2015-09-05  8:45 ` Nicolas Goaziou
  2015-10-12 17:42   ` Vamsi Vytla
  0 siblings, 1 reply; 6+ messages in thread
From: Nicolas Goaziou @ 2015-09-05  8:45 UTC (permalink / raw)
  To: Vamsi Vytla; +Cc: gordienkos, emacs-orgmode

Hello,

Vamsi Vytla <vamsi.vytla@gmail.com> writes:

> M-x 'org-mobile-push', with the latest org-mode leaves the files
> incompatible with MobileOrg Android application. There have been no changes
> in MobileOrg for over a year.
>
> I bisected the latest org-mobile related changes and noticed that reverting
> this one line below "fixes" the issue (at least for me). Since there aren't
> any tests, it's hard to validate changes and understand things easily.
>
> If anybody can point me in the right direction, I would love to look into
> it further.
>
> Cheers!
>
> diff --git a/lisp/org-mobile.el b/lisp/org-mobile.el
> index 6c7c8d0..7e1127c 100644
> --- a/lisp/org-mobile.el
> +++ b/lisp/org-mobile.el
> @@ -446,7 +446,7 @@ agenda view showing the flagged items."
>                                          x))
>                            (cdr entry)))
>         (insert "#+TODO: " (mapconcat 'identity kwds " ") "\n")
> -       (setq dwds (or (member "|" kwds) (last kwds))
> +       (setq dwds (member "|" kwds)
>               twds (org-delete-all dwds kwds)
>               todo-kwds (org-delete-all twds todo-kwds)
>               done-kwds (org-delete-all dwds done-kwds)))

IIRC this change was introduced to fix another bug. It might be useful
to discuss with the author of this change. Another option is to use

  (or (member "|" kwds) (cons "|" (last kwds)))


Regards,

-- 
Nicolas Goaziou

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

* Re: org-mobile-push and MobileOrg compatibility issue
  2015-09-05  8:45 ` Nicolas Goaziou
@ 2015-10-12 17:42   ` Vamsi Vytla
  2015-10-13 19:28     ` Chris Patti
  2015-10-13 20:15     ` Nicolas Goaziou
  0 siblings, 2 replies; 6+ messages in thread
From: Vamsi Vytla @ 2015-10-12 17:42 UTC (permalink / raw)
  To: emacs-orgmode, gordienkos

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

Hi Nicolas,

I can guarantee you that your suggestion did NOT work for me. I haven't
heard anything from Sergey here. Looks like people at MobileOrg-Android
maybe running into the same issue:

https://github.com/matburt/mobileorg-android/issues/472#issuecomment-147403622

Any thoughts?

Vamsi

On Sat, Sep 5, 2015 at 1:44 AM Nicolas Goaziou <mail@nicolasgoaziou.fr>
wrote:

> Hello,
>
> Vamsi Vytla <vamsi.vytla@gmail.com> writes:
>
> > M-x 'org-mobile-push', with the latest org-mode leaves the files
> > incompatible with MobileOrg Android application. There have been no
> changes
> > in MobileOrg for over a year.
> >
> > I bisected the latest org-mobile related changes and noticed that
> reverting
> > this one line below "fixes" the issue (at least for me). Since there
> aren't
> > any tests, it's hard to validate changes and understand things easily.
> >
> > If anybody can point me in the right direction, I would love to look into
> > it further.
> >
> > Cheers!
> >
> > diff --git a/lisp/org-mobile.el b/lisp/org-mobile.el
> > index 6c7c8d0..7e1127c 100644
> > --- a/lisp/org-mobile.el
> > +++ b/lisp/org-mobile.el
> > @@ -446,7 +446,7 @@ agenda view showing the flagged items."
> >                                          x))
> >                            (cdr entry)))
> >         (insert "#+TODO: " (mapconcat 'identity kwds " ") "\n")
> > -       (setq dwds (or (member "|" kwds) (last kwds))
> > +       (setq dwds (member "|" kwds)
> >               twds (org-delete-all dwds kwds)
> >               todo-kwds (org-delete-all twds todo-kwds)
> >               done-kwds (org-delete-all dwds done-kwds)))
>
> IIRC this change was introduced to fix another bug. It might be useful
> to discuss with the author of this change. Another option is to use
>
>   (or (member "|" kwds) (cons "|" (last kwds)))
>
>
> Regards,
>
> --
> Nicolas Goaziou
>

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

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

* Re: org-mobile-push and MobileOrg compatibility issue
  2015-10-12 17:42   ` Vamsi Vytla
@ 2015-10-13 19:28     ` Chris Patti
  2015-10-14  6:29       ` Eric S Fraga
  2015-10-13 20:15     ` Nicolas Goaziou
  1 sibling, 1 reply; 6+ messages in thread
From: Chris Patti @ 2015-10-13 19:28 UTC (permalink / raw)
  To: Vamsi Vytla; +Cc: gordienkos, emacs-orgmode

I must confess I am not wild about the whole way MobileOrg purports to function.

I don't really want to have to push or pull my OrgFiles to/from
mobile.  With mechanisms like Dropbox what I *really* want is to be
able to access and manipulate Org content on my device directly.

-Chris

On Mon, Oct 12, 2015 at 1:42 PM, Vamsi Vytla <vamsi.vytla@gmail.com> wrote:
> Hi Nicolas,
>
> I can guarantee you that your suggestion did NOT work for me. I haven't
> heard anything from Sergey here. Looks like people at MobileOrg-Android
> maybe running into the same issue:
>
> https://github.com/matburt/mobileorg-android/issues/472#issuecomment-147403622
>
> Any thoughts?
>
> Vamsi
>
> On Sat, Sep 5, 2015 at 1:44 AM Nicolas Goaziou <mail@nicolasgoaziou.fr>
> wrote:
>>
>> Hello,
>>
>> Vamsi Vytla <vamsi.vytla@gmail.com> writes:
>>
>> > M-x 'org-mobile-push', with the latest org-mode leaves the files
>> > incompatible with MobileOrg Android application. There have been no
>> > changes
>> > in MobileOrg for over a year.
>> >
>> > I bisected the latest org-mobile related changes and noticed that
>> > reverting
>> > this one line below "fixes" the issue (at least for me). Since there
>> > aren't
>> > any tests, it's hard to validate changes and understand things easily.
>> >
>> > If anybody can point me in the right direction, I would love to look
>> > into
>> > it further.
>> >
>> > Cheers!
>> >
>> > diff --git a/lisp/org-mobile.el b/lisp/org-mobile.el
>> > index 6c7c8d0..7e1127c 100644
>> > --- a/lisp/org-mobile.el
>> > +++ b/lisp/org-mobile.el
>> > @@ -446,7 +446,7 @@ agenda view showing the flagged items."
>> >                                          x))
>> >                            (cdr entry)))
>> >         (insert "#+TODO: " (mapconcat 'identity kwds " ") "\n")
>> > -       (setq dwds (or (member "|" kwds) (last kwds))
>> > +       (setq dwds (member "|" kwds)
>> >               twds (org-delete-all dwds kwds)
>> >               todo-kwds (org-delete-all twds todo-kwds)
>> >               done-kwds (org-delete-all dwds done-kwds)))
>>
>> IIRC this change was introduced to fix another bug. It might be useful
>> to discuss with the author of this change. Another option is to use
>>
>>   (or (member "|" kwds) (cons "|" (last kwds)))
>>
>>
>> Regards,
>>
>> --
>> Nicolas Goaziou



-- 
Christopher Patti - Geek At Large | GTalk: cpatti@gmail.com | AIM:
chrisfeohpatti | P: (260) 54PATTI
"Technology challenges art, art inspires technology." - John Lasseter, Pixar

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

* Re: org-mobile-push and MobileOrg compatibility issue
  2015-10-12 17:42   ` Vamsi Vytla
  2015-10-13 19:28     ` Chris Patti
@ 2015-10-13 20:15     ` Nicolas Goaziou
  1 sibling, 0 replies; 6+ messages in thread
From: Nicolas Goaziou @ 2015-10-13 20:15 UTC (permalink / raw)
  To: Vamsi Vytla; +Cc: gordienkos, emacs-orgmode

Hello,

Vamsi Vytla <vamsi.vytla@gmail.com> writes:

> I can guarantee you that your suggestion did NOT work for me. I haven't
> heard anything from Sergey here. Looks like people at MobileOrg-Android
> maybe running into the same issue:
>
> https://github.com/matburt/mobileorg-android/issues/472#issuecomment-147403622
>
> Any thoughts?

Not really. I hesitate to revert the change since it looks correct. It
may be a problem on the MobileOrg side.

Could you show the difference between an index file generated with the
current org-mobile.el and one with your patched org-mobile.el?


Regards,

-- 
Nicolas Goaziou

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

* Re: org-mobile-push and MobileOrg compatibility issue
  2015-10-13 19:28     ` Chris Patti
@ 2015-10-14  6:29       ` Eric S Fraga
  0 siblings, 0 replies; 6+ messages in thread
From: Eric S Fraga @ 2015-10-14  6:29 UTC (permalink / raw)
  To: Chris Patti; +Cc: emacs-orgmode

On Tuesday, 13 Oct 2015 at 15:28, Chris Patti wrote:
> I must confess I am not wild about the whole way MobileOrg purports to function.
>
> I don't really want to have to push or pull my OrgFiles to/from
> mobile.  With mechanisms like Dropbox what I *really* want is to be
> able to access and manipulate Org content on my device directly.

Yes, we'd all like that but that requires Emacs basically...  you *can*
run Emacs on an Android device, just not very well in my opinion.  You
could try and see if it works for you:

    http://play.google.com/store/apps/details?id=com.zielm.emacs

My solution was to buy and OpenPandora palmtop computer that runs full
Debian Linux (and hence any version of Emacs I want!).

-- 
: Eric S Fraga (0xFFFCF67D), Emacs 25.0.50.2, Org release_8.3.2-161-gd2ac25

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

end of thread, other threads:[~2015-10-14  6:29 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-04 22:28 org-mobile-push and MobileOrg compatibility issue Vamsi Vytla
2015-09-05  8:45 ` Nicolas Goaziou
2015-10-12 17:42   ` Vamsi Vytla
2015-10-13 19:28     ` Chris Patti
2015-10-14  6:29       ` Eric S Fraga
2015-10-13 20:15     ` 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).