Review Request 122023: Expose the site apt property in QApt::Package::site()

Harald Sitter sitter at kde.org
Tue Jan 13 08:29:49 UTC 2015


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

Ship it!



src/package.cpp
<https://git.reviewboard.kde.org/r/122023/#comment51380>

    That probably needs to use ::fromUtf8 because of [1].
    
    And with that said, I think origin actually should be changed to fromUtf8 as well if you find a minute. AFAIK there is no encoding limitiation on apt Release files, so what comes out of apt-pkg could very well be !latin for origin as well.
    
    [1] http://en.wikipedia.org/wiki/Internationalized_domain_name


- Harald Sitter


On Jan. 13, 2015, 2:53 a.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122023/
> -----------------------------------------------------------
> 
> (Updated Jan. 13, 2015, 2:53 a.m.)
> 
> 
> Review request for Kubuntu, Michael Stemle and Harald Sitter.
> 
> 
> Repository: libqapt
> 
> 
> Description
> -------
> 
> This exposes public API in apt, to QApt. Quite straightforward.
> 
> The reason for needing this is that the filtering we're doing currently by Origin name is quite broken and I'd like to use the "Site" property.
> 
> 
> Diffs
> -----
> 
>   src/package.h 19c5f19 
>   src/package.cpp fe58e1b 
> 
> Diff: https://git.reviewboard.kde.org/r/122023/diff/
> 
> 
> Testing
> -------
> 
> Not much, builds, can be called.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kubuntu-devel/attachments/20150113/0c93004d/attachment.html>


More information about the kubuntu-devel mailing list