Review Request 121729: implement DependencyInfo::multiArchAnnotation and filter invalid deps

[Harald Sitter]

> On Dec. 29, 2014, 12:31 a.m., Aleix Pol Gonzalez wrote:
> > I like it, how hard would it be to add a unit test for this?

Should be easy. Though I do wonder if we should just throw some time at figuring out a proper test setup and then take it from there. apt itself allows for 100% mock data to be supplied (package lists, caches, configs, what have you), but we seem to lack some of the facilities to construct a backend with an entire set of mock data.


- Harald


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121729/#review72647
-----------------------------------------------------------


On Dec. 28, 2014, 11:27 p.m., Harald Sitter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121729/
> -----------------------------------------------------------
> 
> (Updated Dec. 28, 2014, 11:27 p.m.)
> 
> 
> Review request for Kubuntu, LibQApt, Aleix Pol Gonzalez, and Michael Stemle.
> 
> 
> Repository: libqapt
> 
> 
> Description
> -------
> 
> this is a QString property that may be empty or contain the multiarch
> annotation of a dependency
> 
> looks like this:
> "Depends: foo:any"
> 
> this will enable deb-installer to properly handle Multi-Arch: allow mode
> 
> 
> Diffs
> -----
> 
>   src/dependencyinfo.h 3ce031ac37b3684b048ddc0f353b5f20b1b92e6f 
>   src/dependencyinfo.cpp 9e58054a92cf96f6ba80f89626b0bc5dd075411a 
> 
> Diff: https://git.reviewboard.kde.org/r/121729/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Harald Sitter
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kubuntu-devel/attachments/20141229/18c9ba71/attachment.html>