Comments (13)
Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).
Weird. I'll look into that.
Thanks!
from efsw.
Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).
I couldn't reproduce the bug, i've seen that you forked the repository and made some commits, it looks interesting, this is fixed in your fork?
I'm interested in merging your changes, the only change that i don't understand is the sys/inotify.h
file added, i assume that it's because you're not using the fixed premake file ( see Issue #40 ) or the CMake file.
I'm really busy this month, but i'll keep an eye on your work.
Thanks for contributing for the library, i'm really happy to see some collaboration.
from efsw.
Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).
I'll close this issue because no response was given. But i'm still interested in knowing how to reproduce it. Mihail if you're still there please let me know.
Thanks
from efsw.
Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).
Fixed issue #48 in develop.
from efsw.
Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).
Fixed issue #48.
from efsw.
Original comment by Mihail Slobodyanuk (Bitbucket: mihail_slobodyanuk, ).
I am sorry. I missed post from 2015-08-19.
I will check the issue again in next couple days.
About fork. I wanted finish QA then made merge request but the process delayed.
Today i pushed a couple additional fixes. We can discuss details then merge
from efsw.
Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).
Hi! Welcome back :)
I already have a branch with some of your changes incorporated but i used my own naming conventions, and i added made a couple of new changes.
The part i did not add are the changes in WatcherWin32
, since at least for me doesn't seem to be working fine, i get intinite number of repeated messages for some reason, i did not look in detail your implementation, and i don't understand why you did those changes in a first place. Can you explain me what's the idea?
The new changes looks good, i'll add them later, except for the inotify.h
header that i don't understand why is there.
Thank you very much for your collaboration.
Regards
from efsw.
Original comment by Mihail Slobodyanuk (Bitbucket: mihail_slobodyanuk, ).
Hello, Martin!
Can you explain me what's the idea?
Are you meant https://bitbucket.org/mihail_slobodyanuk/efsw/commits/782d3f7f4a08c7015acd3f9f0b21e0145e1ec479 ?
My app create FileWatcher instance in one thread and destroys it in another thread. CancelIo() WinAPI require to be called in same thread as ReadDirectoryChangesW(). CancelIoEx() may help us but that one offered only since Windows Vista/Windows Server 2008 API. So i moved CancelIo() to same with ReadDirectoryChangesW() thread. All another changes in that commit are needed to proper thread synchronization.
... except for the inotify.h header that i don't understand why is there.
I moved inotify.h to another folder and include it via CFLAGS in outer make script. Yes i don't using premake every assembly. The main reason why i need own inotify.h is very outdated toolchains for some NAS. The linux kernel on that devices has inotify support, but GLIBC are not offer wrappers. I similar way the problem resolved in inotify-tools project (see inotify-nosys.h and system header detection in configure.ac )
Thanks. Mihail
from efsw.
Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).
My app create FileWatcher instance in one thread and destroys it in another thread. CancelIo() WinAPI require to be called in same thread as ReadDirectoryChangesW(). CancelIoEx() may help us but that one offered only since Windows Vista/Windows Server 2008 API. So i moved CancelIo() to same with ReadDirectoryChangesW() thread. All another changes in that commit are needed to proper thread synchronization.
Oh, now i get what you're saying. The problem is that there's already a patch for that problem, it was introduced later than your branch, i decided to basically drop Windows XP / 2000 support and just used CancelIoEx
. I really don't see the point to support XP this days, by i know this is something that can be discussed, i just didn't want to go into the problem into implementing what you did. I would use your version if i find it stable so i can reintroduce Windows XP support, but with the bug i mentioned i can't, anyways i don't care much about supporting an old OS. Here's the fix: https://bitbucket.org/SpartanJ/efsw/commits/eb15d49e5c51
I moved inotify.h to another folder and include it via CFLAGS in outer make script. Yes i don't using premake every assembly. The main reason why i need own inotify.h is very outdated toolchains for some NAS. The linux kernel on that devices has inotify support, but GLIBC are not offer wrappers. I similar way the problem resolved in inotify-tools project (see inotify-nosys.h and system header detection in configure.ac )
Ok, inotify-nosys.h
is already in the src/efsw/
subfolder. As i understand you wan't that file in another subfolder. I could just move it if that's inconvenient for you. The current implementation works like this:
When the make file is being created by premake4, inotify.h
will be searched in the system, if is not found in the system, EFSW_INOTIFY_NOSYS
will be added as a define to the file compilation. You should be able to do that in your configure.ac
.
Thanks!
from efsw.
Original comment by Mihail Slobodyanuk (Bitbucket: mihail_slobodyanuk, ).
Hi!
I had a little time and made test for the subj:
#!c++
#include <efsw/efsw.hpp>
#include <efsw/Thread.hpp>
#include <iostream>
#include <Windows.h>
using namespace std;
/// Processes a file action
class UpdateListener : public efsw::FileWatchListener {
public:
UpdateListener() {
}
std::string getActionName(efsw::Action action) {
switch (action) {
case efsw::Actions::Add: return "Add";
case efsw::Actions::Modified: return "Modified";
case efsw::Actions::Delete: return "Delete";
case efsw::Actions::Moved: return "Moved";
default: return "Bad Action";
}
}
void handleFileAction(efsw::WatchID watchid, const std::string& dir, const std::string& filename, efsw::Action action, std::string oldFilename = "") {
std::cout << "DIR (" << dir + ") FILE (" + (oldFilename.empty() ? "" : "from file " + oldFilename + " to ") + filename + ") has event " << getActionName(action) << std::endl;
}
};
string path = "C:\\efsw_test";
string file = path + "\\file";
void thread_proc() {
CreateDirectory (path.c_str(), NULL);
FILE* fd = fopen(file.c_str(), "w");
char x = 'x';
fwrite(&x, 1, 1, fd);
fclose(fd);
Sleep(2000);
string cmd = "rmdir /s /q ";
cmd+=path;
system(cmd.c_str());
}
int main(int argc, char* argv[]) {
UpdateListener * ul = new UpdateListener();
efsw::Thread th(thread_proc);
th.launch();
efsw::FileWatcher * fileWatcher(new efsw::FileWatcher(true));
efsw::WatchID err;
if ((err = fileWatcher->addWatch(path, ul, true)) > 0) {
fileWatcher->watch();
std::cout << "Watching directory: " << path.c_str() << std::endl;
} else {
std::cout << "Error trying to watch directory: " << path.c_str() << std::endl;
return 1;
}
Sleep(600*1000);
delete fileWatcher;
return 0;
}
It seems the bug still there. Here is my output:
Watching directory: C:\efsw_test
DIR (C:\efsw_test\) FILE (file) has event Modified
DIR (C:\efsw_test\) FILE (file) has event Delete
DIR (C:\efsw_test\) FILE (file) has event Delete
DIR (C:\efsw_test\) FILE (file) has event Delete
DIR (C:\efsw_test\) FILE (file) has event Delete
DIR (C:\efsw_test\) FILE (file) has event Delete
DIR (C:\efsw_test\) FILE (file) has event Delete
DIR (C:\efsw_test\) FILE (file) has event Delete
DIR (C:\efsw_test\) FILE (file) has event Delete
from efsw.
Original comment by Mihail Slobodyanuk (Bitbucket: mihail_slobodyanuk, ).
Reproduced
from efsw.
Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).
I ran your test in default and develop branches and in both is working fine.
Are you sure you have the repo updated?
Here's my complete output for both cases:
Watching directory: C:\efsw_test
DIR (C:\efsw_test\) FILE (file) has event Delete
from efsw.
Original comment by Mihail Slobodyanuk (Bitbucket: mihail_slobodyanuk, ).
Sorry. My bad somewhere. I re-updated repo and bug was gone.
Thank you!
from efsw.
Related Issues (20)
- Cannot track symlinked file in Linux HOT 10
- Bazel Building HOT 1
- On MacOS, actions happening shortly after starting watcher are not reported HOT 19
- CMake multi config issue HOT 2
- Support Windows long path aware HOT 4
- Check Return Code and Allow Buffer Customization in `ReadDirectoryChangesW()` Function HOT 1
- Paranoid exit code? HOT 2
- MacOS build on server HOT 1
- [Bug] Crash while thread running after FileWatcherInotfiy destructed. HOT 16
- Premake5 macosx after gmake2 ... [CI Server] HOT 1
- Last modified file remembered and failing to fire event under Windows HOT 5
- Premake links in ReadMe is dead HOT 1
- FSEventStreamScheduleWithRunLoop has been deprecated in MacOS 13 HOT 3
- Nested template syntax breaks LEGACY builds HOT 3
- Generic file watcher doesn't report new directories on network drives HOT 2
- memory leak doubt and thread conflict HOT 2
- Cross building on macOS may cause `EFSW_FSEVENTS_NOT_SUPPORTED` incorrectly defined HOT 2
- Incorrect move events reported for simultaneous file moves across monitored subdirectories on Windows HOT 9
- Warning C4834 in Log.cpp
- [Bug] Rename event is misinterpreted as two add events on macOS fsevent when changing case only HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from efsw.