Bug 2158829

Summary:	cython-mode.el:240:10: Compiler warning
Product:	[Fedora] Fedora	Reporter:	Edgar Hoch <edgar.hoch>
Component:	Cython	Assignee:	Edgar Hoch <edgar.hoch>
Status:	NEW ---	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	38	CC:	igor.raits, mhroncok, ndbecker2, python-packagers-sig, steve.traylen
Target Milestone:	---
Target Release:	---
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Edgar Hoch 2023-01-06 17:53:58 UTC

Description of problem:
When I start Emacs 28.2 on Fedora 37 for the first time, I got the following  compiler warning:

Warning (comp): cython-mode.el:240:10: Warning: the function ‘python-nav-end-of-statement’ is not known to be defined. Disable showing Disable logging


Version-Release number of selected component (if applicable):
emacs-cython-mode-0.29.32-2.fc37.noarch
emacs-28.2-1.fc37.x86_64


How reproducible:
First start of Emacs 28.2 on Fedora 37.

Steps to Reproduce:
1. Check in ~/.emacs.d/eln-cache/ exists. Delete (or rename) it
   to simulate a fresh, first start of Emacs on this Fedora version.
2. Start Emacs
3. Read the Warning buffer

Comment 1 Miro Hrončok 2023-01-06 21:13:19 UTC

Do you know how to fix this?

Comment 2 Edgar Hoch 2023-01-06 22:08:10 UTC

I think replacing python-nav-end-of-statement by cython-end-of-statement at line 240 will solve the problem.

Comment 3 Miro Hrončok 2023-01-18 13:31:23 UTC

Edgar, would you be willing to maintain the emacs files for Cython? I must admit that they are not a priority for me.

Comment 4 Edgar Hoch 2023-01-19 16:32:38 UTC

I can maintain it. But currently I don't have the rights to do so.

I had read some documents in the past how to become a maintainer, because I was thinking about providing phonetic software "praat" as a package to Fedora. Currently I build praat packages only four my univerity institute.

Comment 5 Miro Hrončok 2023-01-19 16:51:43 UTC

Thank you!

I can push the buttons for you if you provide the actual changes. You can see the last emacs change done by me:

1) I've opened a PR in https://github.com/cython/emacs-cython-mode/pull/1
2) I've opened a RP in https://src.fedoraproject.org/rpms/Cython/pull-request/37 including a patch derived from the upstream PR

As a non-packager you would follow https://docs.fedoraproject.org/en-US/ci/pull-requests/#_you_are_not_a_packager

Let me know if you hit any blockers or problems. You can find me e.g. in #fedora-python libra IRC chat (bridged to Matrix https://chat.fedoraproject.org/#/room/#python:fedoraproject.org if you prefer that).

Comment 6 Miro Hrončok 2023-07-25 10:45:57 UTC

The emacs files were removed in Fedora 39+. (The emacs-cython-mode package can be added to Fedora as a standalone package, but it has not yet happened.)

This can be fixed in Cython for Fedora 37+38 only now.