On Thu, Mar 2, 2017 at 10:08 AM, Dominik Haumann wrote:
> Hi,
Hi all,
>
> On Wed, Mar 1, 2017 at 4:16 PM, David Faure wrote:
>> On mercredi 1 mars 2017 13:35:47 CET Kevin Funk wrote:
>>> On Tuesday, 28 February 2017 21:31:44 CET Michael Pyne wrote:
>>> > On Tue, Feb 28, 2017 at 08:38:12PM +0100
mart accepted this revision.This revision is now accepted and ready to land.
View RevisionREPOSITORYR242 Plasma Framework (Library)BRANCHmasterREVISION DETAILhttps://phabricator.kde.org/D4851EMAIL PREFERENCEShttps://phabricator.kde.org/settings/panel/emailpreferences/To: davidedmundson, Plasma, mar
mart accepted this revision.This revision is now accepted and ready to land.
View RevisionREPOSITORYR242 Plasma Framework (Library)REVISION DETAILhttps://phabricator.kde.org/D4848EMAIL PREFERENCEShttps://phabricator.kde.org/settings/panel/emailpreferences/To: davidedmundson, Plasma, martCc: mart, b
dfaure accepted this revision.This revision is now accepted and ready to land.
View RevisionREPOSITORYR241 KIOBRANCHuse-kdoctools-sharedREVISION DETAILhttps://phabricator.kde.org/D4850EMAIL PREFERENCEShttps://phabricator.kde.org/settings/panel/emailpreferences/To: ltoscano, dfaureCc: Frameworks
dfaure added inline comments.
View RevisionINLINE COMMENTSView Inlinedocbookxslt.h:1
#ifndef _KDOCTOOLS_XSLT_H_
#define _KDOCTOOLS_XSLT_H_
copyright header missingView Inlinedocbookxslt.h:13
KDOCTOOLS_EXPORT QString transform(const QString &file, const QString &stylesheet,
vpilo added a comment.
View Revision
In D4799#91246, @davidedmundson wrote:
There is another solution that would work without any changes to KNotification.
DBus has a solution to buffer messages and wait for a name to become available, it happens in DBus activation. If plasmashell was DBus activat
ltoscano added inline comments.
View RevisionINLINE COMMENTSView Inlinedfaure wrote in docbookxslt.h:1
copyright header missing
For this and all the other notes about missing copyright: I did not write the original code, just reshuffled it in this header (ok, apart one function), so I guess I shoul
martinkostolny marked 2 inline comments as done.
View RevisionREPOSITORYR39 KTextEditorREVISION DETAILhttps://phabricator.kde.org/D4847EMAIL PREFERENCEShttps://phabricator.kde.org/settings/panel/emailpreferences/To: martinkostolny, dhaumann, KTextEditorCc: dhaumann, graesslin, davidedmundson, palan
dfaure added inline comments.
View RevisionINLINE COMMENTSView Inlineltoscano wrote in docbookxslt.h:1
For this and all the other notes about missing copyright: I did not write the original code, just reshuffled it in this header (ok, apart one function), so I guess I should find the copyright of p
martinkostolny updated this revision to Diff 12054.martinkostolny added a comment.
View RevisionThanks for all the feedback!
I'm updating the diff to reflect most Dominik's points. I'll learn from them next time :). Anyway I was unable to remove the KAuth namespace. It doesn't work without it, thi
broulik created this revision.Restricted Application added projects: Plasma, Frameworks.Restricted Application added subscribers: Frameworks, plasma-devel.
View RevisionREVISION SUMMARYIt is deprecated. Instead re-implement roleNames().
Done in places where only static roles were used.TEST PLANStil
martinkostolny updated this revision to Diff 12058.martinkostolny added a comment.
View RevisionNext iteration, still without autotests, I need to study them more.
This diff includes preserving owner and group when file is saved with elevated privileges. Actually loosing owner was (and is) already
ltoscano added a comment.
View Revision
In D4847#91810, @martinkostolny wrote:
This diff includes preserving owner and group when file is saved with elevated privileges. Actually loosing owner was (and is) already happening before my patch when user has the permission to write the file and is not i
martinkostolny added a comment.
View RevisionI think that leaving it as it is is not a solution
In that case, I favour the second option (direct write to file).REPOSITORYR39 KTextEditorREVISION DETAILhttps://phabricator.kde.org/D4847EMAIL PREFERENCEShttps://phabricator.kde.org/settings/panel/email
hein accepted this revision.This revision is now accepted and ready to land.
View RevisionREPOSITORYR242 Plasma Framework (Library)REVISION DETAILhttps://phabricator.kde.org/D4867EMAIL PREFERENCEShttps://phabricator.kde.org/settings/panel/emailpreferences/To: subdiff, Plasma, heinCc: plasma-devel,
mart accepted this revision.This revision is now accepted and ready to land.
View RevisionREPOSITORYR242 Plasma Framework (Library)REVISION DETAILhttps://phabricator.kde.org/D4891EMAIL PREFERENCEShttps://phabricator.kde.org/settings/panel/emailpreferences/To: broulik, Plasma, martCc: plasma-devel,
martinkostolny updated this revision to Diff 12084.martinkostolny added a comment.
View RevisionAfter reading though bug mentioned by Luigi and this QT bug (https://bugreports.qt.io/browse/QTBUG-56366) I'm not so convinced that we want to directly write to file in order to maintain owner, because w
This revision was automatically updated to reflect the committed changes.Closed by commit R242:daee27f0962d: [Menu] Improve available space correction for openRelative (authored by subdiff).
View RevisionCHANGED PRIOR TO COMMIThttps://phabricator.kde.org/D4867?vs=12015&id=12094#tocREPOSITORYR242 Pl
On čtvrtek 2. března 2017 9:08:25 CET, Ben Cooksley wrote:
In regards to the way email subjects are treated, I've now customised
the metamta.differential.subject-prefix preference on our installation
to be blank, to the "[Differential]" part should no longer be present.
Roughly at the time you
ltoscano updated this revision to Diff 12097.ltoscano added a comment.
View RevisionAdd descriptions and copyrights
Copyrights from the people who touched the signature from this
file and its ancestors.REPOSITORYR238 KDocToolsCHANGES SINCE LAST UPDATEhttps://phabricator.kde.org/D4849?vs=11982&id=1
ltoscano added a comment.
View RevisionEyes bleeding after digging into the history back to the cvs2svn conversion. At least I will be able to finally update the copyright of the other files (even if apparently originally kdoctools was BSD-like, need to investigate more).
Anyway, if this is accepte
ltoscano marked 3 inline comments as done.ltoscano added inline comments.
View RevisionINLINE COMMENTSView Inlinedfaure wrote in docbookxslt_p.h:1
missing copyright header
This is not exported, can I fix it later with the other files?REPOSITORYR238 KDocToolsREVISION DETAILhttps://phabricator.kde.or
apol created this revision.Restricted Application added a project: Frameworks.
View RevisionREVISION SUMMARYIt depends on kactivities, which is also tier3REPOSITORYR159 KActivities StatisticsBRANCHmasterREVISION DETAILhttps://phabricator.kde.org/D4906AFFECTED FILESmetainfo.yamlEMAIL PREFERENCEShttp
ivan added a comment.
View RevisionKActivities is tier 1 even if the metainfo.yaml says otherwise. It only requires boost.
So, this should be changed in KActivities, not in KActivitiesStatsREPOSITORYR159 KActivities StatisticsREVISION DETAILhttps://phabricator.kde.org/D4906EMAIL PREFERENCEShttps:/
mgallien created this revision.Restricted Application added a project: Frameworks.Restricted Application added a subscriber: Frameworks.
View RevisionREVISION SUMMARYAdd new DBus signals sent by Baloo indexer for removed or moved files.
I still need to extend existing automatic tests with checks th
mgallien added a reviewer: kde-frameworks-devel.
View RevisionREPOSITORYR293 BalooREVISION DETAILhttps://phabricator.kde.org/D4911EMAIL PREFERENCEShttps://phabricator.kde.org/settings/panel/emailpreferences/To: mgallien, kde-frameworks-develCc: Frameworks
On Fri, Mar 3, 2017 at 10:41 AM, Jan Kundrát wrote:
> On čtvrtek 2. března 2017 9:08:25 CET, Ben Cooksley wrote:
>>
>> In regards to the way email subjects are treated, I've now customised
>> the metamta.differential.subject-prefix preference on our installation
>> to be blank, to the "[Differenti
27 matches
Mail list logo