Merge lp:~manchicken/kubuntu-debug-installer/kubuntu-debug-installer into lp:kubuntu-debug-installer

[Michael D. Stemle, Jr.]

Big re-factoring here, big problems trying to run quality tools to verify I resolved tools. I have code to remove the ddebs entries, but libqapt isn't removing them in my environment. I suspect a problem with my environment, please let me know if you see something broken.

[Philip Muškovac (yofel)]

I only took a quick glance on the merge, but I think you missed the "release" repository. You have X-updates, X-security and X-proposed, but I see no place where you add X without a suffix

[Michael D. Stemle, Jr. (manchicken)]

I've added that base repository now, please re-review.

[Jonathan Thomas (echidnaman)]

One comment, when the cache update transaction finishes, you'll need to call m_backend->reload() to ensure the new package is available. (Build a new cache based on the new source info) Strange things can happen if you don't.

Looks good otherwise. Sorry for the delay in reviewing; real life has been a bit crazy lately...

review: Needs Fixing

[Harald Sitter (apachelogger)]

Do you want a codestyle review as well? Because I am really good at that :P

Also, Jonathan, what is up with the decorator. It looks like libqapt got shot in the head and now its brainz are spread all over debug-installer's code :O

[Michael D. Stemle, Jr. (manchicken)]

On Jul 23, 2013 2:53 AM, "Harald Sitter" <email address hidden> wrote:
>
> Do you want a codestyle review as well? Because I am really good at that
:P

I'll accept style criticism as long as there's a "why" which isn't
arbitrary :-)

>
> Also, Jonathan, what is up with the decorator. It looks like libqapt got
shot in the head and now its brainz are spread all over debug-installer's
code :O

QApt doesn't have error strings, I put that decorator there to give me some
(credit to JT, I did just sin and paste it mostly verbatim).

> --
>
https://code.launchpad.net/~manchicken/kubuntu-debug-installer/kubuntu-debug-installer/+merge/176318
> You are the owner of
lp:~manchicken/kubuntu-debug-installer/kubuntu-debug-installer.

[Harald Sitter (apachelogger)]

Yeah I don't think that decorator thing makes sense at all, but that's a library thing and libqapts fault really. Why would every application using qapt have to carry a verbatim code copy with strings around?

As for style... here comes the style PITA police :P

Please note that we use 4 space indent in the source code. Also please review your added includes, just from looking at the diff it seems like they are not actually all necessary.

> m_txn = NULL;

By convention we use 0 in Qt software. No clue what a txn is, perhaps a more descriptive name would be good.

> QString headerText = i18n("<b>Adding debug symbol sources</b>");

please have a look at i18nc [1] and in particular you'd want to get rid of the <b> and use <title> for example [2]. this applies to the entire diff.

also, why not simply merge the i18nc() into the next line
m_progress.setLabelText(i18nc(...));

and on that note I would make m_progress a pointer to get it out of the header for compile time's sake and to reduce overall startup workload

> hcRepoList

same as with m_txn, no clue what hc stands for....

> QString(DDEB_REPO_FILE)

constructing using QString::fromLatin1(..) is faster.

> int sourceCount = sourceList.entries().count();

I reckon this value is const, yet the code doesn't say so.

> foreach (QApt::SourceEntry repo, hcRepoList)

Not sure whether that is possible, but generally speaking that should be const QApt::SourceEntry &repo to avoid pointless copying.

