Feed Advanced Search

Sep 4 2017

apol requested changes to D7649: Add nightly for experimental Konversation Qt Quick branch.

Put it in the root, todo/ stuff isn't built, it's for things that don't build.

Sep 4 2017, 11:16 AM
apol accepted D7649: Add nightly for experimental Konversation Qt Quick branch.

Yes, default branch is master.

Sep 4 2017, 11:09 AM
apol committed R304:c381ed0a1758: Don't complain the knsregistry file is not present before it's useful (authored by apol).
Don't complain the knsregistry file is not present before it's useful
Sep 4 2017, 10:25 AM
apol closed D7190: Don't complain the knsregistry file is not present before it's useful by committing R304:c381ed0a1758: Don't complain the knsregistry file is not present before it's useful.
Sep 4 2017, 10:25 AM · Frameworks
apol committed R304:04cc49c71bdb: Detach before setting the d pointer (authored by apol).
Detach before setting the d pointer
Sep 4 2017, 10:24 AM
apol closed D7194: Detach before setting the d pointer by committing R304:04cc49c71bdb: Detach before setting the d pointer.
Sep 4 2017, 10:24 AM · Frameworks
apol added a comment to D7273: Use the new ECMQMLModules to specify all of KWin's runtime dependencies.

ping, anything speaking against pushing it?

Sep 4 2017, 9:53 AM · Plasma

Sep 1 2017

apol accepted D7393: improve packaging of Babe music player.
Sep 1 2017, 12:43 PM
apol added a comment to T6805: Device class in C++ should use the d-pointer pattern.

Why?

Sep 1 2017, 12:32 PM · KDE Connect
apol committed R263:555409eb206b: Save up a bunch of stat() calls on application start (authored by apol).
Save up a bunch of stat() calls on application start
Sep 1 2017, 12:28 PM
apol closed D7230: Save up a bunch of stat() calls on application start by committing R263:555409eb206b: Save up a bunch of stat() calls on application start.
Sep 1 2017, 12:28 PM · Frameworks
apol accepted D7216: refactor kpackagetool away from stringy options.

Won't hurt, benefits from some compiler checking so it's better.

Sep 1 2017, 12:02 PM · Frameworks

Aug 14 2017

apol accepted D7295: Improve flatpak packaging of elisa to have baloo and mpris support.
Aug 14 2017, 12:06 PM

Aug 13 2017

apol accepted D7244: Wipe branch and point in README to new location.

Seems good to me, I'm unsure how useful the script will be, I guess it can't hurt. Haven't tried it though.

Aug 13 2017, 6:59 PM
apol accepted D7209: Fix Notifications in Plasmoid.
Aug 13 2017, 5:50 PM · KDE Connect
apol added a comment to D7273: Use the new ECMQMLModules to specify all of KWin's runtime dependencies.

I tested the patch, for me it worked for all but VirtualKeyboard (which might be indeed missing).

Aug 13 2017, 5:44 PM · Plasma
apol added a comment to D7279: Do not leak symbols of pimpl classes, protect with Q_DECL_HIDDEN.
Aug 13 2017, 5:35 PM · Frameworks
apol added a comment to D7253: Add build-flatpak target if there is a flatpak recipe around.

Ugh my bad ^^', I don't know what happened.

Aug 13 2017, 5:32 PM · Build System, Frameworks

Aug 11 2017

apol added a comment to D6549: Look for QtGraphicalEffects, so packagers don't forget it.

I was thinking: should I wait for a month so that ECM has been released with it maybe? Otherwise it will be a hard dependency and people won't be able to use the master branch...

Aug 11 2017, 11:23 PM · Kirigami
apol added a comment to D7253: Add build-flatpak target if there is a flatpak recipe around.

Main reason is having a simple way to run flatpak-builder with the proper arguments. Otherwise you have to copy-paste the command from somewhere (which could be error-prone).
One usually already has a build folder around, so the idea is you change some code, build it as usual and then you run this target to quickly test the change in flatpak.

Aug 11 2017, 7:04 PM · Build System, Frameworks
apol created T6768: Get builds for other architectures.
Aug 11 2017, 6:41 PM · Flatpak
apol closed T5849: Figure out QtWebEngine ARM32 build as Resolved.

Figured out:
https://commits.kde.org/flatpak-kde-runtime/d016215d753b6178fe615f92263eb4d693ef204a

Aug 11 2017, 6:38 PM · Flatpak
apol committed R257:d016215d753b: Fix arm32 builds (QtWebEngine) (authored by apol).
Fix arm32 builds (QtWebEngine)
Aug 11 2017, 6:30 PM
apol added a comment to T6750: Figure out how to use plugins installed by other apps.

