D11170: Use new kpackage_install_bundled_package macro

markey accepted this revision. markey added a comment. This revision is now accepted and ready to land. Restricted Application removed a project: Amarok. Restricted Application added a subscriber: amarok-devel. Ubuntu 18.04 is now released, and your patch builds fine with it. REPOSITORY

D11170: Use new kpackage_install_bundled_package macro

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D11170 To: malteveerman, #amarok, markey Cc: amarok-devel, markey, heikobecker, #amarok

D12492: Fixed fadebars

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D12492 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber,

D12492: Fixed fadebars

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH master REVISION DETAIL https://phabricator.kde.org/D12492 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise,

D12316: Some analyzer applet tweaks

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D12316 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber,

D12315: Fixed crash at shutdown

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D12315 To: malteveerman, #amarok, markey Cc: anthonyfieroni, #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, asturmlechner,

D12315: Fixed crash at shutdown

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH master REVISION DETAIL https://phabricator.kde.org/D12315 To: malteveerman, #amarok, markey Cc: anthonyfieroni, #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman,

D12223: Improvements to CurrentTrack applet

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH master REVISION DETAIL https://phabricator.kde.org/D12223 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise,

D12223: Improvements to CurrentTrack applet

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D12223 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber,

D11968: More interface stuttering work

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH master REVISION DETAIL https://phabricator.kde.org/D11968 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise,

D11968: More interface stuttering work

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D11968 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber,

D11862: Get rid of scripting

markey added a comment. See, the problem with removing features like this is, there will be a huge outcry from users. That's guaranteed, even without knowing for sure how much the scripting feature is really used in the wild. I was discussing about this with Heiko. Maybe QtScript could

D11868: Reduce interface stuttering

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D11868 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber,

D11867: Logger overhaul

markey closed this revision. markey added a comment. Pushed to master. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D11867 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar,

D11867: Logger overhaul

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH master REVISION DETAIL https://phabricator.kde.org/D11867 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise,

D11788: Fixed wrong slot signatures in BrowserMessageArea

This revision was automatically updated to reflect the committed changes. Closed by commit R181:5176c93e41e4: Fixed wrong slot signatures in BrowserMessageArea (authored by malteveerman, committed by markey). REPOSITORY R181 Amarok CHANGES SINCE LAST UPDATE

D11788: Fixed wrong slot signatures in BrowserMessageArea

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH master REVISION DETAIL https://phabricator.kde.org/D11788 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise,

D11787: Small MySQL libraries fix for tests.

This revision was automatically updated to reflect the committed changes. Closed by commit R181:ef7ec2c70f60: Small MySQL libraries fix for tests. (authored by malteveerman, committed by markey). REPOSITORY R181 Amarok CHANGES SINCE LAST UPDATE

D11787: Small MySQL libraries fix for tests.

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH master REVISION DETAIL https://phabricator.kde.org/D11787 To: malteveerman, #amarok, markey Cc: #amarok, jackyalcine, nhuisman, paul, yaohanchen, malteveerman, markey, cochise,

D11534: Some small changes and fixes

This revision was automatically updated to reflect the committed changes. Closed by commit R181:16d825772150: Some small changes and fixes (authored by malteveerman, committed by markey). REPOSITORY R181 Amarok CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11534?vs=30067=30529

D11534: Some small changes and fixes

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH master REVISION DETAIL https://phabricator.kde.org/D11534 To: malteveerman, #amarok, markey Cc: #amarok, nhuisman, paul, yaohanchen, malteveerman, markey, cochise, tbettler,

D11349: Reactivate wikipedia applet

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R181:7a63540fc23a: Reactivate wikipedia applet (authored by malteveerman, committed by markey). REPOSITORY R181 Amarok

D11349: Reactivate wikipedia applet

markey requested changes to this revision. markey added a comment. This revision now requires changes to proceed. Generally this works for me now, but there are two more issues that I found: If the optional Qt5WebEngine is not found, we get a CMake error anyway:

D11349: Reactivate wikipedia applet

markey requested changes to this revision. markey added a comment. This revision now requires changes to proceed. Fails to build here: /home/mark/devel/src/amarok/src/context/AppletModel.cpp: In member function ‘QUrl Context::AppletModel::imageUrl(const QString&, const QString&)’:

