Re: Please avoid noisy merge commits in frameworks

2012-02-21 Thread Laszlo Papp
Hi, There's some ideas for such hooks on the Internet already: Yes, there are things flowing around, whereas it would be nice to have a Getting started frameworks contribution page, where the advices (requirements) could be mentioned, like this. Is there already something like that ? Best

Re: Please avoid noisy merge commits in frameworks

2012-02-21 Thread Ben Cooksley
On Feb 20, 2012 7:12 AM, Stephen Kelly steve...@gmail.com wrote: Dario Freddi wrote: 2012/2/19 Stephen Kelly steve...@gmail.com: Stephen Kelly wrote: Hi there, I was reviewing the changes in the frameworks branch from yesterday. Something I noticed was that there are a lot of

Re: Please avoid noisy merge commits in frameworks

2012-02-21 Thread Thomas Zander
söndagen den 19 februari 2012 14.36.07 skrev Stephen Kelly: I was reviewing the changes in the frameworks branch from yesterday. Something I noticed was that there are a lot of merge commits that don't need to exist. Heh, sounds familiar, the same thing was brought up when Qt switched to

Re: libkwindowsystem in staging

2012-02-21 Thread Aaron J. Seigo
On Monday, February 20, 2012 18:14:59 Stephen Kelly wrote: Do you intend to port it to Qt5 or know what that involves? I don't know what that involves yet, but this is something that Martin will be working on shortly. With Qt going window system agnostic, should KDE follow? Yes; however we

kmenubar

2012-02-21 Thread Aaron J. Seigo
hi :) KMenuBar uses KSelectionWatcher in libkdeui. the entire purpose of KMenuBar is to export a top-level menu (aka macOS style). it's only relevant on Linux/Unix and these days we have the DBusMenu specification which is supported by Qt itself. admittedly the DBusMenu spec is more limited

[kdelibs/frameworks] /: Add a porting macro QSKIP_PORTING.

2012-02-21 Thread Stephen Kelly
Git commit 59c089ab7bb187b841b386da2153a2c1699d57c4 by Stephen Kelly. Committed on 21/02/2012 at 18:25. Pushed by skelly into branch 'frameworks'. Add a porting macro QSKIP_PORTING. The QSKIP macro has changed its API in Qt5. The more of these hacks we accumulate, the more sense it will make to

Re: Review Request: make folderview compile with Qt 4.7

2012-02-21 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104030/#review10787 --- Ship it! Ship It! - David Faure On Feb. 20, 2012, 6:16

Re: Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-21 Thread David Faure
On Feb. 20, 2012, 7:16 p.m., David Faure wrote: kparts/htmlextension.h, line 245 http://git.reviewboard.kde.org/r/103973/diff/4/?file=50594#file50594line245 KParts::SettingsInterface is a bit generic for a name (it sounds like something that could apply to all parts), maybe this

Re: Review Request: make folderview compile with Qt 4.7

2012-02-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104030/#review10792 --- This review has been submitted with commit

Re: Review Request: make folderview compile with Qt 4.7

2012-02-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104030/#review10793 --- This review has been submitted with commit

Re: Review Request: Replicate the existing KConfigDialog constructor and change one argument type

2012-02-21 Thread Laszlo Papp
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103716/ --- (Updated Feb. 21, 2012, 3:56 p.m.) Review request for kdelibs and David

Re: Review Request: Replicate the existing KConfigDialog constructor and change one argument type

2012-02-21 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103716/#review10795 --- Ship it! kdeui/dialogs/kconfigdialog.h

Re: Review Request: Replicate the existing KConfigDialog constructor and change one argument type

2012-02-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103716/#review10796 --- This review has been submitted with commit

Re: Review Request: Make KFileWidget provide default filename even when the protocol doesn't support listing

2012-02-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103642/#review10805 --- This review has been submitted with commit

Re: Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-21 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103973/#review10807 --- khtml/khtml_ext.cpp

Re: Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-21 Thread Dawit Alemayehu
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103973/ --- (Updated Feb. 21, 2012, 6:35 p.m.) Review request for kdelibs and David

Bugzilla upgrade.

2012-02-21 Thread Tom Albers
Hi, Our bugzilla instance is an old version. We are preparing to upgrade it to a recent version. This will go in two stages. First we will perform a test upgrade. This will happen on Saturday Februari 25th. We will start at 8PM CET. We expect little inpact, though during the database

Re: Bugzilla upgrade.

2012-02-21 Thread Milian Wolff
On Tuesday 21 February 2012 21:00:33 Tom Albers wrote: Hi, Our bugzilla instance is an old version. We are preparing to upgrade it to a recent version. This will go in two stages. First we will perform a test upgrade. This will happen on Saturday Februari 25th. We will start at 8PM CET.

Re: Bugzilla upgrade.

2012-02-21 Thread Ben Cooksley
On Wed, Feb 22, 2012 at 11:37 AM, Milian Wolff m...@milianw.de wrote: On Tuesday 21 February 2012 21:00:33 Tom Albers wrote: Hi, Our bugzilla instance is an old version. We are preparing to upgrade it to a recent version. This will go in two stages. First we will perform a test upgrade.

Re: Bugzilla upgrade.

2012-02-21 Thread Milian Wolff
On Wednesday 22 February 2012 11:47:36 Ben Cooksley wrote: On Wed, Feb 22, 2012 at 11:37 AM, Milian Wolff m...@milianw.de wrote: On Tuesday 21 February 2012 21:00:33 Tom Albers wrote: Hi, Our bugzilla instance is an old version. We are preparing to upgrade it to a recent version.

Re: Review Request: Write to the correct xmlFile in KToolBar::Private::slotContextShowText()

2012-02-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103812/#review10817 --- This review has been submitted with commit

Re: Review Request: Fix KConfigDialogManager fails to handle subclasses of QComboBox with custom property

2012-02-21 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103909/ --- (Updated Feb. 21, 2012, 11:54 p.m.) Review request for kdelibs, Ben

Re: Review Request: Fix KConfigDialogManager fails to handle subclasses of QComboBox with custom property

2012-02-21 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103909/ --- (Updated Feb. 21, 2012, 11:55 p.m.) Review request for kdelibs, Ben

Re: Access indexing

2012-02-21 Thread T.C. Hollingsworth
On Tue, Feb 21, 2012 at 5:57 AM, todd rme toddrme2...@gmail.com wrote: There is already a krunner plugin that provides a permanent input field on the desktop or a panel.  However, it is not available from standard KDE packages, it is only on kde-look.org (although some distros do offer

Re: Bugzilla upgrade.

2012-02-21 Thread Ben Cooksley
On Wed, Feb 22, 2012 at 11:37 AM, Milian Wolff m...@milianw.de wrote: On Tuesday 21 February 2012 21:00:33 Tom Albers wrote: Hi, Our bugzilla instance is an old version. We are preparing to upgrade it to a recent version. This will go in two stages. First we will perform a test upgrade.

Re: Bugzilla upgrade.

2012-02-21 Thread Milian Wolff
On Wednesday 22 February 2012 11:47:36 Ben Cooksley wrote: On Wed, Feb 22, 2012 at 11:37 AM, Milian Wolff m...@milianw.de wrote: On Tuesday 21 February 2012 21:00:33 Tom Albers wrote: Hi, Our bugzilla instance is an old version. We are preparing to upgrade it to a recent version.