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

2023-09-16 Thread David Bremner
Nicholas D Steeves  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



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

2023-09-16 Thread Nicholas D Steeves
Hi Daniel and David,

On Wed, 13 Sept 2023 at 04:17, Daniel Kahn Gillmor 
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.


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

2023-09-13 Thread Daniel Kahn Gillmor
Control: forwarded 1051719 https://github.com/jrblevin/markdown-mode/pull/725

Hi Nicholas--

On Tue 2023-09-12 17:02:55 -0400, Nicholas D Steeves wrote:
> 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.

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?

Thanks for maintaining elpa-markdown-mode in debian!

   --dkg


signature.asc
Description: PGP signature


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

2023-09-12 Thread Nicholas D Steeves
Control: tag -1 upstream

Thanks for reporting!

Daniel Kahn Gillmor  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


signature.asc
Description: PGP signature


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

2023-09-11 Thread 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