aacid added inline comments.

INLINE COMMENTS

> kpackagetool.cpp:534
> +        writer.writeStartElement(QStringLiteral("url"));
> +        writer.writeAttribute(QStringLiteral("type"), 
> QStringLiteral("donation"));
> +        
> writer.writeCharacters(QStringLiteral("https://www.kde.org/donate.php";));

Not sure if i'm saying crap, but should we check if it already has a donation 
url and then give a warning or something else?

REPOSITORY
  R290 KPackage

REVISION DETAIL
  https://phabricator.kde.org/D11080

To: apol, #frameworks
Cc: aacid, michaelh

Reply via email to