Bug#1051719: markdown-mode.el produces excessive warnings

[Daniel Kahn Gillmor]

Package: elpa-markdown-mode
Version: 2.5-1

I get a lot of warnings from markdown-mode when i launch emacs. the
buffer contains the following:

⛔ Warning (comp): markdown-mode.el:189:2: Warning: custom-declare-variable `markdown-indent-on-enter' docstring has wrong usage of unescaped single quotes (use \= or different quoting)
⛔ Warning (comp): markdown-mode.el:491:2: Warning: custom-declare-variable `markdown-split-window-direction' docstring has wrong usage of unescaped single quotes (use \= or different quoting)
⛔ Warning (comp): markdown-mode.el:517:2: Warning: custom-declare-variable `markdown-live-preview-delete-export' docstring has wrong usage of unescaped single quotes (use \= or different quoting)
⛔ Warning (comp): markdown-mode.el:855:2: Warning: defconst `markdown-regex-wiki-link' docstring has wrong usage of unescaped single quotes (use \= or different quoting)
⛔ Warning (comp): markdown-mode.el:1303:2: Warning: defconst `markdown-fenced-block-pairs' docstring has wrong usage of unescaped single quotes (use \= or different quoting)
⛔ Warning (comp): markdown-mode.el:1608:24: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:1608:39: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:1619:45: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:1619:60: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:1630:38: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:1645:17: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:2551:28: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:2633:2: Warning: docstring has wrong usage of unescaped single quotes (use \= or different quoting)
⛔ Warning (comp): markdown-mode.el:2752:47: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:2783:28: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:2966:33: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:3014:2: Warning: docstring has wrong usage of unescaped single quotes (use \= or different quoting)
⛔ Warning (comp): markdown-mode.el:3020:2: Warning: docstring has wrong usage of unescaped single quotes (use \= or different quoting)
⛔ Warning (comp): markdown-mode.el:3042:2: Warning: docstring has wrong usage of unescaped single quotes (use \= or different quoting)
⛔ Warning (comp): markdown-mode.el:3096:45: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:3097:49: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:4462:64: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:4468:26: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:4518:2: Warning: docstring has wrong usage of unescaped single quotes (use \= or different quoting)
⛔ Warning (comp): markdown-mode.el:4960:2: Warning: docstring has wrong usage of unescaped single quotes (use \= or different quoting)
⛔ Warning (comp): markdown-mode.el:5770:58: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:6220:47: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:6220:62: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:6470:15: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:6470:30: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:6499:30: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:6499:45: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:6503:26: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:6503:41: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:6550:30: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:6550:45: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:6554:26: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:6554:41: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:6578:43: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:6833:23: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:6879:35: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:6879:51: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:7659:4: Warning: More than one doc string
⛔ Warning (comp): markdown-mode.el:7668:4: Warning: More than one doc string
⛔ Warning (comp): markdown-mode.el:8134:11: Warning: ‘inhibit-point-motion-hooks’ is an obsolete variable (as of 25.1); use ‘cursor-intangible-mode’ or ‘cursor-sensor-mode’ instead
⛔ Warning (comp): markdown-mode.el:8357:52: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:8375:38: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:8501:31: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:8736:45: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:8736:62: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:8738:45: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:8738:62: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:8824:91: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:8825:89: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:9030:23: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:9061:26: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:9086:46: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:9241:19: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.
⛔ Warning (comp): markdown-mode.el:9242:28: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:9258:46: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:9259:33: Warning: ‘point-at-eol’ is an obsolete function (as of 29.1); use ‘line-end-position’ or ‘pos-eol’ instead.
⛔ Warning (comp): markdown-mode.el:9636:23: Warning: ‘point-at-bol’ is an obsolete function (as of 29.1); use ‘line-beginning-position’ or ‘pos-bol’ instead.

I'm using emacs 1:29.1+1-5 with emacsen-common 3.0.5.

This is well over 50% of all the warnings that are announced by emacs
for the set of elpa packages i have installed.

Looks to me like most or all of these warnings should be cleaned up,
which would make it easier to see meaningful/substantive warnings if
they do appear.

--dkg

[Nicholas D Steeves]

Control: tag -1 upstream

Thanks for reporting!

Daniel Kahn Gillmor <d...@fifthhorseman.net> writes:

> I get a lot of warnings from markdown-mode when i launch emacs. the
> buffer contains the following:

[snip]

> I'm using emacs 1:29.1+1-5 with emacsen-common 3.0.5.
>
> This is well over 50% of all the warnings that are announced by emacs
> for the set of elpa packages i have installed.
>
> Looks to me like most or all of these warnings should be cleaned up,
> which would make it easier to see meaningful/substantive warnings if
> they do appear.

Agreed! Are you interested in forwarding this bug upstream to
https://jblevins.org/projects/markdown-mode, or would you prefer to wait
for someone else to do so? The reason this is important is because
Emacs29 compatibility affects more than just Debian users, so the fix
shouldn't be limited to just us.

Regards,
Nicholas

[Nicholas D Steeves]

Hi Daniel and David,

On Wed, 13 Sept 2023 at 04:17, Daniel Kahn Gillmor <d...@fifthhorseman.net> wrote:

 

thanks for the review!  I poked around at
https://github.com/jrblevin/markdown-mode and it looks like
518191bfd69130a6f788f7cea88033c183e43736 was intended to resolve these
issues (see the PR i've linked here too). Maybe we just need version 2.6
packaged for debian?

Wow, thank you for your investigation!  Yes,  you're right, 2.6 contains commit:518191b, thus this is the correct action.

 

Thanks for maintaining elpa-markdown-mode in debian!

It's a team effort, so on behalf of the team, you're welcome! :)

David, do you think you'll be able to find time for this or do you want me to come back to it in a week or so?

Cheers,

Nicholas

P.S. I'm AFK at the moment, and I hope my phone doesn't do anything weird to the email.

[David Bremner]

Nicholas D Steeves <nste...@gmail.com> writes:
>
> David, do you think you'll be able to find time for this or do you want me
> to come back to it in a week or so?

I'm not likely to look at it soon, go ahead.

d