Right, so this means konsole has to make this desktop file visible to the dolphin part, which at the moment it isn't. We should figure it out.

Aug 11 2017, 4:42 PM · Ark, Flatpak
apol added a comment to D7253: Add build-flatpak target if there is a flatpak recipe around.

Interesting feature.

Aug 11 2017, 4:40 PM · Build System, Frameworks
apol committed R134:8d84b3513e39: Merge branch 'Plasma/5.10' (authored by apol).
Merge branch 'Plasma/5.10'
Aug 11 2017, 4:27 PM
apol committed R134:864b8152cf90: Initially populate the installed snaps (authored by apol).
Initially populate the installed snaps
Aug 11 2017, 4:27 PM
apol committed R134:37cfd9b58e3b: Remove the navigationEnabled concept (authored by apol).
Remove the navigationEnabled concept
Aug 11 2017, 11:50 AM
apol added a comment to T6750: Figure out how to use plugins installed by other apps.

Who installs konsolehere.desktop?

Aug 11 2017, 12:34 AM · Ark, Flatpak
apol committed R224:28f11bd5c9a7: Fix receiving files with invalid URL characters (authored by apol).
Fix receiving files with invalid URL characters
Aug 11 2017, 12:31 AM
apol closed D7224: Fix receiving files with invalid URL characters by committing R224:28f11bd5c9a7: Fix receiving files with invalid URL characters.
Aug 11 2017, 12:31 AM

Aug 10 2017

apol committed R134:a95d91b3a382: Merge branch 'Plasma/5.10' (authored by apol).
Merge branch 'Plasma/5.10'
Aug 10 2017, 5:56 PM
apol committed R134:217316656466: Fix license, comply with our naming scheme (authored by apol).
Fix license, comply with our naming scheme
Aug 10 2017, 5:55 PM
apol committed R134:e28b98bb945b: Don't disable the updates action when there are no updates (authored by apol).
Don't disable the updates action when there are no updates
Aug 10 2017, 5:34 PM
apol committed R323:3d71e59fefbb: Fix build with clang (authored by apol).
Fix build with clang
Aug 10 2017, 2:37 PM
apol committed R325:b3f4ee2587ed: Prefer checking on the target rather than a variable (authored by apol).
Prefer checking on the target rather than a variable
Aug 10 2017, 2:04 PM
apol committed R325:635d1cfa5da5: Move global dependencies outside of the desktop application if (authored by apol).
Move global dependencies outside of the desktop application if
Aug 10 2017, 2:04 PM
apol created D7230: Save up a bunch of stat() calls on application start.
Aug 10 2017, 1:17 PM · Frameworks
apol committed R256:fe4e0b49001d: Include libcurl as discussed in task T6758 (authored by apol).
Include libcurl as discussed in task T6758
Aug 10 2017, 1:02 PM
apol closed T6759: Flatpak: libcurl is built without smtp support as Resolved.

Included libcurl: https://commits.kde.org/flatpak-kde-applications/fe4e0b49001d46c26dd1246cc2376d80caaffe5f

Aug 10 2017, 1:01 PM · Flatpak, Kube
apol committed R94:bd05e300d21e: Add missing linked library on a test (authored by apol).
Add missing linked library on a test
Aug 10 2017, 12:42 PM
apol accepted D7227: Don't show appstream action if we can't find an app to open appstream:// URLs..

LGTM

Aug 10 2017, 10:37 AM · Plasma
apol added a comment to D7192: Receiving a file with a '#' or '?' in the name from Android no longer results in the name cropped..
In D7192#134257, @jeanv wrote:
In D7192#134232, @apol wrote:

I see what you mean. Your solution does seem better.

Aug 10 2017, 10:30 AM · KDE Connect
apol committed R257:8432eb7d00f7: Remove non-understood option (authored by apol).
Remove non-understood option
Aug 10 2017, 2:28 AM

Aug 9 2017

apol committed R240:0cb818ef83d8: Forgot to add the test (authored by apol).
Forgot to add the test
Aug 9 2017, 11:54 PM
apol added a comment to T6759: Flatpak: libcurl is built without smtp support.

Where are we using libcurl? Would it make sense to build it separately in Kube's flatpak manifest?

Aug 9 2017, 11:36 PM · Flatpak, Kube
apol added a comment to T6750: Figure out how to use plugins installed by other apps.

What does the ark plugin do? Does it need ark code? If not maybe it could be moved into dolphin-plugins.

Aug 9 2017, 11:33 PM · Ark, Flatpak
apol added a comment to D7192: Receiving a file with a '#' or '?' in the name from Android no longer results in the name cropped..

Please see https://phabricator.kde.org/D7224

