Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • Users
  • Groups
  • Search
  • Get Qt Extensions
  • Unsolved
Collapse
Brand Logo
  1. Home
  2. Qt Development
  3. General and Desktop
  4. QFileSystemWatcher's signal "directoryChanged" not emitted
Forum Updated to NodeBB v4.3 + New Features

QFileSystemWatcher's signal "directoryChanged" not emitted

Scheduled Pinned Locked Moved Unsolved General and Desktop
qfilesystemwatcwindows10
43 Posts 5 Posters 18.5k Views 2 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • SGaistS SGaist

    Hi,

    What bug is it ?

    S Offline
    S Offline
    stephane78
    wrote on last edited by
    #5

    @SGaist I have reported this bug :https://bugreports.qt.io/browse/QTBUG-59616
    (and my older bug on QfileSystemWatcher is 52226)

    regards

    1 Reply Last reply
    0
    • SGaistS Offline
      SGaistS Offline
      SGaist
      Lifetime Qt Champion
      wrote on last edited by
      #6

      Like suggested in both report, please provide a minimal compilable example.

      For it has two purposes: see what code triggers the problem and allows to build the example on other platforms.

      Interested in AI ? www.idiap.ch
      Please read the Qt Code of Conduct - https://forum.qt.io/topic/113070/qt-code-of-conduct

      S 2 Replies Last reply
      1
      • SGaistS SGaist

        Like suggested in both report, please provide a minimal compilable example.

        For it has two purposes: see what code triggers the problem and allows to build the example on other platforms.

        S Offline
        S Offline
        stephane78
        wrote on last edited by
        #7

        @SGaist I have now uploaded a minimal project to test and display directorychange and filechange but it is not complete because it does'nt add the paths to watcher when adding new path....
        we use Qt -LGPL in our little company and (at home for personal projects)
        if our company buy a commercial license for three months, will be the two bugs be corrected ?

        1 Reply Last reply
        0
        • SGaistS SGaist

          Like suggested in both report, please provide a minimal compilable example.

          For it has two purposes: see what code triggers the problem and allows to build the example on other platforms.

          S Offline
          S Offline
          stephane78
          wrote on last edited by
          #8

          @SGaist if this project is not sufficient I will perhaps provide the whole code but this code belongs to my company......
          Buying a commercial license will perhaps help us to solve the two bugs?

          1 Reply Last reply
          0
          • SGaistS Offline
            SGaistS Offline
            SGaist
            Lifetime Qt Champion
            wrote on last edited by
            #9

            Paying customer bug will have more weight however I don't know how fast it will make things. I'm not working for the Qt Company and can't comment on such matter.

            Interested in AI ? www.idiap.ch
            Please read the Qt Code of Conduct - https://forum.qt.io/topic/113070/qt-code-of-conduct

            1 Reply Last reply
            1
            • VRoninV Offline
              VRoninV Offline
              VRonin
              wrote on last edited by VRonin
              #10

              Might be worth you looking directly at the source code, see if you can get it sorted yourself, and submit a patch.

              Did you try if other environments are affected too? Any linux distribution for example?

              "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
              ~Napoleon Bonaparte

              On a crusade to banish setIndexWidget() from the holy land of Qt

              S 7 Replies Last reply
              1
              • VRoninV VRonin

                Might be worth you looking directly at the source code, see if you can get it sorted yourself, and submit a patch.

                Did you try if other environments are affected too? Any linux distribution for example?

                S Offline
                S Offline
                stephane78
                wrote on last edited by stephane78
                #11

                @VRonin I didn't try all the OS, but I can later, but under linux there isn't any difference between del and shift+del like under windows (because I tested that with del under win10 so when you put the folder in the trashcan, but did'nt test really with shift+del...I think with shift+del under win7 it is ok but not sure 100%
                I will do more test so....
                but I have already looked at the code of QFile SystemWatcher for one year because of the older bug and I don't really understand it....if I could understand it I would have made myself the correction.I will try again to look at the code of QFileSystemWatcher

                1 Reply Last reply
                0
                • VRoninV VRonin

                  Might be worth you looking directly at the source code, see if you can get it sorted yourself, and submit a patch.

                  Did you try if other environments are affected too? Any linux distribution for example?

                  S Offline
                  S Offline
                  stephane78
                  wrote on last edited by
                  #12

                  @VRonin OK there is the same problem under linux debian 8 with kde.when you move to thrash a subdirectory watched the signal directorychanged is not emitted and when you delete it with shift+del the signal is emitted...I have just tested now
                  under windows 7 I can only delete with shift+del it doesn't allow me to delete with del so there isn't any problem
                  I didn't test under mac os

                  1 Reply Last reply
                  0
                  • VRoninV VRonin

                    Might be worth you looking directly at the source code, see if you can get it sorted yourself, and submit a patch.

                    Did you try if other environments are affected too? Any linux distribution for example?

                    S Offline
                    S Offline
                    stephane78
                    wrote on last edited by stephane78
                    #13

                    @VRonin I have looked quickly at the source code of QFileSystemWatcher....(Qt5.6.2)
                    first I don't find the definition of init() and initPollerEngine() of the class QFileSystemWatcherPrivate (qfilesystemwatcher_p.h) that are called in the constructor of QFileSystemWatcher and in the method addPaths.
                    second, I have seen that there is a thread that derivate from QThread for windows, QWindowsFileSystemWatcherEngineThread, and the method run is redefinited l332 in qfilesystemwatcher_win.cpp,so that I understand it but don't understand all the details in the method run...

                    1 Reply Last reply
                    0
                    • VRoninV VRonin

                      Might be worth you looking directly at the source code, see if you can get it sorted yourself, and submit a patch.

                      Did you try if other environments are affected too? Any linux distribution for example?

                      S Offline
                      S Offline
                      stephane78
                      wrote on last edited by
                      #14

                      @VRonin I have seen where it emits the signal directoryChanged when the directory is deleted l.388 and l.385 I see if(fakeRemove||!fileInfo.exists()) but I don't understand the other things well.....

                      1 Reply Last reply
                      0
                      • VRoninV VRonin

                        Might be worth you looking directly at the source code, see if you can get it sorted yourself, and submit a patch.

                        Did you try if other environments are affected too? Any linux distribution for example?

                        S Offline
                        S Offline
                        stephane78
                        wrote on last edited by
                        #15

                        @VRonin I have looked too at the win32 functions findfirstchangenotification (used l.160) and find nextchangenotification (used l.364)....I think I must use a debug version of Qt to debug that because perhaps there is no change notification on the handle when we use a simple del to delete a directory ie send it to trashcan(without shift) ?

                        1 Reply Last reply
                        0
                        • VRoninV VRonin

                          Might be worth you looking directly at the source code, see if you can get it sorted yourself, and submit a patch.

                          Did you try if other environments are affected too? Any linux distribution for example?

                          S Offline
                          S Offline
                          stephane78
                          wrote on last edited by
                          #16

                          @VRonin or perhaps the error l.367 is another error than ERROR_ACCESS_DENIED that should be considered too and the that fakeRemove should be put to true too (or another boolean )

                          1 Reply Last reply
                          0
                          • VRoninV VRonin

                            Might be worth you looking directly at the source code, see if you can get it sorted yourself, and submit a patch.

                            Did you try if other environments are affected too? Any linux distribution for example?

                            S Offline
                            S Offline
                            stephane78
                            wrote on last edited by stephane78
                            #17

                            @VRonin what are the other errors of FindNextChangeNotification?didn't find them in the win32 msdn documentation.
                            And I don't really know how to do debug of Qt ? because it is in QtCore,so I suppose I must take the project and source of QtCore, but my test project use not only QtCore but QtGui and QtWidget too....

                            1 Reply Last reply
                            0
                            • VRoninV Offline
                              VRoninV Offline
                              VRonin
                              wrote on last edited by
                              #18

                              I'll take a quick look and see if I can spot something in the source, no promises though

                              "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
                              ~Napoleon Bonaparte

                              On a crusade to banish setIndexWidget() from the holy land of Qt

                              S 2 Replies Last reply
                              0
                              • VRoninV VRonin

                                I'll take a quick look and see if I can spot something in the source, no promises though

                                S Offline
                                S Offline
                                stephane78
                                wrote on last edited by
                                #19

                                @VRonin Thank you vronin, do you know how I can debug QtCore (or corelib or io) with my test project synchro as executable with QtCreator and mingw-w64(the compiler of qt.io).I suppose I must take the source (single,the .zip) but don't really know how to do that.my project synchro uses core,gui and widget but I suppose I must open corelib or io as project in QtCreator

                                1 Reply Last reply
                                0
                                • VRoninV VRonin

                                  I'll take a quick look and see if I can spot something in the source, no promises though

                                  S Offline
                                  S Offline
                                  stephane78
                                  wrote on last edited by stephane78
                                  #20

                                  @VRonin do you know how to get the version 5.6.2 of qtbase from code.qt.io with git? (to build it in debug and try to debug corelib)

                                  1 Reply Last reply
                                  0
                                  • VRoninV Offline
                                    VRoninV Offline
                                    VRonin
                                    wrote on last edited by
                                    #21

                                    Just clone the repository and check out the 5.6 branch

                                    "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
                                    ~Napoleon Bonaparte

                                    On a crusade to banish setIndexWidget() from the holy land of Qt

                                    S 6 Replies Last reply
                                    1
                                    • VRoninV VRonin

                                      Just clone the repository and check out the 5.6 branch

                                      S Offline
                                      S Offline
                                      stephane78
                                      wrote on last edited by
                                      #22

                                      @VRonin ok thanks vronin...

                                      1 Reply Last reply
                                      0
                                      • VRoninV VRonin

                                        Just clone the repository and check out the 5.6 branch

                                        S Offline
                                        S Offline
                                        stephane78
                                        wrote on last edited by
                                        #23

                                        @VRonin it is done.but to debug qfilesystemwatcher should I open corelib.pro in QtCreator (QtCore) and build it debug and put breakpoints and attach to my app synchro ?

                                        1 Reply Last reply
                                        0
                                        • VRoninV VRonin

                                          Just clone the repository and check out the 5.6 branch

                                          S Offline
                                          S Offline
                                          stephane78
                                          wrote on last edited by stephane78
                                          #24

                                          @VRonin I want to put a breakpoint l.364 of QFileSystemWatcher_win.cpp and watch error l365

                                          1 Reply Last reply
                                          0

                                          • Login

                                          • Login or register to search.
                                          • First post
                                            Last post
                                          0
                                          • Categories
                                          • Recent
                                          • Tags
                                          • Popular
                                          • Users
                                          • Groups
                                          • Search
                                          • Get Qt Extensions
                                          • Unsolved