By convention this should be foreach (...) { ... i.e. opening angle braces on the same line [3]

> m_srcUpdated

variable name again, I guess this should be something like m_srcsListUpdated

> this->find();

convention again, no this unless necessary.

> int m_srcCount;

unused it seems

> add_definitions(-DKDE_DEFAULT_DEBUG_AREA=8675309)

I don't think we'd want a set debug area (seems unnecessary and is only advised for libs/plugins), at any rate we'd not want a static debug area but register one dynamically [4]

> headerText = i18n("<b>Downloading Packages</b>");
like above and indeed it seems like not using headertext to begin with would be a good idea

[1] http://techbase.kde.org/Development/Tutorials/Localization/i18n#Adding_Context_with_i18nc.28.29
[2] http://techbase.kde.org/Development/Tutorials/Localization/i18n_Semantics
[3] https://wiki.kubuntu.org/Kubuntu/Specs/MaverickCodestylePolicy
[4] http://api.kde.org/4.x-api/kdelibs-apidocs/kdecore/html/classKDebug.html#a40c55b7ae17e62f18fa5e44349496e33

[Michael D. Stemle, Jr. (manchicken)]

Doh. I'll be making many of the changes you mentioned. I'll make them
Saturday. Do we have the ability to test now so we can see if it works like
we want it to? I want to get as much lined up for my hackathon Saturday as
I can.
On Jul 24, 2013 4:33 PM, "Harald Sitter" <email address hidden> wrote:

> Yeah I don't think that decorator thing makes sense at all, but that's a
> library thing and libqapts fault really. Why would every application using
> qapt have to carry a verbatim code copy with strings around?
>
> As for style... here comes the style PITA police :P
>
> Please note that we use 4 space indent in the source code. Also please
> review your added includes, just from looking at the diff it seems like
> they are not actually all necessary.
>
> > m_txn = NULL;
>
> By convention we use 0 in Qt software. No clue what a txn is, perhaps a
> more descriptive name would be good.
>
> > QString headerText = i18n("<b>Adding debug symbol sources</b>");
>
> please have a look at i18nc [1] and in particular you'd want to get rid of
> the <b> and use <title> for example [2]. this applies to the entire diff.
>
> also, why not simply merge the i18nc() into the next line
> m_progress.setLabelText(i18nc(...));
>
> and on that note I would make m_progress a pointer to get it out of the
> header for compile time's sake and to reduce overall startup workload
>
> > hcRepoList
>
> same as with m_txn, no clue what hc stands for....
>
> > QString(DDEB_REPO_FILE)
>
> constructing using QString::fromLatin1(..) is faster.
>
> > int sourceCount = sourceList.entries().count();
>
> I reckon this value is const, yet the code doesn't say so.
>
> > foreach (QApt::SourceEntry repo, hcRepoList)
>
> Not sure whether that is possible, but generally speaking that should be
> const QApt::SourceEntry &repo to avoid pointless copying.
>
> By convention this should be foreach (...) { ... i.e. opening angle
> braces on the same line [3]
>
> > m_srcUpdated
>
> variable name again, I guess this should be something like
> m_srcsListUpdated
>
> > this->find();
>
> convention again, no this unless necessary.
>
> > int m_srcCount;
>
> unused it seems
>
> > add_definitions(-DKDE_DEFAULT_DEBUG_AREA=8675309)
>
> I don't think we'd want a set debug area (seems unnecessary and is only
> advised for libs/plugins), at any rate we'd not want a static debug area
> but register one dynamically [4]
>
> > headerText = i18n("<b>Downloading Packages</b>");
> like above and indeed it seems like not using headertext to begin with
> would be a good idea
>
> [1]
> http://techbase.kde.org/Development/Tutorials/Localization/i18n#Adding_Context_with_i18nc.28.29
> [2]
> http://techbase.kde.org/Development/Tutorials/Localization/i18n_Semantics
> [3] https://wiki.kubuntu.org/Kubuntu/Specs/MaverickCodestylePolicy
> [4]
> http://api.kde.org/4.x-api/kdelibs-apidocs/kdecore/html/classKDebug.html#a40c55b7ae17e62f18fa5e44349496e33
> --
>
> https://code.launchpad.net/~manchicken/kubuntu-debug-installer/kubuntu-debug-installer/+merge/176318
> You are the owner of
> lp:~manchicken/kubuntu-debug-installer/kubuntu-debug-installer.
>

[Harald Sitter (apachelogger)]

mh I'll do some QA (been doing that all morning anyway :S)

fwiw, I think I changed some stuff in my LSBRelease, also you may want to use the q_asserts :P

[Harald Sitter (apachelogger)]

krazy2 Analysis

Checkers Run = 39
Files Processed = 37
Total Issues = 13 ...as of July 25 2013 15:07:17 CEST

== messages: For File Type messages ==
1. Check for appending to rc.cpp [rcappend]... Ok!

== cmake: For File Type cmake ==
1. Check for an acceptable copyright [copyright]... Ok!

2. Check that file ends with a newline [endswithnewline]... Ok!

3. Check for an acceptable license [license]... Ok!

4. Check for spelling errors [spelling]... Ok!

== c++: For File Type c++ ==
1. Check for TRUE and FALSE macros [captruefalse]... Ok!

2. Check for methods that return 'const' refs in public classes [constref]... Ok!

3. Check for an acceptable copyright [copyright]... Ok!

4. Check for cpp macros and usage [cpp]... Ok!

5. Check for code that should be considered crashy. [crashy]... Ok!

6. Check single-char QString operations for efficiency [doublequote_chars]... Ok!

7. Check for unwanted doxygen tags in major versions [doxytags]... Ok!

8. Check public classes with private members or d-pointer issues [dpointer]... Ok!

9. Check for QString compares to "" [emptystrcompare]... Ok!

10. Check that file ends with a newline [endswithnewline]... 2 issues found
        ./src/QAptDecorator.cpp: line# 80 (1)
        ./src/LSBRelease.cpp: line# 66 (1)
        Files that do not end with a newline character can cause problems.
        Please add a newline character to the end of the file.

11. Check for C++ ctors that should be declared 'explicit' [explicit]... Ok!

12. Check for foreach loop issues [foreach]... 1 issue found
        ./src/DebugFinder.cpp: non-const ref iterator line#136 (1)
        When not using POD types (int, double, pointer, ...) you should use
        const & for your foreach variables. There are two reasons for this:
        1) Prevents you from the mistake of writing foreach loops that
        modify the list, that is 'foreach(Foo f, list) f.a = f.b = f.c =
        0;' compiles but does not modify the contents of list 2) Saves a
        copy constructor call for each of the list elements

13. Check validity of i18n calls [i18ncheckarg]... 9 issues found
        ./src/LSBRelease.cpp: single adjective as message, probably ambiguous; explain what it refers to following the KUIT context marker line#33
        ./src/DebugFinder.cpp: missing KUIT context marker line#116,196,200,207
        ./src/DebugFinder.cpp: HTML tag 'b' is not advised with KUIT markup line#116,196,200,207
        Make the translators' job easier and detect problems in the usage
        of the i18n() calls. When the fix is not clear, check the Techbase
        article at
        <http://techbase.kde.org/Development/Tutorials/Localization/i18n_Kr
        azy> for more information.

14. Check for invalid icon names [iconnames]... Ok!

15. Check for proper include directives [includes]... 1 issue found
        ./src/QAptDecorator.cpp: include own header first line#26
        Use <..> to include installed headers; cpp file should include
        their own headers first (but below config.h); other rules apply,
        see
        <http://techbase.kde.org/Policies/Library_Code_Policy#Getting_.23in
        cludes_right>. Use include guards in headers with appropriat...

[Michael D. Stemle, Jr. (manchicken)]

Maybe we should package the LSB stuff into its own thing so we can link to
it.
On Jul 25, 2013 8:07 AM, "Harald Sitter" <email address hidden> wrote:

> mh I'll do some QA (been doing that all morning anyway :S)
>
> fwiw, I think I changed some stuff in my LSBRelease, also you may want to
> use the q_asserts :P
>
>
> --
>
> https://code.launchpad.net/~manchicken/kubuntu-debug-installer/kubuntu-debug-installer/+merge/176318
> You are the owner of
> lp:~manchicken/kubuntu-debug-installer/kubuntu-debug-installer.
>

[Harald Sitter (apachelogger)]

please have a look at the cmake macro macro_ensure_version to make sure an appropriate version of libqapt is installed [1]

[1] https://projects.kde.org/projects/kdesupport/phonon/phonon-gstreamer/repository/revisions/master/entry/CMakeLists.txt#L8

[Harald Sitter (apachelogger)]

clang-analyzer is happy

[Michael D. Stemle, Jr. (manchicken)]

That may be tricky until JT gets libqapt ready, yeah?
On Jul 25, 2013 8:21 AM, "Harald Sitter" <email address hidden> wrote:

> please have a look at the cmake macro macro_ensure_version to make sure an
> appropriate version of libqapt is installed [1]
>
> [1]
> https://projects.kde.org/projects/kdesupport/phonon/phonon-gstreamer/repository/revisions/master/entry/CMakeLists.txt#L8
> --
>
> https://code.launchpad.net/~manchicken/kubuntu-debug-installer/kubuntu-debug-installer/+merge/176318
> You are the owner of
> lp:~manchicken/kubuntu-debug-installer/kubuntu-debug-installer.
>

[Harald Sitter (apachelogger)]

smoke results:

- clicked cancel while it was 'updating software softwares' -> cancels only that so it continues looking for debug packages
- adds ddebs entires regardless of whether they are there already
- repeatedly going from 0 to 100 % progress while saying "Updating software sources" is confusing
- debug output needs serious reduction as right now it is flooding .xsession-errors with a bazillion percent values when in debug mode
- canceling the authenticate for the software source changes makes the dailog re-appear, canceling again makes it appear again etc.etc. (in short: there is no way to get rid of it other than actually providing credentials)
- entries to ddebs.list are added 4 times, i.e. after one run I get 4 saucy, 4 saucy-updates, 4 saucy-security etc.etc.
- the update software sources dialog should probably reuse the existing one instead of creating a new one, as it is right now a user will get up to 4 cascaded windows (drkonqi < dbginstallerdialog < debugfinder < software sources)
- when the installer ask whether the new dbg packages should be installed and one cancels the application doesn't actually exit() or anything (also not when actually looking for the packages - may actually be entirely un-exitable)
- conseqeuently to above (presumably) the ddebs.list file is not removed on exit etc.
- entries to ddebs.list are added regardless of whether they are already there, i.e. every run another 4x4 entires will be added to the file
- not sure this is related to the chnanges but worrying regardless....when looking for rekonq and qt the finder will only offer to install libqt4-dbg (no clue why it goes for that, but IIRC we may have special handling that makes that happen), removing the qt bits form the file list will make it recommend rekonq-dbgsym as well

testing cmd:
kubuntu-debug-installer "/usr/bin/rekonq" "/usr/lib/x86_64-linux-gnu/libQtCore.so.4" "/usr/lib/kde4/libkdeinit/libkdeinit4_rekonq.so" "/usr/lib/x86_64-linux-gnu/libQtGui.so.4

[Harald Sitter (apachelogger)]

About the LSB thing... /etc/lsb-release is not backed by a spec or anything and long-term will be replaced by /etc/os-release which is backed by a spec so I should hope that we get a KDElibs way to get os-release data. So, I am not sure it is worth the effort TBH.

[Harald Sitter (apachelogger)]

QA done :P

review: Needs Fixing

[Harald Sitter (apachelogger)]

> That may be tricky until JT gets libqapt ready, yeah?

Yes. No. Maybe. :P

It seems to me libqapt 2.0.65 has the sourceslist API so I guess that would be the required version right now.

I just did the analysis, how/whether the issue can be resolved or whether they were even caused by the diff at hand, I did not look at ;)

