Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The --ignore parameter does not take effect #654

Open
Vzz1c opened this issue Sep 14, 2023 · 9 comments
Open

The --ignore parameter does not take effect #654

Vzz1c opened this issue Sep 14, 2023 · 9 comments
Labels
bug Something's not right! need info More information needed before this can proceed

Comments

@Vzz1c
Copy link

Vzz1c commented Sep 14, 2023

  • Watchexec's version 1.23
  • The OS you're using windows 10 latest
    watchexec -i "target/**" make test
    Invalid filtering, still monitors files in the target directory for changes.
@Vzz1c Vzz1c added the bug Something's not right! label Sep 14, 2023
@passcod passcod added this to the Ignore handling issues milestone Sep 14, 2023
@passcod
Copy link
Member

passcod commented Sep 14, 2023

Try **/target/**

@Vzz1c
Copy link
Author

Vzz1c commented Sep 14, 2023

尝试**/target/**

Try **/target/**

I've tried everything. It doesn't work.

@Vzz1c
Copy link
Author

Vzz1c commented Sep 14, 2023

Try **/target/**

Also, on Win11, it doesn't work at all, typing watchexec produces nothing.

@passcod
Copy link
Member

passcod commented Sep 14, 2023

typing watchexec produces nothing.

well, i'm gonna need a log file for that

@passcod passcod added the need info More information needed before this can proceed label Sep 14, 2023
@tminich
Copy link

tminich commented Nov 16, 2023

Fairly sure I'm having the same issue.

Exclude file:

**/.idea
**/.idea/**/*
**/.git
**/.git/**/*
**/node_modules
**/node_modules/**/*

Log:

{"timestamp":"2023-11-16T10:28:34.439960Z","level":"INFO","fields":{"message":"action constructed","action":"Action { events: [Event { tags: [Source(Filesystem), FileEventKind(Modify(Any)), Path { path: \"C:\\\\devel\\\\laravel-demo\\\\node_modules\\\\caniuse-lite\\\\data\\\\features\\\\mdn-text-decoration-shorthand.js\", file_type: Some(File) }], metadata: {} }], outcome: OnceCell(Uninit) }"},"target":"watchexec::action::worker"}
{"timestamp":"2023-11-16T10:28:34.440936Z","level":"INFO","fields":{"message":"outcome resolved","outcome":"Both(Wait, Start)"},"target":"watchexec::action::worker"}
{"timestamp":"2023-11-16T10:28:35.445059Z","level":"INFO","fields":{"message":"action constructed","action":"Action { events: [Event { tags: [Source(Filesystem), FileEventKind(Modify(Any)), Path { path: \"C:\\\\devel\\\\laravel-demo\\\\node_modules\\\\effect\\\\dist\\\\cjs\\\\Exit.js\", file_type: Some(File) }], metadata: {} }], outcome: OnceCell(Uninit) }"},"target":"watchexec::action::worker"}
{"timestamp":"2023-11-16T10:28:35.446084Z","level":"INFO","fields":{"message":"outcome resolved","outcome":"Both(Wait, Start)"},"target":"watchexec::action::worker"}

Technically these files shouldn't trigger in the first place, because they do not change, but I'm going to blame that part on Windows. In any case the ignore file should have caught these.

@passcod
Copy link
Member

passcod commented Nov 17, 2023

cool, still need a logfile (at debug or trace level)

@tminich
Copy link

tminich commented Nov 17, 2023

Sure. This time I had to provoke it via changing an ignored file as whatever is triggering it doesn't want to do it today (further suggesting that part is Windows' fault).
watchexec.2023-11-17T08-25-15Z.log

@passcod
Copy link
Member

passcod commented Nov 17, 2023

Hmm. I wonder if something changed out from under me and isn't respecting windows paths anymore. Does it change if you do e.g. **node_modules**

@tminich
Copy link

tminich commented Nov 17, 2023

No, unfortunately it does not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something's not right! need info More information needed before this can proceed
Projects
None yet
Development

No branches or pull requests

3 participants