On Tue, Jan 05, 2010 at 11:40:41PM +0000, Roger Leigh wrote: > On Wed, Jan 06, 2010 at 12:29:10AM +0100, Ferenc Wagner wrote: > > After a second thought, my patch probably won't solve the problem of > > experimental (or other NotAutomatic) dependencies: in the best case it > > fixes NotAutomatic upgrades only. I'll look in the dependency problem > > if this track is deemed reasonable. > > Thanks! I'll look over your patch over the weekend. You might want to > also check out the dependency resolver changes on the buildd branch, > since I will be reviewing and applying these first. As a result, your > patch will be easier to merge if it applies on top of these changes.
If you check out the current sbuild from git://git.debian.org/git/sbuild this now has support for the old style "internal" dependency resolver (moved from lib/Sbuild/Build.pm to lib/Sbuild/InternalBuildDepSatisfier.pm) and an alternative resolver which uses aptitude. Your patch should ideally be against the internal resolver, though I would appreciate your feedback on whether the new aptitude resolver works correctly for you. WRT your comments about unnecessary whitespace changes, that's not a problem. If reorganising the code is going to give a cleaner patch, then please do so. The main concern patching the internal resolver is not to alter the existing code such that it behaves differently when working with a single distribution which could cause packages to FTBFS. Regards, Roger -- .''`. Roger Leigh : :' : Debian GNU/Linux http://people.debian.org/~rleigh/ `. `' Printing on GNU/Linux? http://gutenprint.sourceforge.net/ `- GPG Public Key: 0x25BFB848 Please GPG sign your mail.
signature.asc
Description: Digital signature
