Navigation Menu

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

--exclude not working #117

Closed
andrewwatson opened this issue Nov 11, 2013 · 5 comments
Closed

--exclude not working #117

andrewwatson opened this issue Nov 11, 2013 · 5 comments

Comments

@andrewwatson
Copy link

./bin/phpmd  library/ text tests/settings/phpmd.xml --exclude=library/vendor/* --minimumpriority=5

it's ignoring my command line options completely! the documentation says to use --ignore but when I use that it complains and says that is deprecated.

@acrobat
Copy link

acrobat commented Dec 9, 2013

@andrewwatson i had the same problem, but manu on the irc helped me.

You sould do this (exclude with the =)

./bin/phpmd  library/ text tests/settings/phpmd.xml --exclude library/vendor/*

instead of

./bin/phpmd  library/ text tests/settings/phpmd.xml --exclude=library/vendor/* 

@ghost
Copy link

ghost commented Mar 7, 2014

@acrobat Problem IMHO is that phpmd does not work correctly with relative paths for exclusions.

@reubenavery
Copy link

The documentation states that --exclude is a "comma-separated string of patterns that are used to ignore directories", but the patterns I'm trying (regex) don't seem to have any effect.

The only way I could get this --exclude to work is by passing a complete path, and no asterisk wildcards, (much less regex).

This is definitely suboptimal. Definite feature request here.

A related pull request: #100

@manuelpichler
Copy link
Contributor

@see #100

@piotr-cz
Copy link

In v1.5 on windows --exclude seems to work only with backslashes (doesn't matter if path is absolute or relative to).

Can't say about version 2.x (I'm tied to 1.x because can't update pdepend dependency becase of bug in 2).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

5 participants