D11299: Fixed a crash in magnatune service

This revision was automatically updated to reflect the committed changes. Closed by commit R181:d527cce9963c: Fixed a crash in magnatune service (authored by malteveerman, committed by markey). REPOSITORY R181 Amarok CHANGES SINCE LAST UPDATE

D11299: Fixed a crash in magnatune service

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH master REVISION DETAIL https://phabricator.kde.org/D11299 To: malteveerman, #amarok, markey Cc: #amarok, nhuisman, paul, yaohanchen, malteveerman, markey, cochise, tbettler,

D11171: Port NetworkAccessManager to new signal/slot syntax

This revision was automatically updated to reflect the committed changes. Closed by commit R181:12d7ecbfe6e8: Port NetworkAccessManager to new signal/slot syntax (authored by malteveerman, committed by markey). REPOSITORY R181 Amarok CHANGES SINCE LAST UPDATE

D11171: Port NetworkAccessManager to new signal/slot syntax

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH master REVISION DETAIL https://phabricator.kde.org/D11171 To: malteveerman, #amarok, markey Cc: markey, #amarok, nhuisman, paul, yaohanchen, malteveerman, cochise, tbettler,

D11192: Ported playdar collection away from qjson

This revision was automatically updated to reflect the committed changes. Closed by commit R181:16d4997c3792: Ported playdar collection away from qjson (authored by malteveerman, committed by markey). REPOSITORY R181 Amarok CHANGES SINCE LAST UPDATE

D11192: Ported playdar collection away from qjson

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH master REVISION DETAIL https://phabricator.kde.org/D11192 To: malteveerman, #amarok, markey Cc: #amarok, nhuisman, paul, yaohanchen, malteveerman, markey, cochise, tbettler,

D11171: Port NetworkAccessManager to new signal/slot syntax

markey requested changes to this revision. markey added a comment. This revision now requires changes to proceed. This does not build for me (GCC 7.2.0). Error: https://pastebin.com/5Pmk1ELp REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D11171 To: malteveerman,

Re: Amarok 2.9.0 "Hibernaculum" released

So if you don't want to go through the hassle of checking it out from Git, you could simply delete that one line in the source code, as indicated by Heiko's linked patch. Sorry for the trouble :) Mark On Fri, Mar 9, 2018 at 8:42 AM, Heiko Becker wrote: > Hello Michał, >

D11150: Use Kirigami's spacing and icon sizes and remove own implementation

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D11150 To: malteveerman, #amarok, markey Cc: markey, #amarok, nhuisman, paul, yaohanchen, malteveerman, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber, heikobecker

D11150: Use Kirigami's spacing and icon sizes and remove own implementation

markey accepted this revision. markey added a comment. This revision is now accepted and ready to land. Kirigami2 seems really nifty. Nice choice. REPOSITORY R181 Amarok BRANCH kf5 REVISION DETAIL https://phabricator.kde.org/D11150 To: malteveerman, #amarok, markey Cc: markey,

D11033: Fix some deprecation warnings

This revision was automatically updated to reflect the committed changes. Closed by commit R181:c9257ade2df7: Fix some deprecation warnings (authored by malteveerman, committed by markey). REPOSITORY R181 Amarok CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11033?vs=28634=28974

D11033: Fix some deprecation warnings

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH kf5 REVISION DETAIL https://phabricator.kde.org/D11033 To: malteveerman, #amarok, markey Cc: #amarok, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar,

Re: A proposal to release 2.9

Hi, no crash on exit for me with latest master. I'll try to figure this out with Heiko on IRC. Is anyone else here getting this crash? Mark On Sun, Mar 4, 2018 at 1:50 PM, Heiko Becker wrote: > Hi, > > On 02/24/18 23:58, Stefano Pettini wrote: > > it looks like

D10923: Make Webkit optional

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH webkit-optional REVISION DETAIL https://phabricator.kde.org/D10923 To: heikobecker, #amarok, markey Cc: markey, paul, yaohanchen, malteveerman, cochise, tbettler, Smar,

D10923: Make Webkit optional

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D10923 To: heikobecker, #amarok, markey Cc: markey, paul, yaohanchen, malteveerman, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber, heikobecker

D10924: Disable Discogs and Google cover search temporarily

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D10924 To: heikobecker, #amarok, markey Cc: paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber, heikobecker

D10924: Disable Discogs and Google cover search temporarily

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH disable-cover-search-providers REVISION DETAIL https://phabricator.kde.org/D10924 To: heikobecker, #amarok, markey Cc: paul, yaohanchen, malteveerman, markey, cochise, tbettler,

D10978: Port tests to Qt5/KF5

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH kf5 REVISION DETAIL https://phabricator.kde.org/D10978 To: malteveerman, #amarok, markey Cc: #amarok, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar,

D10971: Fixed a crash in analyzer applet when sample rate reported by engine controller is zero

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D10971 To: malteveerman, #amarok, markey Cc: #amarok, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber, heikobecker

D10971: Fixed a crash in analyzer applet when sample rate reported by engine controller is zero

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH kf5 REVISION DETAIL https://phabricator.kde.org/D10971 To: malteveerman, #amarok, markey Cc: #amarok, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar,

D10900: Ported importers to Qt5/KF5

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D10900 To: malteveerman, #amarok, markey Cc: #amarok, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber, heikobecker

D10900: Ported importers to Qt5/KF5

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH kf5 REVISION DETAIL https://phabricator.kde.org/D10900 To: malteveerman, #amarok, markey Cc: #amarok, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar,

D10923: Make Webkit optional

markey added a comment. Is this patch meant for master? Because the branch is "webkit-optional". REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D10923 To: heikobecker, #amarok Cc: markey, paul, yaohanchen, malteveerman, cochise, tbettler, Smar, ricktimmis,

D10831: Ported some services to Qt5/KF5

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH kf5 REVISION DETAIL https://phabricator.kde.org/D10831 To: malteveerman, #amarok, markey Cc: asturmlechner, #amarok, paul, yaohanchen, malteveerman, markey, cochise, tbettler,

D10831: Ported some services to Qt5/KF5

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D10831 To: malteveerman, #amarok, markey Cc: asturmlechner, #amarok, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, schweingruber

D10819: Fix for Amarok::saveLocation() not setting separator before subdirectory.

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D10819 To: malteveerman, #amarok, markey Cc: #amarok, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber

D10819: Fix for Amarok::saveLocation() not setting separator before subdirectory.

markey added a comment. Hey Malte, please fix your real name for the next patch. KDE commit hook rejects this: remote: Audit failure - Commit 9ecce24b6dfe11a2c67a395d5826f6a7f4296749 - Non-full name: maldela REPOSITORY R181 Amarok BRANCH kf5 REVISION DETAIL

D10708: New context area

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D10708 To: malteveerman, #amarok, markey Cc: asturmlechner, heikobecker, #amarok, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, schweingruber

D10708: New context area

markey accepted this revision. This revision is now accepted and ready to land. REPOSITORY R181 Amarok BRANCH kf5 REVISION DETAIL https://phabricator.kde.org/D10708 To: malteveerman, #amarok, markey Cc: asturmlechner, heikobecker, #amarok, paul, yaohanchen, malteveerman, markey,

Re: A proposal to release 2.9

