D28745: Skip caching thumbnails on encrypted filesystems

2020-08-19 Thread David Faure
dfaure requested changes to this revision.
dfaure added inline comments.
This revision now requires changes to proceed.


> marcingu wrote in thumbnail.cpp:776
> No. The m_thumbnailDirDeviceId is set to 0 and only changed if lstat executes 
> properly. Then it takes value of st_dev. I don't know if st_dev can be 0 
> though and I couldn't find this information in manual either. If it can, we 
> should change it.

I don't see how stat would succeed and st_dev would be 0, but I could be wrong. 
Maybe assert that it's not 0, at least?

> thumbnail.cpp:779
> +if (lstat(QFile::encodeName(m_thumbBasePath).data(), ) == 
> -1) {
> +qCWarning(KIO_THUMBNAIL_LOG) << "Cannot read information about 
> filesystem";
> +return false;

print out the path in the warning (same on line 786)

> thumbnail.cpp:785
> +struct stat fileStat;
> +if(lstat(QFile::encodeName(path).data(), ) == -1) {
> +qCWarning(KIO_THUMBNAIL_LOG) << "Cannot read information about 
> filesystem";

missing space after `if`


> thumbnail.h:93
>  qint64 m_maxFileSize;
> +dev_t m_thumbnailDirDeviceId = 0;
>  };

This will break compilation on Windows, I assume?

On Unix, is the corresponding include present in this file? The lack of context 
makes reviewing difficult indeed. Any chance to move this to gitlab? (see 

  R320 KIO Extras


To: marcingu, ivan, broulik, #dolphin, ngraham, meven, bruns, dfaure
Cc: dfaure, thiago, bruns, meven, ngraham, kde-frameworks-devel, kfm-devel, 
waitquietly, azyx, nikolaik, pberestov, iasensio, aprcela, fprice, LeGast00n, 
cblack, fbampaloukas, alexde, Codezela, feverfew, michaelh, spoorun, 
navarromorales, firef, andrebarros, emmanuelp, rdieter, mikesomov

Syntax-Highlighter questions

2020-08-19 Thread Andreas Müller

a while back I needed syntax-highlighting in a QML based application.
Since I could not find some QML support for syntax-highlighter, I
wrote a wrapper library [1]. That worked perfectly fine but now that I
wrote an example (screenshot in [1]) unexpected behaviour pops up. The
example is different because it can set highlighter's definition and
theme during runtime which were set once statically in the initial

Now my issue/question:
Highlight definition: Although SyntaxHighlighter::setDefinition calls
rehighlight(), the content of the text field remains unchanged. By
changing the size of the window, the new highlighting is drawn. Do you
have any suggestions what can be done to fix that?

Hoping for cure :) and thanks in advance,


[1] https://github.com/schnitzeltony/ksyntax-highlighting-wrapper

Re: KIO on Android Failure

2020-08-19 Thread David Faure
On mardi 18 août 2020 13:52:50 CEST Ben Cooksley wrote:
> Hi all,
> At some point recently functionality was added to KIO which broke the
> build on Android.

Is there any reason why Android doesn't appear at

This is the reason why I didn't notice this problem.

David Faure, fa...@kde.org, http://www.davidfaure.fr
Working on KDE Frameworks 5

KDE CI: Frameworks » kio » kf5-qt5 FreeBSDQt5.15 - Build # 119 - Unstable!

2020-08-19 Thread CI System
 Build URL
kf5-qt5 FreeBSDQt5.15
 Date of build:
Wed, 19 Aug 2020 11:14:59 +
 Build duration:
10 min and counting
   JUnit Tests
  Name: projectroot Failed: 1 test(s), Passed: 54 test(s), Skipped: 0 test(s), Total: 55 test(s)Failed: projectroot.autotests.applicationlauncherjob_forkingtestName: projectroot.autotests Failed: 0 test(s), Passed: 6 test(s), Skipped: 0 test(s), Total: 6 test(s)Name: projectroot.src.ioslaves.trash Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)Name: projectroot.src.kpasswdserver Failed: 0 test(s), Passed: 1 test(s), Skipped: 0 test(s), Total: 1 test(s)

Re: KIO on Android Failure

2020-08-19 Thread Ben Cooksley
On Wed, Aug 19, 2020 at 3:01 AM Ahmad Samir  wrote:
> On 18/08/2020 13:52, Ben Cooksley wrote:
> > Hi all,
> >
> > At some point recently functionality was added to KIO which broke the
> > build on Android.
> >
> > I'm not sure why we're building KIO on Android, but it appears that
> > some applications may be using it - and this in turn causes the
> > Dependency Build jobs to fail.
> >
> > Could someone take a look please?
> > https://build.kde.org/job/Administration/job/Docker%20Generate%20AndroidQt5.14%20KF5-SDK/lastFailedBuild/console
> >
> > Thanks,
> > Ben
> >
> It looks like it's caused by systemd integration in kprocessrunner; also it 
> seems Q_OS_UNIX includes
> Android systems, so we need to guard it with !defind(Q_OS_ANDROID). Should 
> hopefully be fixed by
> https://invent.kde.org/frameworks/kio/-/merge_requests/107 .

Many thanks for quickly addressing this Ahmad, it's appreciated.


> --
> Ahmad Samir