Bug 2158829 - cython-mode.el:240:10: Compiler warning
Summary: cython-mode.el:240:10: Compiler warning
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Cython
Version: 38
Hardware: x86_64
OS: Linux
unspecified
low
Target Milestone: ---
Assignee: Edgar Hoch
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-01-06 17:53 UTC by Edgar Hoch
Modified: 2024-05-07 21:35 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2024-05-07 21:35:07 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

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.

Comment 7 Aoife Moloney 2024-05-07 15:55:01 UTC
This message is a reminder that Fedora Linux 38 is nearing its end of life.
Fedora will stop maintaining and issuing updates for Fedora Linux 38 on 2024-05-21.
It is Fedora's policy to close all bug reports from releases that are no longer
maintained. At that time this bug will be closed as EOL if it remains open with a
'version' of '38'.

Package Maintainer: If you wish for this bug to remain open because you
plan to fix it in a currently maintained version, change the 'version' 
to a later Fedora Linux version. Note that the version field may be hidden.
Click the "Show advanced fields" button if you do not see it.

Thank you for reporting this issue and we are sorry that we were not 
able to fix it before Fedora Linux 38 is end of life. If you would still like 
to see this bug fixed and are able to reproduce it against a later version 
of Fedora Linux, you are encouraged to change the 'version' to a later version
prior to this bug being closed.

Comment 8 Edgar Hoch 2024-05-07 21:35:07 UTC
Sorry for the delay. I hadn't the time to look for it, and I don't use the package myself. I only was providing it on the computers on our institute in case someone would need it.
Now the packages doesn't exist on Fedora 39+, I close this bug report. Thanks for the information!


Note You need to log in before you can comment on or make changes to this bug.