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

Already on GitHub? Sign in to your account

path.local should match paths properly #41

Open
pietia opened this Issue Feb 19, 2014 · 5 comments

Comments

Projects
None yet
5 participants

pietia commented Feb 19, 2014

For example:

    path:
        local: "dist/"

matches:

[ + ] File uploaded: client/bower_components/spin.js/dist/spin.js:
[ + ] File uploaded: client/bower_components/spin.js/dist/spin.min.js:
[ + ] File uploaded: dist/_font_init.html:
[ + ] File uploaded: dist/_footer.html:

but it should be only:

[ + ] File uploaded: dist/_font_init.html:
[ + ] File uploaded: dist/_footer.html:

Using "./dist" doesn't help with it.

Owner

lucasmotta commented Feb 20, 2014

I've noticed that a few people are having troubles with the paths, I think it may be something related to the way I am using minimatch.
I will have a look to fix this issues. Meanwhile you could add the path client/bower_components to the exclude option.

@lucasmotta lucasmotta added duplicate bug and removed duplicate labels Feb 20, 2014

Just wanted to let you know I'm unable to specify a local path. Here is what I'm using:

live:
    scheme: "sftp"
    host: "XXX.XXX.XXX.XXX"
    port: "22"
    user: "myusername"
    check: "true"
    privateKey: "~/.ssh/myprivatekey"
    branch: "master"
    path:
        local: "dist/"
        remote: "/var/www/example.com/httpdocs"

When I'm doing dploy live, I can only upload the .rev file...

juuuuuu commented Nov 19, 2014

I have the same problem.

   path:
       local: "test/"

matches:

[ + ] File uploaded: test/test.txt
[ + ] File uploaded: web/modules/simpletest/files/README.txt
[ + ] File uploaded: web/sites/all/modules/contrib/linkit/test/linkit.test
[ + ] File uploaded: web/sites/all/modules/contrib/features/tests/features_test/features_test.features.inc
...etc
Owner

lucasmotta commented Nov 19, 2014

hey @juuuuuu, can you for now try adding this to your "exclude" parameter:

exclude: ["web/**"]

let me know if that fixes your issue for now.

Same here:

path:
    local: "dist/"

matches :

[ + ] File uploaded: build/bower_components/whatever/dist/whatever.js:

I resolved it adding an exclude parameter, but I think it would be more effective if dploy runs only on the specified folder.

@lucasmotta lucasmotta added this to the version 2 milestone Jun 18, 2015

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