[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