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

Jump to bottom

allowed_media_dirs Wildcard to match / #37

Open
z33ky opened this issue Dec 9, 2013 · 2 comments
Open

allowed_media_dirs Wildcard to match / #37

z33ky opened this issue Dec 9, 2013 · 2 comments
Labels
enhancement need info request
Milestone
waiting

Comments

@z33ky
Copy link

z33ky commented Dec 9, 2013

I would like to request a Wildcard for allowed_media_dirs that also matches /.
The zsh shell has */ for this, so I would suggest either that or just **.
This was referenced May 23, 2014
Closed
Open
@IgnorantGuru IgnorantGuru added this to the limbo milestone Apr 8, 2015
@IgnorantGuru IgnorantGuru modified the milestones: 0.4.4, limbo Apr 20, 2015
@IgnorantGuru
Copy link
Owner

One proposed solution for this is being discussed in #51 - input and testing welcome.

IgnorantGuru pushed a commit that referenced this issue Apr 28, 2015
support git-style /** suffix for some udevil.conf patterns #37
cb33c54 
allowed_files forbidden_files allowed_media_dirs patterns accept a /** suffix
to indicate recursive match, if no other wildcards are used.  ** is not
accepted elsewhere.
@IgnorantGuru
Copy link
Owner

Pending resolution of #51, I have added minimal support for this in the next branch cb33c54. This accepts a /** suffix (only) for the allowed_files, forbidden_files, and allowed_media_dirs patterns. No other wildcard may be used in the pattern when this suffix is used. No other use of ** (or */) is accepted.

See the updated comments in udevil.conf for usage.

I don't really understand the need for this in allowed_media_dirs - should be used with caution if at all. Perhaps you should clarify a use case for wildcards matching /. I don't think git's full flexibility is needed here.

Testing of the next branch on this is appreciated - please see BUILD NEXT in README for instructions.

@IgnorantGuru IgnorantGuru modified the milestones: waiting, 0.4.4 Apr 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement need info request
Projects
None yet
Milestone
waiting
Development

No branches or pull requests
2 participants
@z33ky @IgnorantGuru