On Mon, Jan 06, 2014 at 05:45:20PM +0100, Michael Schaller wrote: > > On 01/06/2014 05:35 PM, Julian Andres Klode wrote: > >It still needs test cases and documentation in doc, but should work. I > >thus did not commit it. > > > That looks good to me. Do you intend to submit it? > If so I can make the needed apt_get.Dependency and > apt.package.BaseDependency changes...
Well, you can already start. I'm only missing stuff not relevant for you (tests & docs), so there's no problem if you go ahead and do the remaining stuff before I commit my part. > > >It does something completely different. The point of __dstr > >is not to make the self.relation member something modern, > >but to make client code that does > > dep.relation == "<<" > >checks work. > > > You are correct. My proposal wouldn't work for this use case. > > Now that we've discussed all this how about documenting > BaseDependency.__dstr properly so that a reader knows what it is > for? What else do you want documented there? It seems to be explained """Helper to make > match >> and < match <<""" -- Julian Andres Klode - Debian Developer, Ubuntu Member See http://wiki.debian.org/JulianAndresKlode and http://jak-linux.org/. Please do not top-post if possible. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
