On 26/07/11 06:31, Benedikt Morbach wrote:
Resend, as I've incorporated Dan's suggestions from last time.

The feature set is still the same, but I've thought about how removal of 
optdepends should work.
See https://wiki.archlinux.org/index.php/User:Moben/Pacman_OptDepends

I took a quick look at your proposal and they seem fine to me. BTW, feel free to add these to "my" page so that the ideas for this can be kept all in one place. Either that, or add a link through to your page.

And when I say one place...

You could add details in the roadmap:
https://wiki.archlinux.org/index.php/Pacman_Roadmap

And your git repo details here:
https://wiki.archlinux.org/index.php/Pacman_Development


This implements most features from 
https://wiki.archlinux.org/index.php/User:Allan/Pacman_OptDepends

What is there:
     - No regressions afaics
     - Only show uninstalled optdepends during install/upgrade
     - In package info, show [installed] after installed optdepends
     - In package info (local or -ii) show packages which optionally depend on 
the queried package
     - '-Qt' doesn't consider optdepends to be orphans, unless '--nooptdeps/-n' 
is given

I have not taken a decent look at the code yet (although, opt_cmp in src/pacman/util.c needs to be declared "static"), but I did take the patches for a spin in a chroot to see this in action. Everything that is in the list above worked for me!


What is still missing:
     - Show which of the packages listed by '-Qtn' is an optdepend and what 
optdeps on it
     - In package info display the description alongside the "reverse optdeps"
     - Recursive removal of unneeded optdeps
     - Anything listed under "Other Ideas"
     - Tests (mainly needed for the package removal stuff I think)
     - Docs with less sucky english ;-)


---
Benedikt

PS: I'll start working on the rest soon, but that might be delayed
     due to me going on vacation in the first two weeks of August ;-)

Cool. There is not rush here as I think these patches will probably be scheduled to be included in 4.1 given we appear to be thinking about a 4.0 release in the not to distant future. This will also allow us to correct the optdepends syntax enforcement for the addition of ":" due to epoch in the 4.0 release.

Allan


Reply via email to