[Michael D. Stemle, Jr. (manchicken)]

OK. My hackathon is Saturday the 27th at 12:30 for four hours (in
Champaign, IL USA if anyone wants to join), I should be able to get
something done.
On Jul 25, 2013 8:54 AM, "Harald Sitter" <email address hidden> wrote:

> > That may be tricky until JT gets libqapt ready, yeah?
>
> Yes. No. Maybe. :P
>
> It seems to me libqapt 2.0.65 has the sourceslist API so I guess that
> would be the required version right now.
>
> I just did the analysis, how/whether the issue can be resolved or whether
> they were even caused by the diff at hand, I did not look at ;)
> --
>
> https://code.launchpad.net/~manchicken/kubuntu-debug-installer/kubuntu-debug-installer/+merge/176318
> You are the owner of
> lp:~manchicken/kubuntu-debug-installer/kubuntu-debug-installer.
>

[Harald Sitter (apachelogger)]

much better. I reran the entire test set but there are still some issues, in particular the krazy and smoke issues are blocking for me...

------------------ style ------------------

- indention in DebugFinder changes still has lines with 0 spaces, 2 spaces, tabs, 4 spaces. should all be 4 spaces
- also please note that our style guides says for/foreach/if/swtich-case have their opening curly braces on the same line as the keyword (foreach(...) {)
- numerous lines with indention but nothing after the indention

> m_aptTxn = 0;

better, but not perfect (http://finance.yahoo.com/q?s=TXN). please call it m_transaction or m_aptTransaction :P

> QString(DDEB_REPO_FILE)

constructing using QString::fromLatin1(..) is still faster.

new:
> QApt::SourceEntryList hardCodedRepoList;

if it is hardcoded, then it probably should be static and only be built/filled once.

> #include <KProgressDialog>

in DebugFinder.h this can now be a forward decl

> connect(m_progress, SIGNAL(cancelClicked()), m_aptTxn, SLOT(cancel()));

I personally like to line break after the signal to make it easier to spot which objects are being wired here...

connect(m_progress, SIGNAL(cancelClicked()),
        m_aptTxn, SLOT(cancel()));

------------------ smoke ------------------

(see previous smoke results
- Authentication still can not be cancelled
- ddebs entries are still added ad infinitum
- debug output still too excessive with percent values being debugged :P
- ddebs.list is still not removed at exit() if it was initially created by debug-installer

new:
- Crash when cancelling adding of sources and package lookup (http://paste.kde.org/p256c29ae/)

------------------ clang-analyzer ------------------

still happy xD

------------------ krazy2 ------------------

still issues:

krazy2 Analysis

Checkers Run = 39
Files Processed = 13
Total Issues = 9 ...as of July 30 2013 13:25:14 CEST

== messages: For File Type messages ==
1. Check for appending to rc.cpp [rcappend]... Ok!

== cmake: For File Type cmake ==
1. Check for an acceptable copyright [copyright]... Ok!

2. Check that file ends with a newline [endswithnewline]... Ok!

3. Check for an acceptable license [license]... Ok!

4. Check for spelling errors [spelling]... Ok!

== c++: For File Type c++ ==
1. Check for TRUE and FALSE macros [captruefalse]... Ok!

2. Check for methods that return 'const' refs in public classes [constref]... Ok!

3. Check for an acceptable copyright [copyright]... Ok!

4. Check for cpp macros and usage [cpp]... Ok!

5. Check for code that should be considered crashy. [crashy]... Ok!

6. Check single-char QString operations for efficiency [doublequote_chars]... Ok!

7. Check for unwanted doxygen tags in major versions [doxytags]... Ok!

8. Check public classes with private members or d-pointer issues [dpointer]... Ok!

9. Check for QString compares to "" [emptystrcompare]... Ok!

10. Check that file ends with a newline [endswithnewline]... 2 issues found
        ./src/QAptDecorator.cpp: line# 80 (1)
        ./src/LSBRelease.cpp: line# 66 (1)
        Files that do not end with a newline character can cause problems.
        Please add a newline character to the end of the file.

11. Check for ...

review: Needs Fixing

[Michael D. Stemle, Jr. (manchicken)]

In trying to get the entries removed, it looks like libqapt doesn't fully remove entries if you put them into a sources.list file... at least not the way I'm doing it. My next submission will include code to remove those sources lines, let me know if - upon testing - the file still doesn't go away. Right now I'm seeing that libqapt is expanding the sources list directory as ".", which I think may be more related to how my environment is set up.

Unmerged revisions

108. By Michael D. Stemle, Jr. <email address hidden> on 2013-08-04

Made a number of changes per Harald's list: https://code.launchpad.net/~manchicken/kubuntu-debug-installer/kubuntu-debug-installer/+merge/177273/comments/399863

107. By Michael D. Stemle, Jr. <email address hidden> on 2013-07-27

Fixed some code indentation.

106. By Michael D. Stemle, Jr. <email address hidden> on 2013-07-27

Fixed some text formatting.

105. By Michael D. Stemle, Jr. <email address hidden> on 2013-07-27

Made a series of changes requested by Harold.

104. By Michael D. Stemle, Jr. <email address hidden> on 2013-07-22

Made change per JT's recommendation.

103. By Michael D. Stemle, Jr. <email address hidden> on 2013-07-17

Added the base repo.

102. By Michael D. Stemle, Jr. <email address hidden> on 2013-07-16

Fixed a small bug when debug packages weren't found.

101. By Michael D. Stemle, Jr. <email address hidden> on 2013-07-16

Added the changes for sources.list and refreshing cache: https://wiki.ubuntu.com/DebuggingProgramCrash#Debug_Symbol_Packages

100. By Michael D. Stemle, Jr. <email address hidden> on 2013-07-06

I now have the sources add working.

99. By Michael D. Stemle, Jr. <email address hidden> on 2013-07-05

Adding LSB stuff from Herald Sitter (THANKS!) and adding some code to set up repos.

=== modified file 'src/CMakeLists.txt'
--- src/CMakeLists.txt	2010-09-16 14:19:01 +0000
+++ src/CMakeLists.txt	2013-08-04 08:05:33 +0000
@@ -2,6 +2,8 @@
 include_directories(${CMAKE_CURRENT_BINARY_DIR})
 
 set(installdbgsymbols_SRCS
+    QAptDecorator.cpp
+    LSBRelease.cpp
     DebugFinder.cpp
     DebugInstaller.cpp
     main.cpp

=== modified file 'src/DebugFinder.cpp'
--- src/DebugFinder.cpp	2012-10-30 15:16:47 +0000
+++ src/DebugFinder.cpp	2013-08-04 08:05:33 +0000
@@ -2,6 +2,7 @@
   Copyright © 2010 Harald Sitter <[e-mail]>
   Copyright © 2010 Felix Geyer <[e-mail]>
   Copyright © 2010 Jonathan Thomas <[e-mail]>
+  Copyright © 2013 Michael D. Stemle, Jr. <[e-mail]>
 
   This program is free software; you can redistribute it and/or
   modify it under the terms of the GNU General Public License as
@@ -21,17 +22,45 @@
 */
 
 #include "DebugFinder.h"
-
-#include <QtCore/QStringList>
-#include <QtCore/QThread>
+#include "LSBRelease.h"
+#include "QAptDecorator.h"
 
 #include <LibQApt/Backend>
+#include <LibQApt/SourceEntry>
+#include <LibQApt/SourcesList>
+#include <LibQApt/Transaction>
+#include <iostream>
+
+#include <KDebug>
+#include <KLocalizedString>
+#include <KProgressDialog>
+#include <KMessageBox>
+#include <qfile.h>
 
 DebugFinder::DebugFinder(QObject *parent) :
     QObject(parent),
     m_backend(new QApt::Backend(this)),
-    m_stop(false)
+    m_progress(new KProgressDialog()),
+    m_ddebsSources(new QApt::SourceEntryList()),
+    m_sourcesList(new QApt::SourcesList(this)),
+    m_stop(false), 
+    m_sourcesChangedSinceChecked(false)
 {
+    kDebug() << "INIT!!!!!\n";
+    m_aptTransaction = 0;
+    m_progress->progressBar()->setMaximum(100);
+    QString repoFile(QString::fromLatin1(DDEB_REPO_FILE));
+
+    // Construct our list hard-coded sources...
+    if (m_ddebsSources->count() == 0) {
+        LSBRelease lsb;
+        QString codename = lsb.codename();
+        m_ddebsSources->append(QApt::SourceEntry(QString::fromLatin1(REPO_ARG_BASE).arg(codename), repoFile));
+        m_ddebsSources->append(QApt::SourceEntry(QString::fromLatin1(REPO_ARG_UPDATES).arg(codename), repoFile));
+        m_ddebsSources->append(QApt::SourceEntry(QString::fromLatin1(REPO_ARG_SECURITY).arg(codename), repoFile));
+        m_ddebsSources->append(QApt::SourceEntry(QString::fromLatin1(REPO_ARG_PROPOSED).arg(codename), repoFile));
+    }
+    
     m_backend->init();
 }
 
@@ -69,17 +98,18 @@
     return 0;
 }
 
-void DebugFinder::find(const QString &file)
+void DebugFinder::find()
 {
     if (m_stop) {
         return;
     }
 
-    QApt::Package *package = m_backend->packageForFile(file);
+    QApt::Package *package = m_backend->packageForFile(m_file);
 
     QApt::Package *dbgPkg = getDbgPkg(package);
     if (!dbgPkg) {
-        emit foundNoDbgPkg(file);
+        kDebug() << "No debug package found for: " << m_file;
+        emit foundNoDbgPkg(m_file);
     } else if (dbgPkg->isInstalled()) {
         emit alreadyInstalled();
     } else {
@@ -90,4 +120,178 @@
 void DebugFinder::stop()
 {
     m_stop = true;
+    removeDdebs();
+}
+
+void DebugFinder::startFinding(const QString &file)
+{
+    m_progress->setLabelText(i18nc("@title", "Adding debug symbol sources"));
+    m_file = file;
+    connect(this, SIGNAL(aptCacheUpdateFinished()),
+            this, SLOT(onAptCacheUpdateFinished()));
+
+    if (hasDdebs()) {
+        // We already have debs...
+        kDebug() <<  "SKIPPING DDEBS!";
+        emit aptCacheUpdateFinished();
+    } else {
+        installDdebs();
+    }
+
+    kDebug() << "Done with startFinding()!";
+}
+
+// The update is finished!
+void DebugFinder::onAptCacheUpdateFinished()
+{
+    kDebug() << "Done with updateCache(), reloading cache...";
+    m_backend->reloadCache();
+
+    kDebug() << "I'm done.";
+    m_progress->progressBar()->setValue(100);
+
+    find();
+
+    return;
+}
+
+void DebugFinder::onTxnStatusChanged(QApt::TransactionStatus status)
+{
+    kDebug() << "cache widget: transaction status changed" << status;
+
+    if (!m_aptTransaction) {
+        kDebug() << "No valid transaction.";
+        return;
+    }
+
+    switch (status) {
+    case QApt::DownloadingStatus:
+        if (m_aptTransaction->role() == QApt::UpdateCacheRole) {
+            m_progress->setLabelText(i18nc("@title", "Updating software sources"));
+        } else {
+            m_progress->setLabelText(i18nc("@title", "Downloading Packages"));
+        }
+
+        break;
+
+    case QApt::FinishedStatus:
+        m_progress->setLabelText(i18nc("@title", "Finished."));
+        break;
+
+    default:
+        break;
+    }
+}
+
+void DebugFinder::onAptError(QApt::ErrorCode errcode)
+{
+    QString errmsg = QAptDecorator::errorText(errcode, m_aptTransaction);
+    kDebug() << "Got an error: " << errmsg;
+
+    KMessageBox::error(
+        NULL,
+        i18nc("@info Error message", "Error updating sources"
+              " after configuring debug sources"),
+        i18nc("@title Error dialog", "Failed updating sources")
+    );
+}
+
+void DebugFinder::onProgressChanged(int progress)
+{
+    int presentProgress = progress;
+
+    if (progress >= 100) {
+        presentProgress = 0;
+    }
+    m_progress->progressBar()->setValue(presentProgress);
+}
+
+bool DebugFinder::hasDdebs()
+{
+    static int ddeb_count = 0;
+
+    if (!m_sourcesChangedSinceChecked && (ddeb_count == m_ddebsSources->count())) {
+        kDebug() <<  "m_sourcesChangedSinceChecked == " <<  m_sourcesChangedSinceChecked << " and m_ddebsSources->count() == " <<  m_ddebsSources->count() <<  " and ddeb_count == " <<  ddeb_count;
+        return true;
+    }
+
+    const QApt::SourceEntryList &existingEntries = m_sourcesList->entries();
+
+    foreach (const QApt::SourceEntry &repo, *m_ddebsSources) {
+        if (existingEntries.contains(repo)) {
+            kDebug() <<  "LOOP,  CONTAINS!";
+            ddeb_count += 1;
+        }
+        else { kDebug() <<  "LOOP,  DOESNT CONTAIN!"; }
+    }
+    kDebug() <<  "(2) m_sourcesChangedSinceChecked == " <<  m_sourcesChangedSinceChecked << " and m_ddebsSources->count() == " <<  m_ddebsSources->count() <<  " and ddeb_count == " <<  ddeb_count;
+
+    m_sourcesChangedSinceChecked = false;
+
+    return (ddeb_count == m_ddebsSources->count());
+}
+
+void DebugFinder::updateAptCache()
+{
+    m_progress->progressBar()->setValue(0);
+    m_aptTransaction = m_backend->updateCache();
+
+    m_aptTransaction->setLocale(QLatin1String(setlocale(LC_MESSAGES, 0)));
+
+    connect(m_progress, SIGNAL(cancelClicked()),
+            m_aptTransaction, SLOT(cancel()));
+    connect(m_aptTransaction, SIGNAL(errorOccurred(QApt::ErrorCode)),
+            this, SLOT(onAptError(QApt::ErrorCode)));
+    connect(m_aptTransaction, SIGNAL(statusChanged(QApt::TransactionStatus)),
+            this, SLOT(onTxnStatusChanged(QApt::TransactionStatus)));
+    connect(m_aptTransaction, SIGNAL(progressChanged(int)),
+            this, SLOT(onProgressChanged(int)));
+    connect(m_aptTransaction, SIGNAL(finished(QApt::ExitStatus)),
+            this, SLOT(onAptCacheUpdateFinished()));
+
+    m_aptTransaction->run();
+    kDebug() << "FinishedStatus: " << QApt::FinishedStatus;
+}
+
+void DebugFinder::installDdebs()
+{
+    // If it's already there, don't do it!
+    if (hasDdebs()) {
+        return;
+    }
+
+    foreach (const QApt::SourceEntry &repo, *m_ddebsSources) {
+        m_sourcesList->addEntry(repo); // This method has a dup check
+    }
+
+    m_sourcesList->save();
+    m_sourcesChangedSinceChecked = true;
+
+    kDebug() << "Entries saved, updating backend cache...";
+    updateAptCache();
+    
+    return;
+}
+
+void DebugFinder::removeDdebs()
+{
+    m_progress->setLabelText(i18nc("@title", "Cleaning up sources"));
+    // If it's NOT already there, don't do it!
+    if (!hasDdebs()) {
+        return;
+    }
+    kDebug() << "Has DDEBS! Cleaning!";
+
+    foreach (const QApt::SourceEntry &repo, *m_ddebsSources) {
+        kDebug() << "Removing " << repo.toString();
+        m_sourcesList->removeEntry(repo);
+    }
+
+    m_sourcesList->save();
+    m_sourcesChangedSinceChecked = true;
+
+    kDebug() << "Entries saved, updating backend cache...";
+    updateAptCache();
+    
+    return;
 }

=== modified file 'src/DebugFinder.h'
--- src/DebugFinder.h	2012-10-30 15:16:47 +0000
+++ src/DebugFinder.h	2013-08-04 08:05:33 +0000
@@ -1,6 +1,7 @@
 /*
   Copyright © 2010 Harald Sitter <[e-mail]>
   Copyright © 2010 Jonathan Thomas <[e-mail]>
+  Copyright © 2013 Michael D. Stemle, Jr. <[e-mail]>
 
   This program is free software; you can redistribute it and/or
   modify it under the terms of the GNU General Public License as
@@ -22,13 +23,26 @@
 #ifndef DEBUGFINDER_H
 #define DEBUGFINDER_H
 
+#include <LibQApt/Globals>
+
 #include <QtCore/QObject>
 #include <QtCore/QStringList>
 
+#define REPO_ARG_BASE       "deb http://ddebs.ubuntu.com %1 main restricted universe multiverse"
+#define REPO_ARG_UPDATES    "deb http://ddebs.ubuntu.com %1-updates main restricted universe multiverse"
+#define REPO_ARG_SECURITY   "deb http://ddebs.ubuntu.com %1-security main restricted universe multiverse"
+#define REPO_ARG_PROPOSED   "deb http://ddebs.ubuntu.com %1-proposed main restricted universe multiverse"
+#define DDEB_REPO_FILE      "/etc/apt/sources.list.d/ddebs.list"
+
+class KProgressDialog;
+
 namespace QApt {
     class Backend;
     class Package;
-}
+    class SourceEntry;
+    class SourcesList;
+    class Transaction;
+};
 
 class DebugFinder : public QObject
 {
@@ -36,20 +50,38 @@
 public:
     explicit DebugFinder(QObject *parent = 0);
 
+    void performUpdate();
+
 public slots:
-    void find(const QString &file);
+    void startFinding(const QString &file);
+    void find();
     void stop();
-
+    void onTxnStatusChanged(QApt::TransactionStatus status);
+    void onAptError(QApt::ErrorCode errcode);
+    void onAptCacheUpdateFinished();
+    void onProgressChanged(int progress);
+    bool hasDdebs();
+    void installDdebs();
+    void removeDdebs();
+    void updateAptCache();
+    
 signals:
     void foundDbgPkg(const QString &dbgpkg);
-    void foundNoDbgPkg(const QString &file);
+    void foundNoDbgPkg(const QString &dbgpkg);
     void alreadyInstalled();
+    void aptCacheUpdateFinished();
 
 private:
     QApt::Package *getDbgPkg(QApt::Package *package);
+    
     QApt::Backend *m_backend;
-
+    QApt::Transaction *m_aptTransaction;
+    KProgressDialog *m_progress;
+    QList<QApt::SourceEntry> *m_ddebsSources;
+    QApt::SourcesList *m_sourcesList;
+    QString m_file;
     bool m_stop;
+    bool m_sourcesChangedSinceChecked;
 };
 
 #endif // DEBUGFINDER_H

=== modified file 'src/DebugInstaller.cpp'
--- src/DebugInstaller.cpp	2011-09-26 13:35:14 +0000
+++ src/DebugInstaller.cpp	2013-08-04 08:05:33 +0000
@@ -1,6 +1,7 @@
 /*
   Copyright © 2010 Harald Sitter <[e-mail]>
   Copyright © 2010 Jonathan Thomas <[e-mail]>
+  Copyright © 2013 Michael D. Stemle, Jr. <[e-mail]>
 
   This program is free software; you can redistribute it and/or
   modify it under the terms of the GNU General Public License as
@@ -68,6 +69,8 @@
     if (install.exitCode() != 0) {
         EXIT(ERR_RANDOM_ERR);
     }
+    
+    m_finder->removeDdebs();
 
     close();
 }
@@ -163,17 +166,20 @@
     progressBar()->setMaximum(m_args.count());
     incrementProgress();
 
-    m_finder = new DebugFinder;
-    connect(m_finder, SIGNAL(foundDbgPkg(QString)), this, SLOT(foundDbgPkg(QString)));
-    connect(m_finder, SIGNAL(foundNoDbgPkg(QString)), this, SLOT(foundNoDbgPkg(QString)));
-    connect(m_finder, SIGNAL(alreadyInstalled()), this, SLOT(alreadyInstalled()));
+    m_finder = new DebugFinder(this);
+    connect(m_finder, SIGNAL(foundDbgPkg(QString)),
+            this, SLOT(foundDbgPkg(QString)));
+    connect(m_finder, SIGNAL(foundNoDbgPkg(QString)),
+            this, SLOT(foundNoDbgPkg(QString)));
+    connect(m_finder, SIGNAL(alreadyInstalled()),
+            this, SLOT(alreadyInstalled()));
 
     m_finderThread = new QThread(this);
     m_finder->moveToThread(m_finderThread);
     m_finderThread->start();
 
     foreach (const QString &file, m_args) {
-        QMetaObject::invokeMethod(m_finder, "find", Qt::QueuedConnection,
+        QMetaObject::invokeMethod(m_finder, "startFinding", Qt::QueuedConnection,
                                   Q_ARG(QString, file));
     }
 }

=== added file 'src/LSBRelease.cpp'
--- src/LSBRelease.cpp	1970-01-01 00:00:00 +0000
+++ src/LSBRelease.cpp	2013-08-04 08:05:33 +0000
@@ -0,0 +1,66 @@
+/*
+  Copyright (C) 2012 Harald Sitter <[e-mail]>
+ 
+  This program is free software; you can redistribute it and/or
+  modify it under the terms of the GNU General Public License as
+  published by the Free Software Foundation; either version 2 of
+  the License or (at your option) version 3 or any later version
+  accepted by the membership of KDE e.V. (or its successor approved
+  by the membership of KDE e.V.), which shall act as a proxy
+  defined in Section 14 of version 3 of the license.
+ 
+  This program is distributed in the hope that it will be useful,
+  but WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+  GNU General Public License for more details.
+ 
+  You should have received a copy of the GNU General Public License
+  along with this program.  If not, see <http://www.gnu.org/licenses/>.
+*/
+ 
+#include "LSBRelease.h"
+ 
+#include <QtCore/QFile>
+ 
+#include <KDebug>
+#include <KLocalizedString>
+ 
+LSBRelease::LSBRelease()
+{
+    QFile file("/etc/lsb-release");
+    if (!file.open(QIODevice::ReadOnly | QIODevice::Text)) {
+        kDebug() << "Could not open /etc/lsb-release, aborting LSB";
+        QString unknown = i18nc("@label", "Unknown Release");
+        m_id = unknown;
+        m_release = unknown;
+        m_codename = unknown;
+        m_description = unknown;
+        return;
+    }
+    
+    // List of valid column identifiers
+    QString line;
+    QStringList comps;
+    while (!file.atEnd()) {
+        line = file.readLine();
+        comps = line.split(QChar('='));
+        Q_ASSERT(comps.size() == 2);
+        QString key = comps.at(0);
+        QString value = comps.at(1).trimmed();
+        if (key == QLatin1String("DISTRIB_ID"))
+            m_id = value;
+        else if (key == QLatin1String("DISTRIB_RELEASE"))
+            m_release = value;
+        else if (key == QLatin1String("DISTRIB_CODENAME"))
+            m_codename = value;
+        else if (key == QLatin1String("DISTRIB_DESCRIPTION"))
+            m_description = value;
+        else
+            Q_ASSERT(false); // Should not happen!
+    }
+    
+    if (m_id.isEmpty() || m_release.isEmpty()) {
+        Q_ASSERT(false); // meh.
+    }
+}
+

=== added file 'src/LSBRelease.h'
--- src/LSBRelease.h	1970-01-01 00:00:00 +0000
+++ src/LSBRelease.h	2013-08-04 08:05:33 +0000
@@ -0,0 +1,43 @@
+/*
+  Copyright (C) 2012 Harald Sitter <[e-mail]>
+
+  This program is free software; you can redistribute it and/or
+  modify it under the terms of the GNU General Public License as
+  published by the Free Software Foundation; either version 2 of
+  the License or (at your option) version 3 or any later version
+  accepted by the membership of KDE e.V. (or its successor approved
+  by the membership of KDE e.V.), which shall act as a proxy
+  defined in Section 14 of version 3 of the license.
+
+  This program is distributed in the hope that it will be useful,
+  but WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+  GNU General Public License for more details.
+
+  You should have received a copy of the GNU General Public License
+  along with this program.  If not, see <http://www.gnu.org/licenses/>.
+*/
+
+#ifndef LSBRELEASE_H
+#define LSBRELEASE_H
+
+#include <QtCore/QString>
+
+class LSBRelease
+{
+public:
+    LSBRelease();
+
+    QString id() { return m_id; }
+    QString release() { return m_release; }
+    QString codename() { return m_codename; }
+    QString description() { return m_description; }
+
+private:
+    QString m_id;
+    QString m_release;
+    QString m_codename;
+    QString m_description;
+};
+
+#endif // LSBRELEASE_H

=== added file 'src/QAptDecorator.cpp'
--- src/QAptDecorator.cpp	1970-01-01 00:00:00 +0000
+++ src/QAptDecorator.cpp	2013-08-04 08:05:33 +0000
@@ -0,0 +1,81 @@
+/*
+  Copyright © 2010 Jonathan Thomas <[e-mail]> (cacheupdate example in QApt)
+  Copyright © 2013 Michael D. Stemle, Jr. <[e-mail]>
+
+  This program is free software; you can redistribute it and/or
+  modify it under the terms of the GNU General Public License as
+  published by the Free Software Foundation; either version 2 of
+  the License or (at your option) version 3 or any later version
+  accepted by the membership of KDE e.V. (or its successor approved
+  by the membership of KDE e.V.), which shall act as a proxy
+  defined in Section 14 of version 3 of the license.
+
+  This program is distributed in the hope that it will be useful,
+  but WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+  GNU General Public License for more details.
+
+  You should have received a copy of the GNU General Public License
+  along with this program.  If not, see <http://www.gnu.org/licenses/>.
+*/
+
+#include "QAptDecorator.h"
+
+#include <QtCore/QString>
+
+#include <LibQApt/Globals>
+#include <KLocalizedString>
+
+QString QAptDecorator::errorText(QApt::ErrorCode error, QApt::Transaction *trans)
+{
+    QString text;
+
+    switch (error) {
+    case QApt::InitError:
+        text = i18nc("@label", "The package system could not be initialized, your "
+                     "configuration may be broken.");
+        break;
+    case QApt::LockError:
+        text = i18nc("@label",
+                     "Another application seems to be using the package "
+                     "system at this time. You must close all other package "
+                     "managers before you will be able to install or remove "
+                     "any packages.");
+        break;
+    case QApt::DiskSpaceError:
+        text = i18nc("@label",
+                     "You do not have enough disk space in the directory "
+                     "at %1 to continue with this operation.", trans->errorDetails());
+        break;
+    case QApt::FetchError:
+        text = i18nc("@label", "Could not download packages");
+        break;
+    case QApt::CommitError:
+        text = i18nc("@label", "An error occurred while applying changes:");
+        break;
+    case QApt::AuthError:
+        text = i18nc("@label",
+                     "This operation cannot continue since proper "
+                     "authorization was not provided");
+        break;
+    case QApt::WorkerDisappeared:
+        text = i18nc("@label", "It appears that the QApt worker has either crashed "
+                     "or disappeared. Please report a bug to the QApt maintainers");
+        break;
+    case QApt::UntrustedError:
+        text = i18ncp("@label",
+                      "The following package has not been verified by its author. "
+                      "Downloading untrusted packages has been disallowed "
+                      "by your current configuration.",
+                      "The following packages have not been verified by "
+                      "their authors. "
+                      "Downloading untrusted packages has "
+                      "been disallowed by your current configuration.",
+                      trans->untrustedPackages().size());
+        break;
+    default:
+        break;
+    }
+
+    return text;
+}

=== added file 'src/QAptDecorator.h'
--- src/QAptDecorator.h	1970-01-01 00:00:00 +0000
+++ src/QAptDecorator.h	2013-08-04 08:05:33 +0000
@@ -0,0 +1,34 @@
+/*
+  Copyright © 2010 Jonathan Thomas <[e-mail]> (cacheupdate example in QApt)
+  Copyright © 2013 Michael D. Stemle, Jr. <[e-mail]>
+
+  This program is free software; you can redistribute it and/or
+  modify it under the terms of the GNU General Public License as
+  published by the Free Software Foundation; either version 2 of
+  the License or (at your option) version 3 or any later version
+  accepted by the membership of KDE e.V. (or its successor approved
+  by the membership of KDE e.V.), which shall act as a proxy
+  defined in Section 14 of version 3 of the license.
+
+  This program is distributed in the hope that it will be useful,
+  but WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+  GNU General Public License for more details.
+
+  You should have received a copy of the GNU General Public License
+  along with this program.  If not, see <http://www.gnu.org/licenses/>.
+*/
+
+#ifndef QAPTDECORATOR_H
+#define QAPTDECORATOR_H
+
+#include <LibQApt/Globals>
+#include <LibQApt/Transaction>
+
+class QAptDecorator
+{
+public:
+    static QString errorText(QApt::ErrorCode error, QApt::Transaction *trans);
+};
+
+#endif // QAPTDECORATOR_H