Aug 9 2017, 11:30 PM · KDE Connect
apol created D7224: Fix receiving files with invalid URL characters.
Aug 9 2017, 11:29 PM
apol added a comment to D7219: Add separate interface library KDev::IExecute for execute/iexecuteplugin.h.

Should we move (some of) this into kdevplatform/interfaces?

Aug 9 2017, 11:07 PM
apol added a comment to D7217: Separate internal macro into KDevPlatformMacrosInternal.cmake.

The advantage of using BUILD_INTEFACE is that you're setting the paths depending on what you're linking against rather than depending on where they are located which is essentially arbitrary.

Aug 9 2017, 11:05 PM
apol requested changes to D7192: Receiving a file with a '#' or '?' in the name from Android no longer results in the name cropped..
Aug 9 2017, 8:52 PM · KDE Connect
apol added a comment to D7192: Receiving a file with a '#' or '?' in the name from Android no longer results in the name cropped..

To me it sounds like the issue you are trying to solve is elsewhere.

Aug 9 2017, 8:50 PM · KDE Connect
apol committed R99:fdb0df568637: Fix look-up of gtk preview modules (authored by apol).
Fix look-up of gtk preview modules
Aug 9 2017, 4:22 PM
apol added a comment to D7216: refactor kpackagetool away from stringy options.

if that makes you happy, it makes me happy.
+1

Aug 9 2017, 3:39 PM · Frameworks
apol committed R256:7fe3d8be23c7: Include builds for the portal test (authored by apol).
Include builds for the portal test
Aug 9 2017, 2:09 PM
apol updated the diff for D6549: Look for QtGraphicalEffects, so packagers don't forget it.

Use the macro in ECM now

Aug 9 2017, 11:14 AM · Kirigami
apol committed R240:e5301edf1daf: Include a module for finding qml imports as runtime dependencies (authored by apol).
Include a module for finding qml imports as runtime dependencies
Aug 9 2017, 11:10 AM
apol closed D7094: Include a module for finding qml imports as runtime dependencies by committing R240:e5301edf1daf: Include a module for finding qml imports as runtime dependencies.
Aug 9 2017, 11:10 AM · Build System, Frameworks
apol updated the diff for D7094: Include a module for finding qml imports as runtime dependencies.

if no qmlplugindump

Aug 9 2017, 10:31 AM · Build System, Frameworks
apol committed R252:7524bf1535b4: Fix accepting dialogs with ctrl+return when buttons are renamed (authored by apol).
Fix accepting dialogs with ctrl+return when buttons are renamed
Aug 9 2017, 10:27 AM
apol closed D7196: Fix accepting dialogs with ctrl+return when buttons are renamed by committing R252:7524bf1535b4: Fix accepting dialogs with ctrl+return when buttons are renamed.
Aug 9 2017, 10:27 AM · Frameworks
apol committed R288:35b442449166: Initialize the "Pause" button state in the widget tracker (authored by apol).
Initialize the "Pause" button state in the widget tracker
Aug 9 2017, 10:26 AM
apol closed D7058: Initialize the "Pause" button state in the widget tracker by committing R288:35b442449166: Initialize the "Pause" button state in the widget tracker.
Aug 9 2017, 10:26 AM · Frameworks
apol added inline comments to D7094: Include a module for finding qml imports as runtime dependencies.
Aug 9 2017, 2:06 AM · Build System, Frameworks
apol updated the diff for D7094: Include a module for finding qml imports as runtime dependencies.

Address sitter's comments

Aug 9 2017, 2:06 AM · Build System, Frameworks
apol committed R32:73b08bb61ff0: Move some logic still from kdevplatform into kdevelop (authored by apol).
Move some logic still from kdevplatform into kdevelop
Aug 9 2017, 12:58 AM

Aug 8 2017

