vs-deploy icon indicating copy to clipboard operation
vs-deploy copied to clipboard

"exclude" property on "packages" not working when deploying a folder from context menu

Open iFarmGolems opened this issue 7 years ago • 2 comments

Hey,

I found that when I deploy some folder to SFTP via context menu, the excluded property on package seems not to work.

For example:

"exclude": [
    "**/.*",
    ".editorconfig",
    ".vscode/*",
    "**/CVS"
]

What I see on deploy:

Deploying '/modules/ims/ims_config_viewer/CVS/Entries' to '/var/www/html/sites/all/modules/ims/ims_config_viewer/CVS'... [OK]
Deploying '/modules/ims/ims_config_viewer/CVS/Entries.Extra' to '/var/www/html/sites/all/modules/ims/ims_config_viewer/CVS'... [OK]
Deploying '/modules/ims/ims_config_viewer/CVS/Entries.Extra.Old' to '/var/www/html/sites/all/modules/ims/ims_config_viewer/CVS'... [OK]
Deploying '/modules/ims/ims_config_viewer/CVS/Entries.Old' to '/var/www/html/sites/all/modules/ims/ims_config_viewer/CVS'... [OK]
Deploying '/modules/ims/ims_config_viewer/CVS/Repository' to '/var/www/html/sites/all/modules/ims/ims_config_viewer/CVS'... [OK]
Deploying '/modules/ims/ims_config_viewer/CVS/Root' to '/var/www/html/sites/all/modules/ims/ims_config_viewer/CVS'... [OK]
Deploying '/modules/ims/ims_config_viewer/CVS/Template' to '/var/www/html/sites/all/modules/ims/ims_config_viewer/CVS'... [OK]
Deploying '/modules/ims/ims_config_viewer/ims_config_viewer.info' to '/var/www/html/sites/all/modules/ims/ims_config_viewer'... [OK]
Deploying '/modules/ims/ims_config_viewer/ims_config_viewer.module' to '/var/www/html/sites/all/modules/ims/ims_config_viewer'... [OK]
Finished

iFarmGolems avatar May 11 '17 09:05 iFarmGolems

@iFarmGolems

I am not sure, maybe the 4th argument could be wrong.

Try use **/CVS/**/* instead,

mkloubert avatar May 11 '17 09:05 mkloubert

Still the same.

Although you were right, I had to change my pattern to **/CVS/** to actually not deploy files in CVS folder (tested with deployOnSave=true).

Tested here: http://www.globtester.com/#p=eJzT0tJ3DgvW19ICAAmzAfM%3D&r=eJyzScksU0jOSSwutlXKTSxJzlCy088vKNF3DgvWT80rKao0tNEHKrGzwVRnU1yQmKdQXFKZk2qrlJafV6JbnFmVaqVgaKlnVFBhjW6QkX5FhX5lpY0%2BSJudTVKRHabJUD05%2BckgjMVqYm2FmmCEYRsAMGVQbw%3D%3D&

But the original issue still persists.

iFarmGolems avatar May 11 '17 10:05 iFarmGolems