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

spec file rpm preventing removal #9558

Closed
2 tasks done
ilippert opened this issue Jan 12, 2023 · 6 comments · Fixed by #9604
Closed
2 tasks done

spec file rpm preventing removal #9558

ilippert opened this issue Jan 12, 2023 · 6 comments · Fixed by #9604
Labels
bug Confirmed bugs or reports that are very likely to be bugs build-system good first issue An issue intended for project-newcomers. Varies in difficulty. os: linux

Comments

@ilippert
Copy link
Contributor

ilippert commented Jan 12, 2023

JabRef version

5.9 (latest release)

Operating system

GNU / Linux

Details on version and operating system

No response

Checked with the latest development build

  • I made a backup of my libraries before testing the latest development version.
  • I have tested the latest development version and the problem persists

Steps to reproduce the behaviour

I have installed various JabRef versions – well so it seems because the upgrade/removal does not work properly.
For reference, I have
jabref.x86_64 5.9.6-1
jabref.x86_64 5.10.10-1
jabref.x86_64 6.0.3206-1

And as @Siedlerchr suggested in #9554 (comment) the error message reported in #9554 (comment) might be related to a missing check for file existence in the spec file.

Appendix

No response

@Siedlerchr Siedlerchr added bug Confirmed bugs or reports that are very likely to be bugs os: linux build-system good first issue An issue intended for project-newcomers. Varies in difficulty. labels Jan 12, 2023
@github-project-automation github-project-automation bot moved this to Normal priority in Prioritization Jan 12, 2023
@github-project-automation github-project-automation bot moved this to Free to take in Good First Issues Jan 12, 2023
@sambo57u
Copy link

I can confirm this. It did create a mess. I could remove them one by one via:
rpm -e --allmatches --nodeps --noscripts PackageName

@Siedlerchr
Copy link
Member

The spec file is here: https://github.com/JabRef/jabref/blob/main/buildres/linux/jabref.spec
Feel free to create a PR modifying it

@Siedlerchr
Copy link
Member

@ilippert can you test if this version works with installing/uninstalling? https://builds.jabref.org/pull/9604/merge/
I modified the rm command, but have no Linux system here to test this

@ilippert
Copy link
Contributor Author

ilippert commented Feb 7, 2023

@ilippert can you test if this version works with installing/uninstalling? https://builds.jabref.org/pull/9604/merge/ I modified the rm command, but have no Linux system here to test this

installing works.

removing results in

Running transaction
  Preparing        :                                                                                                                          1/1 
  Running scriptlet: jabref-5.10.30121-1.x86_64                                                                                               1/1 
/var/tmp/rpm-tmp.AcSTvP: line 12: UTILITY_SCRIPTS: command not found
/var/tmp/rpm-tmp.AcSTvP: line 15: uninstall_default_mime_handler: command not found
error: %preun(jabref-5.10.30121-1.x86_64) scriptlet failed, exit status 127

Error in PREUN scriptlet in rpm package jabref
  Verifying        : jabref-5.10.30121-1.x86_64                                                                                               1/1 

Failed:
  jabref-5.10.30121-1.x86_64                                                                                                                      

Error: Transaction failed

an error still.

@Siedlerchr
Copy link
Member

Thanks, thing I found the difference, the UTITLY Scripts command was replaced in the recent jdk version. Pushed an update
A new version on that branch is currently building (Will take roughly 20 to 30 minutes).

@ilippert
Copy link
Contributor Author

ilippert commented Feb 7, 2023

this installs and removes without friction. thanks a lot!

@github-project-automation github-project-automation bot moved this from Normal priority to Done in Prioritization Feb 8, 2023
@github-project-automation github-project-automation bot moved this from Free to take to Done in Good First Issues Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed bugs or reports that are very likely to be bugs build-system good first issue An issue intended for project-newcomers. Varies in difficulty. os: linux
Projects
Archived in project
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants