[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-core-devel
Subject: Re: KIO fixes in kdelibs and kdebase
From: Vlad <vladc6 () yahoo ! com>
Date: 2008-05-09 3:28:13
Message-ID: 719796.75164.qm () web54407 ! mail ! yahoo ! com
[Download RAW message or body]
--- David Faure <faure@kde.org> wrote:
> On Wednesday 07 May 2008, Vlad wrote:
> > respect_copyjob_permissions.diff:
> > - Fix KIO::file_move to respect its permissions parameter instead
> of
> > just ignoring it. KIO::file_copy already respects the
> user-specified
> > permissions.
>
> The chmod should only be done if d->m_permissions is not -1.
> The patches look fine otherwise, as far as I'm concerned.
I've changed the patch to ensure that d->m_permissions is not -1
before performing the chmod.
> A unit test for this change (in tests/jobtest.cpp) would be
> excellent :)
Sure, I'll add a unit test.
> > kdelibs_largefile.diff:
> > - Compile kdelibs with -D_LARGE_FILES instead of
> > -D_LARGEFILE64_SOURCE, and drop the redundant
> -D_LARGEFILE64_SOURCE
> > flag in kio.
>
> I think you meant compile with -D_LARGEFILE64_SOURCE instead of
> -D_LARGE_FILES :)
Correct, I misspoke in the description. The patch removes
-D_LARGE_FILES and adds -D_LARGEFILE64_SOURCE.
Thanks for reviewing the changes. I've committed them as revisions
805652, 805653, 805654, 805655, 805656, and 805658.
Vlad
____________________________________________________________________________________
Be a better friend, newshound, and
know-it-all with Yahoo! Mobile. Try it now. http://mobile.yahoo.com/;_ylt=Ahu06i62sR8HDtDypao8Wcj9tAcJ
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic