bled for a long time and
have not observed any problems. But, for the moment, please report if
the module being disabled causes any problems for you or with any other
applications. If needed it can be reenabled on the "Background
Services" page in System Settings until KMix is next restarte
in a "files only" mode, in the same way as "directories
only" is currently supported? If it turns out that KDirOperator::setMode()
is intended to be have that effect, then I'd be happy to investigate why
it doesn't.
Best regards,
--
Jonathan Marten http://
ork/marten/modernise' branch. If you use Kuickshow and can
compile from source, pull this branch and try it out. Feedback welcome!
Best regards,
--
Jonathan Marten http://www.keelhaul.me.uk
Twickenham, UK j...@keelhaul.me.uk
all
imlib), or even eliminating both X11 and imlib and using the
corresponding Qt facilities instead (more work, but removes a
dependency). Hopefully Kuickshow will live on!
--
Jonathan Marten http://www.keelhaul.me.uk
Twickenham, UK j...@keelhaul.me.uk
This revision was automatically updated to reflect the committed changes.
Closed by commit R241:6452a34cf01d: Fix service file specifying Run in
terminal giving an error code 100 (authored by marten).
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
marten created this revision.
marten added reviewers: Frameworks, dfaure.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
marten requested review of this revision.
REVISION SUMMARY
https://bugs.kde.org/show_bug.cgi?id=421374 describes how a service file
marten added a comment.
I've thought about this a bit more and can't find any explanation as to why
the DBus signal does not loop back to the sending process as well as all other
listeners. It may be an unspecified detail of the DBus implementation that
could possibly change at any time
marten added a comment.
I'm happy to work on the refactoring if you think it's the right thing to do.
Do you mean splitting SchedulerPrivate::slotReparseSlaveConfiguration() up
into two halves, the first part (KProtocolManager::reparseConfiguration through
to NetRC::self()->reload -
This revision was automatically updated to reflect the committed changes.
Closed by commit R266:0a5dd2972b62: Allow building from a read-only source
location (authored by marten).
REPOSITORY
R266 Breeze Icons
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D28755?vs=79867=80219
marten updated this revision to Diff 79867.
marten added a comment.
Yes, that would mean fewer changes to the validate_svg.sh script.
REPOSITORY
R266 Breeze Icons
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D28755?vs=79845=79867
REVISION DETAIL
marten created this revision.
marten added a reviewer: Breeze.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
marten requested review of this revision.
REVISION SUMMARY
In the situation, for example, where a master source tree may be shared among
a number
marten created this revision.
marten added reviewers: Frameworks, dfaure.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
marten requested review of this revision.
REVISION SUMMARY
I've been looking at porting Konqueror's User Agent Changer plugin to current
marten added a comment.
@ahmadsamir: Yes, the problem could be fixed in Dolphin, but that leaves a
potential trap for any other KPart that reimplements openUrl().
As a minimum, if it is decided that the fix belongs in the application then
KParts::ReadOnlyPart should have a caution in
marten added a comment.
Yes, I'd concluded that the real place to fix the problem was at the KParts
level, but not being a KParts expert wanted to leave that decision to its
maintainers. +1 for the elegant fix.
REPOSITORY
R306 KParts
REVISION DETAIL
https://phabricator.kde.org/D27148
marten abandoned this revision.
REPOSITORY
R320 KIO Extras
REVISION DETAIL
https://phabricator.kde.org/D7820
To: marten, #plasma, kfm-devel, mkoller
Cc: ltoscano, kde-frameworks-devel, plasma-devel, aprcela, fprice, LeGast00n,
sbergeron, jraleigh, fbampaloukas, alexde, GB_2, feverfew,
marten added a comment.
Confirmed that man:clang(1) now correctly displays the man page with no
spurious numbers shown. Would be happy to abandon this review request.
REPOSITORY
R320 KIO Extras
REVISION DETAIL
https://phabricator.kde.org/D7820
To: marten, #plasma, kfm-devel, mkoller
ed Extragear (now Self Released?).
I'll start to work through the review process checklist
(https://community.kde.org/ReleasingSoftware), and then try to submit
Kooka to kdereview - although that probably won't be able to be until
after the summer holidays.
Best regards, Jonathan
--
Jonathan Mart
marten abandoned this revision.
marten added a comment.
Appears to have been already implemented by
https://phabricator.kde.org/R244:b2b873d612a018d33866ddfd497f87bb7dd79573
REPOSITORY
R244 KCoreAddons
REVISION DETAIL
https://phabricator.kde.org/D19332
To: marten, #frameworks
Cc: apol,
This revision was automatically updated to reflect the committed changes.
Closed by commit R320:1b56cc62c93d: kio_smb: Change incorrect use of
QUrl::adjusted() (authored by marten).
REPOSITORY
R320 KIO Extras
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D19439?vs=52884=54927
marten added a comment.
Ping - anyone able to review please?
REPOSITORY
R320 KIO Extras
REVISION DETAIL
https://phabricator.kde.org/D19439
To: marten, #plasma, #frameworks
Cc: kde-frameworks-devel, kfm-devel, alexde, feverfew, meven, michaelh,
spoorun, navarromorales, firef, ngraham,
This revision was automatically updated to reflect the committed changes.
Closed by commit R241:534ebe58c4ca: Delete/Trash confirmation dialogue: Fix
misleading title (authored by marten).
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D19847?vs=54145=54178
marten created this revision.
marten added reviewers: Frameworks, dfaure.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
marten requested review of this revision.
REVISION SUMMARY
The misleading title that brought this dialogue to my attention is where
marten updated this revision to Diff 52898.
marten added a comment.
Use the same condition as the Qt header uses
REPOSITORY
R244 KCoreAddons
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D19332?vs=52601=52898
REVISION DETAIL
https://phabricator.kde.org/D19332
AFFECTED FILES
marten added a reviewer: kfm-devel.
Herald added projects: Dolphin, Frameworks.
Herald added a subscriber: kde-frameworks-devel.
REPOSITORY
R320 KIO Extras
REVISION DETAIL
https://phabricator.kde.org/D7820
To: marten, #plasma, kfm-devel
Cc: kde-frameworks-devel, plasma-devel, jraleigh,
marten added a reviewer: kfm-devel.
Herald added projects: Dolphin, Frameworks.
Herald added a subscriber: kde-frameworks-devel.
REPOSITORY
R320 KIO Extras
REVISION DETAIL
https://phabricator.kde.org/D9033
To: marten, #plasma, kfm-devel
Cc: kde-frameworks-devel, apol, plasma-devel,
marten created this revision.
marten added reviewers: Plasma, Frameworks.
Herald added projects: Dolphin, Frameworks.
Herald added subscribers: kfm-devel, kde-frameworks-devel.
marten requested review of this revision.
REVISION SUMMARY
Compiling this module produces a number of warnings all
This revision was automatically updated to reflect the committed changes.
Closed by commit R293:604a2f402da7: Check string length to avoid crash for
tags:/ URL (authored by marten).
CHANGED PRIOR TO COMMIT
https://phabricator.kde.org/D19371?vs=52681=52703#toc
REPOSITORY
R293 Baloo
CHANGES
marten created this revision.
marten added a reviewer: Baloo.
Herald added projects: Frameworks, Baloo.
Herald added a subscriber: kde-frameworks-devel.
marten requested review of this revision.
REVISION SUMMARY
Bug https://bugs.kde.org/show_bug.cgi?id=400594 was fixed by
marten added inline comments.
INLINE COMMENTS
> apol wrote in kprocess.h:332
> Also should check for QT_DISABLE_DEPRECATED_BEFORE no?
Do you mean conditionalise instead on #if !QT_DEPRECATED_SINCE(5, 13), so that
if the functions are not disabled in Qt then they are hidden here? That would
marten created this revision.
marten added a reviewer: Frameworks.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
marten requested review of this revision.
REVISION SUMMARY
The functions QProcess::setReadChannelMode() and QProcess::readChannelMode()
are
marten added a comment.
In https://phabricator.kde.org/D7823#164879, @cfeck wrote:
> Are the copies in the attic directory still needed for compatibility
reasons, or can they get removed?
No needed by anything within Frameworks, Plasma or Applications as far as I'm
aware. Shall
This revision was automatically updated to reflect the committed changes.
Closed by commit R240:ee5b036c1df4: Add FindGLIB2.cmake and
FindPulseAudio.cmake (authored by marten).
CHANGED PRIOR TO COMMIT
https://phabricator.kde.org/D7823?vs=19611=21961#toc
REPOSITORY
R240 Extra CMake Modules
marten added a comment.
Ping anyone?
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D7823
To: marten, #frameworks, #build_system
Cc: cgiboudeaux, cfeck, heikobecker
marten added a comment.
FYI: KMail uses Ctrl+comma and Ctrl+dot as standard shortcuts for "Collapse
all threads" and "Expand all threads" respectively.
REPOSITORY
R237 KConfig
REVISION DETAIL
https://phabricator.kde.org/D8296
To: ngraham, #frameworks, #vdg
Cc: marten, graesslin,
This revision was automatically updated to reflect the committed changes.
Closed by commit R241:c76c1486ec79: kioexec: Watch the file when it has
finished copying (authored by marten).
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7841?vs=19864=19885
REVISION
marten updated this revision to Diff 19864.
marten added a comment.
Updated in accordance with review comments. Go easy - it's the first lambda
that I've submitted...
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7841?vs=19565=19864
REVISION DETAIL
marten marked 2 inline comments as done.
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D7841
To: marten, #frameworks, dfaure
Cc: elvisangelaccio, dfaure
marten marked an inline comment as done.
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D7823
To: marten, #frameworks, #build_system
Cc: cgiboudeaux, cfeck, heikobecker
marten updated this revision to Diff 19611.
marten added a comment.
Diff updated to move library target definition after
find_package_handle_standard_args.
If these modules are accepted into ECM then KMix needs a further commit
anyway, in order to remove the final requirement for
marten marked an inline comment as done.
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D7823
To: marten, #frameworks, #build_system
Cc: cgiboudeaux, cfeck, heikobecker
marten updated this revision to Diff 19576.
marten marked an inline comment as done.
marten added a comment.
Updated as per review comments.
REPOSITORY
R240 Extra CMake Modules
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7823?vs=19570=19576
REVISION DETAIL
marten marked 17 inline comments as done.
marten added a comment.
Diff updated.
INLINE COMMENTS
> cgiboudeaux wrote in FindPulseAudio.cmake:8-19
> It's a new module, change PULSEAUDIO to PulseAudio to match the module name.
PULSEAUDIO -> PulseAudio changed throughout
REPOSITORY
R240
marten updated this revision to Diff 19570.
marten added a comment.
Modules updated with standard header, documentation and copyright; endif()
used throughout.
Yes, Phonon does not use ECM but the modules there are very different to
those in ECM anyway.
Within Plasma+applications
marten created this revision.
Restricted Application added a project: Frameworks.
REVISION SUMMARY
As described in bug https://bugs.kde.org/show_bug.cgi?id=384500, there
appears to be a problem when the receiving application of a file needs a
temporary copy to be made (because of %F/%f in its
marten added a comment.
It's the same source really - the only differences between those in
kdelibs4support and ecm/attic are that the former uses endif(same_as_if) and
the latter uses endif().
Nothing else within Frameworks, Plasma or applications uses anything from
ecm/attic directly.
marten added a subscriber: heikobecker.
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D7823
To: marten, #frameworks, #build_system
Cc: heikobecker
marten created this revision.
Restricted Application added projects: Frameworks, Build System.
REVISION SUMMARY
These modules are used in a number of places within Frameworks, Plasma and
dependencies:
FindGLIB2:
plasma-desktop/applets/kimpanel/cmake/FindGLIB2.cmake
This revision was automatically updated to reflect the committed changes.
Closed by commit R267:1b641f89d752: Remove obviously wrongly-named symbolic
links (authored by marten).
REPOSITORY
R267 Oxygen Icons
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D5871?vs=14560=14564
REVISION
marten created this revision.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY
It's not clear where these entries came from, but they are clearly a mistake
- either scripting or copy-and-paste.
The
This revision was automatically updated to reflect the committed changes.
Closed by commit R226:c27a79e2dbab: Restore the ability to select Konqueror as
the default file manager (authored by marten).
CHANGED PRIOR TO COMMIT
https://phabricator.kde.org/D4463?vs=10973=10978#toc
REPOSITORY
marten created this revision.
marten added reviewers: Frameworks, dfaure.
marten set the repository for this revision to R226 Konqueror.
REVISION SUMMARY
Konqueror was originally able to be selected as the default file manager by
using kcmshell5 componentchooser ("Default Applications"). Now
marten added a comment.
Agreed that a decision needs to be made, but I'm not an expert on the
KPackage system and so wasn't sure whether the warning may be useful to package
developers in some cases - in which case leaving it commented out would make it
easier to reinstate than removing it
marten created this revision.
marten added reviewers: Frameworks, Plasma.
marten set the repository for this revision to R290 KPackage.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
As described in
This revision was automatically updated to reflect the committed changes.
Closed by commit R244:bbf76499d9e2: KAboutData: Document that bug email address
can also be a URL (authored by marten).
REPOSITORY
R244 KCoreAddons
CHANGES SINCE LAST UPDATE
marten added a dependent revision: D4080: KAboutData: Document that bug email
address can also be a URL.
REPOSITORY
R263 KXmlGui
REVISION DETAIL
https://phabricator.kde.org/D4072
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: marten, #frameworks
Cc:
marten added a dependency: D4072: Bug reporter: Allow a URL (not just as an
email address) for custom reporting.
REPOSITORY
R244 KCoreAddons
REVISION DETAIL
https://phabricator.kde.org/D4080
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: marten,
marten created this revision.
marten added a reviewer: Frameworks.
marten set the repository for this revision to R244 KCoreAddons.
Restricted Application added a project: Frameworks.
REVISION SUMMARY
Differential https://phabricator.kde.org/D4072 requests a change to the bug
reporting system
marten added a comment.
Yes, I'll submit a separate diff for that. Fortunately BC, since there are
no actual code changes needed - only the parameter name. The getter/setter
bugAddress/setBugAddress will suffice as they are (with apidoc changes).
REPOSITORY
R263 KXmlGui
REVISION DETAIL
marten created this revision.
marten added a reviewer: Frameworks.
marten set the repository for this revision to R263 KXmlGui.
Restricted Application added a project: Frameworks.
REVISION SUMMARY
The rather old bug https://bugs.kde.org/show_bug.cgi?id=277142, although
raised against KDE4
marked as submitted.
Review request for Dolphin and KDE Base Apps.
Changes
---
Submitted with commit 16db90c7938d09bfa8916afef8cce8dd258fa00d by Jonathan
Marten to branch master.
Bugs: 369523
https://bugs.kde.org/show_bug.cgi?id=369523
Repository: dolphin
Description
---
When
marked as submitted.
Review request for KDE Frameworks and David Faure.
Changes
---
Submitted with commit 842ac3256137b0d4fde129844ff6d5aac9da0090 by Jonathan
Marten to branch master.
Repository: khtml
Description
---
This is currently built and installed, but it is not usable
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129448/#review101299
---
Ping anyone?
- Jonathan Marten
On Nov. 20, 2016, 3:23
e embedded viewer part in
Konqueror and other applications.
Thanks,
Jonathan Marten
marked as submitted.
Review request for KDE Base Apps and David Faure.
Changes
---
Submitted with commit 7adf64addec6c9ca62dec8277e164c3ba93f837c by Jonathan
Marten to branch master.
Repository: konqueror
Description
---
This settings was originally commented out in the sources
--
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129518/#review101013
---
On Nov. 21, 2016, 12:25 p.m., Jonathan Marten wrote:
>
> -
of the preferred
association for the HTML MIME types.
Thanks,
Jonathan Marten
ons. Also verified by bug submitter.
Thanks,
Jonathan Marten
marked as submitted.
Review request for KDE Base Apps.
Changes
---
Submitted with commit aa247198f16179cf86768cfdeed94e7c557e864c by Jonathan
Marten to branch master.
Repository: keditbookmarks
Description
---
These actions are created by QUndoStack in order that it can manage
/commandhistory.cpp 53a8931
Diff: https://git.reviewboard.kde.org/r/129423/diff/
Testing
---
Built keditbookmarks (split repository master version) with this change,
observed correct appearance and operation of Undo/Redo actions.
Thanks,
Jonathan Marten
marked as submitted.
Review request for KDE Frameworks.
Changes
---
Submitted with commit 3f7465b8d7c1649b54eb6ce72e88d2a3080e2234 by Jonathan
Marten to branch master.
Repository: kconfigwidgets
Description
---
This function is intended to return the action for the StandardAction
erwriting an existing file,
unless the QFileDialog::DontConfirmOverwrite option is specified.
Thanks,
Jonathan Marten
ff: https://git.reviewboard.kde.org/r/127944/diff/
Testing
---
Built plasma-intergration with this change, confirmed correct operation of file
dialogues and that confirmation is requested when overwriting an existing file,
unless the QFileDialog::DontConfirmOverwrite option is specified.
Thanks,
Jonathan Marten
eated in build/autotests. The
> relevant one would be kfiledialog_unittest and kfiledialogqml_unittest
>
> Jonathan Marten wrote:
> Ok, found them and how - thanks.
>
> It doesn't appear to be possible to just check that the QFileDialog
> option is passed correctly th
rg/r/127944/#review95535
---
On May 17, 2016, 3:20 p.m., Jonathan Marten wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127944/
> --
reviewboard.kde.org/r/127944/#review95532
---
On May 17, 2016, 3:20 p.m., Jonathan Marten wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
>
kde.org/r/127944/diff/
Testing
---
Built plasma-intergration with this change, confirmed correct operation of file
dialogues and that confirmation is requested when overwriting an existing file,
unless the QFileDialog::DontConfirmOverwrite option is specified.
Thanks,
Jonathan Marten
n of file
dialogues and that confirmation is requested when overwriting an existing file,
unless the QFileDialog::DontConfirmOverwrite option is specified.
Thanks,
Jonathan Marten
marked as submitted.
Review request for KDE Frameworks.
Changes
---
Submitted with commit 75a55d449f2f070c5410e47a9529a984403c3297 by Jonathan
Marten to branch master.
Repository: kdelibs4support
Description
---
In KDE4 this widget displayed the appropriate country flag at the start
in dialogue.
Thanks,
Jonathan Marten
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
marked as submitted.
Review request for KDE Frameworks and David Faure.
Changes
---
Submitted with commit ab2f9fb2b7780c913a417b01700e33d3bb986f9c by Jonathan
Marten to branch master.
Repository: kparts
Description
---
This dialogue is used, e.g. in Konqueror, when asking to open
-bc92-f0da3b9b5b3e__konqueror-openwith-frameworks-breeze-after.png
Thanks,
Jonathan Marten
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
marked as submitted.
Review request for KDE Base Apps and David Faure.
Changes
---
Submitted with commit f868ba3b34b4f55111c3b6fb1ff0e6bcaa1d2797 by Jonathan
Marten to branch frameworks.
Repository: kde-baseapps
Description
---
This happens when a local file URL which ends
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127138/#review92652
---
On Feb. 22, 2016, 3:12 p.m., Jonathan Marten wrote:
>
>
ill on
> > the first level and then a "More..." option with further actions?
>
> Jonathan Marten wrote:
> I wouldn't have said "rarely used": certainly in my experience I use all
> of these options occasionally, but nowhere near as oft
atically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127046/#review92299
-------
On Feb. 11, 2016, 6:18 p.m., Jonathan Marten wrote:
>
> ---
&
iew.cpp 3707c7a
Diff: https://git.reviewboard.kde.org/r/127138/diff/
Testing
---
Built kde-baseapps with these changes, observed correct display of base name in
Konqueror window title while navigating.
Thanks,
Jonathan Marten
marked as submitted.
Review request for KDE Base Apps.
Changes
---
Submitted with commit fc5e0160d81c4fad2e54e2cf31e09395634d7783 by Jonathan
Marten to branch frameworks.
Repository: kde-baseapps
Description
---
These changes to Konqueror's setting up of actions eliminate
1/55c12311-73a3-472a-8a9a-6b6b9fce9de7__before.png
Popup menu after
https://git.reviewboard.kde.org/media/uploaded/files/2016/02/11/313dbe3b-efb4-475c-bf76-8516f5bc85b4__after.png
Thanks,
Jonathan Marten
___
Kde-frameworks-devel mailing list
Kde-
r and
runtime error messages, checked correct operation of key shortcuts.
Thanks,
Jonathan Marten
marked as submitted.
Review request for KDE Frameworks.
Changes
---
Submitted with commit 5ac5dfb28014035e3704ae8bf8076001e5955ceb by Jonathan
Marten to branch master.
Bugs: 355711
https://bugs.kde.org/show_bug.cgi?id=355711
Repository: kparts
Description
---
This appears
marked as submitted.
Review request for KDE Frameworks.
Changes
---
Submitted with commit 1ae59b5f0ea4f50b7350f03fd2b1fe223f66a261 by Jonathan
Marten to branch master.
Bugs: 352471
https://bugs.kde.org/show_bug.cgi?id=352471
Repository: kcmutils
Description
---
The API
described in the bug report. Observed assert/crash
without this change, and correct operation with it.
Thanks,
Jonathan Marten
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
ciated review). Observed no crash when exiting the
application. Also checked correct operation of Konqueror and Kate.
Thanks,
Jonathan Marten
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listin
how details was not being saved correctly. Using
!isHidden() instead of isVisible() gets the correct information.
Diffs
-
src/ioslaves/http/kcookiejar/kcookiewin.cpp 56a283f
Diff: https://git.reviewboard.kde.org/r/126245/diff/
Testing
---
Built kio with these changes, checked oper
,
Jonathan Marten
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
marked as submitted.
Review request for KDE Frameworks.
Changes
---
Submitted with commit 3bdd02a2fbcac6d7bc6a188ce2396160c4f22ed7 by Jonathan
Marten to branch master.
Repository: kio
Description
---
When starting this module, from the command line or similar, the error message
/interfaces/default/interface.cpp 505e0aa
Diff: https://git.reviewboard.kde.org/r/115183/diff/
Testing
---
Built kde-workspace with this change, verified operation of runner history.
Also verified by another bug reporter.
Thanks,
Jonathan Marten
/
Testing
---
Built kde-workspace with this change, verified operation of runner history.
Also verified by another bug reporter.
Thanks,
Jonathan Marten
.
Regards, Jonathan
--
Jonathan Marten http://www.keelhaul.demon.co.uk
Twickenham, UK j...@keelhaul.demon.co.uk
mode both by toolbar and menu, and that the correct items in each case are
checked.
Thanks,
Jonathan Marten
1 - 100 of 128 matches
Mail list logo
