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

Consecutive separator behaviour #46

Open
erikgeiser opened this issue Nov 17, 2020 · 0 comments
Open

Consecutive separator behaviour #46

erikgeiser opened this issue Nov 17, 2020 · 0 comments

Comments

@erikgeiser
Copy link

erikgeiser commented Nov 17, 2020

As was already discussed in #20, patterns containing **/* may lead to unexpected results for most users. After porting https://github.com/goreleaser/nfpm to glob, this was reported as a regression (goreleaser/nfpm#256).

Currently, the pattern test/**/* does not match test/file but it does match test//file (https://play.golang.org/p/geU9afuijny). While this is technically correct, it is not what users expect, since test/file and test//file are identical in the context of file systems.

This situation could be solved by modifying the AST. This could also be added as an option using the pattern described in #45.

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

No branches or pull requests

1 participant