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

Syncing all changed files #25

Closed
m4recek opened this issue Aug 19, 2012 · 9 comments
Closed

Syncing all changed files #25

m4recek opened this issue Aug 19, 2012 · 9 comments
Assignees

Comments

@m4recek
Copy link

m4recek commented Aug 19, 2012

Hi, I am using your plugin with LESS-build plugin which is building CSS files from LESS files. So after editing file style.less new file style.css is created by LESS-build plugin. Problem with this is that FTP Sync is syncing only files edited by me (user) not files edited by aditional plugins. So, style.less is uploaded, but style.css is untouched.

Maybe you can check (after save) for changed files all over the project folder and upload them as well?

PS: Maybe some "sync timeout setting" will be good, to let other plugins do their work and build files, and start syncing after 1s for example.

Thanks :)

@NoxArt
Copy link
Owner

NoxArt commented Aug 20, 2012

Hi, I'll take a look at it. Would be optional, but still I wonder about performance ... will test it on 3342 files project. Or you perhaps you might flag only some "watched" paths ... this issue would be pretty much gone then.

The timeout is tricky when the build time vary (might also be unexpected spike - OS may be swapping, running anti-virus, giving priority to other processes etc.) ... maybe 'm wrong, but it doesn't sound as a possible solution to these situations at the moment.

Thank you for idea!

@ghost ghost assigned NoxArt Aug 20, 2012
@m4recek
Copy link
Author

m4recek commented Aug 20, 2012

Thank for your answer, I am looking forward to it :)

I was thinking same about sync timeout, but if I know that usually it takes 2-3s to build. I can setup timeout for 5s. In most cases it will be enough and I can deal with others :)
Second option which came to my mind was oportunity to run commands before syncing. (no idea if it is possible)

@NoxArt
Copy link
Owner

NoxArt commented Aug 21, 2012

Added a delay in caf9179 ... I'd be glad for a feedback

@m4recek
Copy link
Author

m4recek commented Aug 23, 2012

Thank you. Timeout is working good. Looking forward for uploading all changed files.

@NoxArt
Copy link
Owner

NoxArt commented Sep 2, 2012

Hi ... so I implemented a0d446d the afterwatch ;) I'd be glad for feedback

Also check out a Wiki entry on that topic https://github.com/NoxArt/SublimeText2-FTPSync/wiki/Why-and-how-to-use-afterwatch

@m4recek
Copy link
Author

m4recek commented Sep 4, 2012

Hi, unfortunately i cant get it working :/

After trying various settings I keep getting this error:

Writing file ..........\style.less with encoding UTF-8
FTPSync > Loading config: cache hit (key: ..........\style.less)
Running D:\programy\Sublime Text 2\Data\Packages\LESS-build\dotless.Compiler.exe ..........\style.less style.css
FTPSync > Connection cache hit (key: 496c60ccc7e371709c8a28a2c1f80af7)
Exception in thread Thread-19:
Traceback (most recent call last):
File ".\threading.py", line 532, in __bootstrap_inner
File ".\FTPSync.py", line 1288, in run
File ".\FTPSync.py", line 766, in execute
File ".\FTPSync.py", line 739, in scanWatched
ValueError: too many values to unpack

@NoxArt
Copy link
Owner

NoxArt commented Sep 4, 2012

:/ ... and may I ask you for your settings file? At least the relevant part
thank you

@m4recek
Copy link
Author

m4recek commented Sep 4, 2012

OK, its working now, I had to delete & create ftpsync.settings, with new config its working as expected :) (Probably my bad configuration). So, thank you for your time :)

@NoxArt
Copy link
Owner

NoxArt commented Sep 4, 2012

Glad to hear that, great

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

No branches or pull requests

2 participants