apol requested changes to D7209: Fix Notifications in Plasmoid.
Aug 8 2017, 11:47 PM · KDE Connect
apol added inline comments to D7213: Use ecm_add_test and benefit from BUILD_TESTING awareness.
Aug 8 2017, 11:43 PM · Frameworks
apol accepted D7201: Install required runtime for flatpakref if it's specified.
Aug 8 2017, 9:00 PM · Plasma
apol committed R256:0ac1ff9d8884: kdevplatform is in kdevelop now (authored by apol).
kdevplatform is in kdevelop now
Aug 8 2017, 5:15 PM
apol committed R32:731e87d4c8d6: Remove unneeded includes (authored by apol).
Remove unneeded includes
Aug 8 2017, 5:15 PM
apol committed R32:6199b7918fca: Merge KDevPlatform into KDevelop (authored by apol).
Merge KDevPlatform into KDevelop
Aug 8 2017, 5:15 PM
apol committed R32:391e53faef2f: Have kdevplatform be compiled within kdevelop (authored by apol).
Have kdevplatform be compiled within kdevelop
Aug 8 2017, 5:15 PM
apol committed R32:7c0f522ae729: Use docker itself to create the file system that kdev will access (authored by apol).
Use docker itself to create the file system that kdev will access
Aug 8 2017, 5:14 PM
apol committed R32:b708c79f680d: Fix crash when removing toolview via context menu (authored by croick).
Fix crash when removing toolview via context menu
Aug 8 2017, 5:12 PM
apol committed R32:d68205557404: Fix crash when removing toolview via context menu (authored by croick).
Fix crash when removing toolview via context menu
Aug 8 2017, 5:12 PM
apol committed R32:e7ac5512b44f: Also test the project directory conversion (authored by apol).
Also test the project directory conversion
Aug 8 2017, 5:12 PM
apol committed R32:a57a1e06a9fd: Be less restrictive about the documentation index type (authored by apol).
Be less restrictive about the documentation index type
Aug 8 2017, 5:12 PM
apol committed R32:3bba8e3592a9: Fix build with Qt 5.6 (authored by apol).
Fix build with Qt 5.6
Aug 8 2017, 5:12 PM
apol committed R32:0545244f0c6a: Improve docker test (authored by apol).
Improve docker test
Aug 8 2017, 5:12 PM
apol committed R32:63aeffa9979f: Don't report as test failed if docker is not set up on the system (authored by apol).
Don't report as test failed if docker is not set up on the system
Aug 8 2017, 5:12 PM
apol committed R32:fbf12250b196: The focus goes to the widget (authored by apol).
The focus goes to the widget
Aug 8 2017, 5:12 PM
apol committed R32:3b4dc9960aef: Merge branch '5.1' (authored by apol).
Merge branch '5.1'
Aug 8 2017, 5:12 PM
apol committed R32:be6918b90436: Merge branch '5.1' (authored by apol).
Merge branch '5.1'
Aug 8 2017, 5:12 PM
apol committed R32:e4070795b680: Fix build (authored by apol).
Fix build
Aug 8 2017, 5:12 PM
apol committed R32:c48b593d2a3b: Make sure we don't get an empty element in the context menu (authored by apol).
Make sure we don't get an empty element in the context menu
Aug 8 2017, 5:12 PM
apol committed R32:e4be520223c7: Fix history management in StandardDocumentationView + QtWebEngine (authored by apol).
Fix history management in StandardDocumentationView + QtWebEngine
Aug 8 2017, 5:12 PM
apol committed R32:0bc255343db2: Fix JS warning in QtWebEngine (authored by apol).
Fix JS warning in QtWebEngine
Aug 8 2017, 5:12 PM
apol committed R32:e56ff47dc61f: Fix warning, properly construct pointer (authored by apol).
Fix warning, properly construct pointer
Aug 8 2017, 5:12 PM
apol committed R32:8d4fc76a89e1: Remove the last '\n' when copying from an output view (authored by apol).
Remove the last '\n' when copying from an output view
Aug 8 2017, 5:12 PM
apol committed R32:4fd52c832bd9: Include a template for Dockerfiles (authored by apol).
Include a template for Dockerfiles
Aug 8 2017, 5:11 PM
apol committed R32:81bc3136299c: Add a flatpak builder manifest template (authored by apol).
Add a flatpak builder manifest template
Aug 8 2017, 5:11 PM
apol committed R32:4bc02aafcf78: Remove unneeded member (authored by apol).
Remove unneeded member
Aug 8 2017, 5:11 PM
apol committed R32:94f6a4427079: Display local paths as paths rather than as file:/// urls (authored by apol).
Display local paths as paths rather than as file:/// urls
Aug 8 2017, 5:11 PM
apol committed R32:c7d1b1cfacb5: Fix porting error (authored by apol).
Fix porting error
Aug 8 2017, 5:11 PM
apol committed R32:1e6bcc721462: Expose flatpak build calls to dbus (authored by apol).
Expose flatpak build calls to dbus
Aug 8 2017, 5:11 PM
apol committed R32:a5d3cb524e5a: Fix issues detected by the test (authored by apol).
Fix issues detected by the test
Aug 8 2017, 5:11 PM
apol committed R32:448b9c84ff14: Convert the executed application by default (authored by apol).
Convert the executed application by default
Aug 8 2017, 5:11 PM
apol committed R32:d5c0dd246ffd: Also pass the finishArgs when executing processes (authored by apol).
Also pass the finishArgs when executing processes
Aug 8 2017, 5:11 PM