emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: Bernt Hansen <bernt@norang.ca>
To: Marco Wahl <marcowahlsoft@gmail.com>
Cc: emacs-orgmode@gnu.org
Subject: Re: Regression - problem with links
Date: Fri, 15 Mar 2019 16:34:07 -0400	[thread overview]
Message-ID: <87h8c3ancw.fsf@norang.ca> (raw)
In-Reply-To: <87lg1fao3c.fsf@norang.ca> (Bernt Hansen's message of "Fri, 15 Mar 2019 16:18:15 -0400")

Bernt Hansen <bernt@norang.ca> writes:

> Marco Wahl <marcowahlsoft@gmail.com> writes:
>
>> Hi,
>>
>> [...]
>>
>>> My links are like this:
>>>
>>>
>>> [[https://somewebsiteaddress.net/path/to/page.do][Timekeeper]]
>>> [[id:eb1085ed-15fe-46a3-8b5e-3971587d1123][OTL 2019]]
>>>
>>> and I want to see
>>>
>>> [1] Timekeeper
>>> [2] OTL 2019
>>>
>>> but instead I get the address like this which is much less useable
>>>
>>> [1] https://somewebsiteaddress.net/path/to/page.do
>>> [2] id:eb1085ed-15fe-46a3-8b5e-3971587d1123
>>
>> This sounds like a bug fixed a few days ago.
>>
>>     https://code.orgmode.org/bzg/org-mode/commit/b1429cdfe043b199bb7be6dc5c97a299954b4e0d
>>
>> Can you check if the fix is in your copy of Org?
>
> I have that commit in my current git version.
>
> Last week I noticed agenda clock reports had file: text before this fix
> and reverted back to normal looking clock reports after however the menu
> from C-c C-o doesn't display link descriptions for me.
>
> I'll try to isolate which commit breaks this.
>
> Regards,
> Bernt

My current master  is

f0dc3a06d (test-org-clock: Fix one test, 2019-03-15)

This may be another case that isn't covered by the fix.
maint displays links correctly but master does not for me in the menu
when you select a link to jump to with C-c C-o from the heading or from
the agenda.

git bisect identifies the following commit:

2b00d6281606ef295dadbe4ddd929ebdd6716d60 is the first bad commit
commit 2b00d6281606ef295dadbe4ddd929ebdd6716d60
Author: Nicolas Goaziou <mail@nicolasgoaziou.fr>
Date:   Sat Mar 9 10:58:41 2019 +0100

    Change bracket link escape syntax

    * contrib/lisp/org-link-edit.el (org-link-edit--link-data):
    * lisp/ob-tangle.el (org-babel-tangle-comment-links): Update
    match-group.
    (org-babel-detangle): Remove unnecessary `org-link-escape' call.
    (org-babel-tangle-jump-to-org): Update match group.
    (org-link-url-hexify):
    (org-link-escape-chars): Remove variables.
    * lisp/ol.el (org-link--decode-compound): Renamed from
      `org-link--unescape-compound'.
    (org-link--decode-single-byte-sequence): Renamed from
    `org-link--unescape-single-byte-sequence'.
    (org-link-make-regexps): Update `org-link-bracket-re' syntax.
    (org-link-encode): New function, renamed from `org-link-escape'.
    (org-link-decode): New function, renamed from `org-link-unescape'.
    (org-link-escape):
    (org-link-unescape): Use new escape syntax.
    (org-link-make-string): Apply new escaping rules.
    (org-link-display-format):
    (org-insert-link): Update match group.
    * lisp/org-agenda.el (org-diary):
    (org-agenda-format-item):
    (org-agenda-to-appt): Update match group.
    * lisp/org-clock.el (org-clocktable-write-default): Update match
    group.
    * lisp/org-element.el (org-element-link-parser): Update match group.
    * lisp/org-mobile.el (org-mobile-escape-olp):
    (org-mobile-locate-entry): Apply function renaming.
    * lisp/org-protocol.el (org-protocol-split-data):
    (org-protocol-parse-parameters): Apply function renaming.
    * lisp/org.el (org-refile): Update match group.
    * testing/README (Interactive testing from within Emacs): Fix
      examples.
    * testing/lisp/test-ol.el (test-ol/encode): Merge old escape tests.
    (test-ol/decode): Merge old unescape tests.
    (test-ol/escape):
    (test-ol/unescape):
    (test-ol/make-string): New tests.
    * testing/lisp/test-org-clock.el (test-org-clock/clocktable/link):
    * testing/lisp/test-org.el (test-org/custom-id):
    (test-org/fuzzy-links):
    * testing/lisp/test-ox.el (test-org-export/resolve-fuzzy-link):
    Update
      tests.

:040000 040000 474dd71a09f6396ef593cf061aa0001968fb5412
976cda41aeecca75da7c5982aeb89a2cde5e5567 M      contrib
:040000 040000 2ab4f30b6deac5cfc4c9e83fc1f2ac013232f8da
d7b6b3c0ff0adda1a557dfe87bffd0d8f17f6659 M      lisp
:040000 040000 133b3e5e20e848a7347ca4c73c517bca4120461d
28f9768c483af5bce6a242889360747be4ac865a M      testing

Regards,
Bernt.

  reply	other threads:[~2019-03-15 20:34 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-15 14:27 Regression - problem with links Bernt Hansen
2019-03-15 15:02 ` Marco Wahl
2019-03-15 20:18   ` Bernt Hansen
2019-03-15 20:34     ` Bernt Hansen [this message]
2019-03-15 21:56       ` Marco Wahl
2019-03-16 14:46         ` Bernt Hansen
2019-03-16 15:54           ` Bernt Hansen
2019-03-16 17:06             ` Nicolas Goaziou
2019-03-16 19:27               ` Bernt Hansen
2019-03-15 15:02 ` Eric S Fraga

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

  List information: https://www.orgmode.org/

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

  git send-email \
    --in-reply-to=87h8c3ancw.fsf@norang.ca \
    --to=bernt@norang.ca \
    --cc=emacs-orgmode@gnu.org \
    --cc=marcowahlsoft@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 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).