Can i still merge this, small KDirWatch improvement.

[Jos Poortvliet]

On Wed, Aug 7, 2013 at 7:37 PM, Albert Astals Cid <aacid at kde.org> wrote:
> El Dimecres, 7 d'agost de 2013, a les 13:13:36, Mark va escriure:
>> On Wed, Aug 7, 2013 at 10:38 AM, Albert Astals Cid <aacid at kde.org> wrote:
>> >> Hi Release team,
>> >
>> > Hi
>> >
>> >> I just got the OK to ship [1] but considering the very close release
>> >> of KDE 4.11 it seems right to ask the release team for permission as
>> >> well.
>> >>
>> >> The patch simply prevents one stat call with KDirWatch and makes the
>> >> code a little bit better to understand. Tests pass just fine and othe
>> >> additional (visual and gdb) testing hasn't shown any regression.
>> >>
>> >> If i am not allowed to push it (due to the very close 4.11 tag), when
>> >> can i push it then? Right after the tag i assume?
>> >
>> > Honestly, if this is just a [small] speed optimization and doesn't fix any
>> > bug, i don't see it belonging to 4.11 at all, just commit it to master
>> > and leave it for 4.12.
>> >
>> > Unless is a so huge optimization our users are going to send us money for
>> > the speed difference :D
>> >
>> > Cheers,
>> >
>> >   Albert
>>
>> Hi Albert,
>>
>> I'm OK with this not going into 4.11.0 since that is just a too short
>> notice. But i do like to get this in the next bugfix release (4.11.1). It
>> might not be a strict "bugfix", but that's only because nobody ever
>> opened a bug report for it.
>> Specially because 4.11 is LTS! Users and likely some company's are
>> going to be running that version for years to come. They should have
>> this fix.
>
> Let me repease, 4.11 is LTS *only for kde-workspace*. This patch is for
> kdelibs, which will have a regular 4.12 release, so they'll get this
> optimization in 5 months.

Just to check, distro's using the 4.11 LTS for workspaces are likely,
you think, to upgrade the KDElibs to 4.12 underneath it so they will
get this patch? If not, the argument that there'll be a 4.12 doesn't
really have much value... And, as KDELibs is so frozen-y, perhaps it
should be LTS too, instead of doing a 4.12 release...

> Cheers,
>   Albert
>
>>
>> No, it won't get you or KDE any money, but it does save one stat call
>> in a very common code path (creating files) thus you could argue that
>> this patch will extend the life of storage devices ;-)
>>
>> >> Kind regards,
>> >> Mark
>> >>
>> >> [1] https://git.reviewboard.kde.org/r/111870/
>>
>> _______________________________________________
>> release-team mailing list
>> release-team at kde.org
>> https://mail.kde.org/mailman/listinfo/release-team
>
> _______________________________________________
> release-team mailing list
> release-team at kde.org
> https://mail.kde.org/mailman/listinfo/release-team