Hello Heiko, this sounds like a good proposal. I can also provide some help as I've done releases in the past. About merging KF5 branch into master, I'm not sure how feasible this is considering that both branches have diverged a lot. Perhaps we will need to ask the admins to switch (or rather

D10526: Port core application away from kdelibs4

markey closed this revision. REPOSITORY R181 Amarok REVISION DETAIL https://phabricator.kde.org/D10526 To: malteveerman, #amarok, markey Cc: heikobecker, asturmlechner, stikonas, kpiwowarski, markey, spettini, #amarok, paul, yaohanchen, malteveerman, cochise, tbettler, Smar, ricktimmis,

D10526: Port core application away from kdelibs4

markey accepted this revision. markey added a comment. This revision is now accepted and ready to land. Looks good to me. Builds and works fine too. REPOSITORY R181 Amarok BRANCH kf5 REVISION DETAIL https://phabricator.kde.org/D10526 To: malteveerman, #amarok, markey Cc: heikobecker,

D10526: Port core application away from kdelibs4

markey added a comment. In D10526#206680 , @spettini wrote: > Can I suggest we release 2.9 as planned, with the current state and maybe > some more bugfixes, before integrating this patch? Once migration to kf5 is > complete, and new

Fwd: D8229: streamlined plugin system

Forwarded to get more exposure. FYI, sysadmins have now also subscribed this mailing list to Phabricator, so in the future we will get notifications. -- Forwarded message -- From: Malte Veerman Date: Mon, Oct 9, 2017 at 11:53 PM Subject: D8229:

Re: Review Request 115154: Analyzer applet ASCII visualizer

. Shall we commit this patch for you, or do you have commit rights for the KDE git repo? - Mark Kretschmann On Jan. 20, 2014, 4:04 p.m., Matej Repinc wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 118861: GSoC Review 1: parsing and highlighting of lyrics

a const reference. - Mark Kretschmann On June 21, 2014, 11:42 a.m., Vedant Agarwala wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118861

Invitation for Randa Meetings 2014

the surroundings. More detailed information (booking, etc) will soon be available from our organizer Mario Fux. It would be nice if we could already get a general overview of who would be interested in coming. Please reply to this email thread. Regards, Mark. -- Mark Kretschmann Amarok Developer

Re: Review Request 110036: WIP - Simple equalizer scripting

On July 2, 2013, 10:50 a.m., Mark Kretschmann wrote: Any news on this? Could you please upload the patch again? Matěj Laitl wrote: There has been no feedback for 6 months and I believe this could have been superseded by Scripting GSoC 2013 project - should this be discarded? Ok

[amarok] tests: Fix compilation with unit tests enabled.

Git commit 5b3c84b29c16931e46bf7e1ddba16095245ce070 by Mark Kretschmann. Committed on 14/12/2013 at 09:25. Pushed by markey into branch 'master'. Fix compilation with unit tests enabled. The problem was clashing symbols between KDE's kWarning macro and gmock. Thanks go to Edward for help

Re: Review Request 114406: Fetch cover picture from METADATA_BLOCK_PICTURE tag

not to be taken literally. Of course we can commit it for you :) I'll do it tonight when I'm back home. - Mark Kretschmann On Dec. 12, 2013, 10:51 a.m., Martin Brodbeck wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Build errors with GMock

Erm, I meant s/Clang 1.7/GMock 1.7 On Fri, Dec 13, 2013 at 6:57 AM, Mark Kretschmann kretschm...@kde.org wrote: Hey all, lately I'm getting strange errors when building Git master with unit tests. At first I thought it was just my configuration (GMock 1.6, Clang), but now another user on IRC

Re: Review Request 114406: Fetch cover picture from METADATA_BLOCK_PICTURE tag

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114406/#review45583 --- Ship it! Ship It! - Mark Kretschmann On Dec. 11, 2013, 2

Re: Review Request 113876: Linking error in newly created files

://git.reviewboard.kde.org/r/113876/#comment32396 * Q_OBJECT Macro is missing * AcustId looks like a typo of AcoustId. - Mark Kretschmann On Nov. 15, 2013, 9:54 a.m., Vedant Agarwala wrote: --- This is an automatically generated e

Re: Review Request 113384: GSoC 2013 Revamping Scripting - Part 1/6 : Scripting wrappers

://git.reviewboard.kde.org/r/113384/#comment31447 I think it is confusing to abuse the slots keyword like this. Is there an alternative? Or at least a comment should be added. - Mark Kretschmann On Oct. 22, 2013, 4:07 p.m., Anmol Ahuja wrote

Re: Review Request 113386: GSoC 2013 Revamping Scripting - Part 3/6 : Scripts

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113386/#review43812 --- Ship it! Ship It! - Mark Kretschmann On Oct. 22, 2013, 4

Re: Review Request 113389: GSoC 2013 Revamping Scripting - Part 5/6 : Misc shared/

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113389/#review43819 --- Ship it! Ship It! - Mark Kretschmann On Oct. 22, 2013, 4

Re: Review Request 113390: GSoC 2013 Revamping Scripting - Part 6/6 : Misc src/

://git.reviewboard.kde.org/r/113390/#comment31458 Can be changed to: return findSpecFile( subdir ); - Mark Kretschmann On Oct. 22, 2013, 4:06 p.m., Anmol Ahuja wrote: --- This is an automatically generated e-mail. To reply

[amarok] /: Enable C++11 features, as discussed with other core devs.

Git commit 271ebaed8bd29a7ca78707fa6a696418c7c008b3 by Mark Kretschmann. Committed on 20/10/2013 at 16:09. Pushed by markey into branch 'master'. Enable C++11 features, as discussed with other core devs. Let the fun begin :) CCMAIL: amarok-devel@kde.org M +2-0CMakeLists.txt M +2

[amarok] /: Rewrite Block Analyzer to use pure OpenGL instead of QGLPaintEngine2.

Git commit f4a3f4fcf59cc0c592bd6d703542cb162a19721c by Mark Kretschmann. Committed on 19/07/2013 at 08:37. Pushed by markey into branch 'master'. Rewrite Block Analyzer to use pure OpenGL instead of QGLPaintEngine2. This rewrite should fix a number of issues that users had, especially with Intel

Changes in GMock

if we can find a better solution. -- Mark Kretschmann Amarok Developer Fellow of the Free Software Foundation Europe http://amarok.kde.org - http://fsfe.org ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok

Re: Added support for Visualizations using VSXu

On Tue, Sep 10, 2013 at 7:50 PM, Konrad Zemek konrad.ze...@gmail.com wrote: On 10.09.2013 13:22, Mark Kretschmann wrote: On Sun, Sep 8, 2013 at 9:04 PM, Dinesh saidine...@gmail.com wrote: Hi Folks, Finally done adding support for Visualizations using VSXu.(!!!) In order to test this out

Re: Added support for Visualizations using VSXu

be an issue with my Intel driver, or something else). If any of you folks want to try it too, that would be great, so we can see if the problem is reproducible elsewhere. If you have any build issues we can help. -- Mark Kretschmann Amarok Developer Fellow of the Free Software Foundation Europe http

Re: Amarok Scripting API

___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel -- Mark Kretschmann Amarok Developer Fellow of the Free Software Foundation Europe http://amarok.kde.org - http://fsfe.org ___ Amarok-devel

Re: Review Request 112417: FIX: 283022 - Performance with large feed (6000 items) very poor

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112417/#review39570 --- Ship it! Looks good to me. - Mark Kretschmann On Aug. 31

Re: SoK - Implementing cue sheet and embedded cue sheet support in Amarok

@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel -- Mark Kretschmann Amarok Developer Fellow of the Free Software Foundation Europe http://amarok.kde.org - http://fsfe.org ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org

Re: Review Request 112266: Fix reading Album Artist / Compilation / Disc Number in APE tags

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112266/#review38939 --- Ship it! Ship It! - Mark Kretschmann On Aug. 30, 2013, 1

Re: Amarok Scripting API

. -- Mark Kretschmann Amarok Developer Fellow of the Free Software Foundation Europe http://amarok.kde.org - http://fsfe.org ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Need a mentor for an SoK idea

at my propoasl from the SoK site, I see that all the formatting, HTML links and even whitespaces other than space, have been removed. Now it just looks like a ugly long paragraph :( That's terrible indeed. Could you edit the text and format it better? -- Mark Kretschmann Amarok Developer Fellow

Re: Review Request 112248: 313504: prefer QLatin1String

deteriorates code readability for very little performance gain. Frank, most of the checks on EBN are for micro-optimizations. They are low hanging fruit, and as such they generally make sense to follow, but there are corner cases like this one where the cost outweighs the benefits. - Mark

Re: Review Request 112169: Stop after current track to tray menu

though why the keyboard shortcut doesn't suffice. - Mark Kretschmann On Aug. 20, 2013, 2:50 a.m., Shitikanth Kashyap wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112169

Re: Review Request 112221: 313504: Prefer QChar overloads over the QString ones for efficiency (left overs)

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112221/ --- (Updated Aug. 25, 2013, 8:45 a.m.) Status -- This change has been

Re: Review Request 111824: Adds multithreaded transcoding to IpodCollection ie fixes BUG 317093

this? - Mark Kretschmann On July 31, 2013, 6:22 p.m., Vedant Agarwala wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111824

Re: Review Request 109369: Bug #254404: Copy files to USB storage devices in display order by sorting tracks in CollectionLocation.cpp

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109369/#review38520 --- What's the current status of this patch? - Mark Kretschmann

Re: Review Request 112266: Fix reading Album Artist / Compilation / Disc Number in APE tags

On Aug. 25, 2013, 12:06 p.m., Matěj Laitl wrote: shared/tag_helpers/APETagHelper.cpp, lines 33-37 http://git.reviewboard.kde.org/r/112266/diff/1/?file=184486#file184486line33 Hmm, is there a specification somewhere that says what the proper identifiers are? I fear of backwards

Re: ToolTips and Automatic Formatting

On Fri, Aug 16, 2013 at 10:44 AM, Mark Kretschmann kretschm...@kde.org wrote: Heya, as we noticed recently, many of Amarok's ToolTips look very ugly and have readability issues because they are not formatted. Therefore a longer ToolTip is being rendered as one giant long line. The Qt API

BACKPORT keyword

Heya, I just had a smart idea, which happens rarely enough: If we make commits where we already know that they should be backported later, put the keyword BACKPORT in the commit log. Later when we assemble the stable branch we can then simply grep over the whole log. -- Mark Kretschmann Amarok

ToolTips and Automatic Formatting

you think is the best way to go forward? -- Mark Kretschmann Amarok Developer Fellow of the Free Software Foundation Europe http://amarok.kde.org - http://fsfe.org ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo

Re: ToolTips and Automatic Formatting

-0 sed -i 's/subdomainA.example.com/subdomainB.example.com/g' -- Mark Kretschmann Amarok Developer Fellow of the Free Software Foundation Europe http://amarok.kde.org - http://fsfe.org ___ Amarok-devel mailing list Amarok-devel@kde.org https

Re: Introduction: Season of KDE 2013

helps out the others if they have any questions. So I don't think that it's a huge issue, as you would still be able to get the advice and guidance you need. If you are ok with this, we could continue discussing the details of your project. -- Mark Kretschmann Amarok Developer Fellow of the Free

Re: Review Request 112044: Fixes: 322220 - Wikipedia applet shouldn't reload content if the artist/album hasn't changed

. - Mark Kretschmann On Aug. 13, 2013, 6:41 a.m., Frank Meerkoetter wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112044

Re: Review Request 111991: Better handling of non-writeable UMS collections

On Aug. 10, 2013, 7:26 p.m., Mark Kretschmann wrote: I'm not so sure about the warning that is shown when an unwritable UMS collection is loaded. First of all it seems to me this would belong on a higher level, as other collections could be read-only as well. Second, I'm not sure

Re: Review Request 111991: Better handling of non-writeable UMS collections

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111991/#review37511 --- Ship it! Ship It! - Mark Kretschmann On Aug. 11, 2013, 11

Re: Review Request 111991: Better handling of non-writeable UMS collections

http://git.reviewboard.kde.org/r/111991/#comment27707 As per KDE HIG, please don't use contractions in user visible strings. src/core-impl/collections/umscollection/UmsCollectionLocation.cpp http://git.reviewboard.kde.org/r/111991/#comment27708 Make this const, please. - Mark Kretschmann

Re: Review Request 111991: Better handling of non-writeable UMS collections

failed? - Mark Kretschmann On Aug. 10, 2013, 4:44 p.m., Frank Meerkoetter wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111991

Triggering Copy to Collection from within a Service

function as well, but of course I'd rather do it without duplicating the code. Can I trigger Copy to Collection from within the service in a clean way? -- Mark Kretschmann Amarok Developer Fellow of the Free Software Foundation Europe http://amarok.kde.org - http://fsfe.org

Re: SoK idea

that are online at a given time usually help out the others. So it's not that much of a problem I would think. If you are generally ok with that, we could go ahead with your proposal. -- Mark Kretschmann Amarok Developer Fellow of the Free Software Foundation Europe http://amarok.kde.org - http

  1   2   3   >