linutil icon indicating copy to clipboard operation
linutil copied to clipboard

fix(system-cleanup): deprecated zypper `tidy` command

Open Jaredy899 opened this issue 11 months ago • 5 comments

Type of Change

  • [ ] New feature
  • [x] Bug fix
  • [ ] Documentation update
  • [ ] Refactoring
  • [ ] Hotfix
  • [ ] Security patch
  • [ ] UI/UX improvement

Description

Tidy command no longer works for zypper. replaced with cleaning repos.

Testing

Tested in tumbleweed and leap.

Impact

Cleanup system

Issues / other PRs related

  • Resolves #999

Additional Information

Checklist

  • [x] My code adheres to the coding and style guidelines of the project.
  • [x] I have performed a self-review of my own code.
  • [x] I have commented my code, particularly in hard-to-understand areas.
  • [x] I have made corresponding changes to the documentation.
  • [x] My changes generate no errors/warnings/merge conflicts.

Jaredy899 avatar Jan 12 '25 04:01 Jaredy899

Does this work on older versions of the package manager as well?

lj3954 avatar Jan 12 '25 19:01 lj3954

Tumbleweed is rolling release so if it's updated it works there. Leap 15.0 didn't work there. But it's 6 years old. Does work on Leap 15.2, 15.5 and 15.6. Also works on Suse linux enterprise 15.5 SP5.

Jaredy899 avatar Jan 12 '25 20:01 Jaredy899

we could add a check for Leap 15.0 and older

adamperkowski avatar Jan 16 '25 17:01 adamperkowski

I retested on a fresh proxmox vm with 15.0 and both commands do work. As far as I can see (and I'm new to suse so please correct me if I'm wrong), but it looks like tidy is a package you have to install?

Jaredy899 avatar Jan 16 '25 18:01 Jaredy899

It's in a different pr. Still trying to get the hang of doing prs

Jaredy899 avatar Jan 28 '25 21:01 Jaredy899