Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update RPM related stuff to 4.16.1.3 #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Thaodan
Copy link

@Thaodan Thaodan commented Oct 27, 2022

  • Clean file follow flycheck

(cherry picked from commit ​4fdad909b8d2d47b9d776d4fadecb38a6f6bc6be)

@jn64
Copy link

jn64 commented Aug 12, 2023

Most of the functions changed here don't work for me on Emacs 28.2. Specifically, all of the functions that have no arguments after the removal of &optional arg, but still contain (interactive "p"), don't work when called interactively with M-x.

For example doing M-x rpm-insert-tag results in this error:

funcall-interactively: Wrong number of arguments: ((t) nil "Insert or change a tag." (interactive "p") (if current-prefix-arg (rpm-change) (rpm-insert))), 1

Should the (interactive "p") for those functions with no args be changed to (interactive)? It seems to fix the problem for me, but I don't fully understand the implication of the "p".


I also had to pick e95001c to get the package working at all. So the problem can be repro'd with master + e95001c + 93259f7 (this PR) on Emacs 28.2.

Same with the version at https://github.com/Thaodan/rpm-spec-mode/tree/thaodan/personal_merger, meaning the other collected patches don't affect this problem.

@jn64
Copy link

jn64 commented Aug 12, 2023

I guess this is why @zzndb reverted most of these changes in zzndb@9fd1536 (leaving only the changes actually related to RPM 4.16.1.3). Problem is solved by picking zzndb@9fd1536 on top of previously mentioned.

You might want to pick that for https://github.com/Thaodan/rpm-spec-mode/tree/thaodan/personal_merger too.

@Thaodan
Copy link
Author

Thaodan commented Aug 12, 2023 via email

@jn64
Copy link

jn64 commented Aug 13, 2023 via email

* clean file follow flycheck

(cherry picked from commit ​4fdad909b8d2d47b9d776d4fadecb38a6f6bc6be)

Signed-off-by: Björn Bidar <[email protected]>
@Thaodan
Copy link
Author

Thaodan commented Feb 22, 2024

Do you mean that the functions work for you when called interactively? And on what version of Emacs?

Sorry for the late reply. I didn't test, the functions did not work fine, I just didn't test all of the changes.
I reverted the changes that were causing the errors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants