'symfony assets:dump' not called during deployment

[kingcrunch]

Somehow the roles get lost capistrano invokes symfony:command from deploy:assets:install. Somebody with more Ruby-experience can probably debug this further, but this one at least works.

Fixes #10

[blaugueux]

I also have this issue on a project. For me none tasks using invoke "symfony:command" works except the first one.

You can see bellow that the "symfony:command" call only run the first time.

Example:

** Invoke symfony:cache:warmup (first_time)
** Execute symfony:cache:warmup
** Invoke symfony:command (first_time)
** Execute symfony:command
executing
...
** Invoke deploy:assets:install (first_time)
** Execute deploy:assets:install
** Invoke symfony:command 
** Invoke symfony:assetic:dump (first_time)
** Execute symfony:assetic:dump
** Invoke symfony:command 
...

[blaugueux]

I've made some other tests. My analyse seems to be right. All calls to the "symfony:command" task after the first one are not executed.

I'm looking to fix that but i'm not a ruby expert. Any help can be really useful here.

@kingcrunch BTW this PR does not solve the issue but only your case.

[kingcrunch]

@blaugueux

BTW this PR does not solve the issue but only your case.

Not, that I havent said that 😉

I must admit, that I don't understand your example. "invoke X (first_time)" vs "execute X"? Especially since it doesn't invoke symfony:command even the first time, because else I would have assets installed and I wouldn't have a problem 😉

Update: Don't get me wrong. If you can fix both issues while still using symfony:command, I'd vote for that 😄

[blaugueux]

@kingcrunch I know it's ok ;)

When you run deploy --trace you can see that if your deploy need to call the symfony:command task more than one time (symfony:asset:install and symfony:cache:warmup for example) only the first call will be executed.

In my example (my log extract), the cache warmup will be executed but neither the assets install or assetic dump.

[kingcrunch]

@blaugueux Nice to know, thanks. I'll try it out.

[blaugueux]

@kingcrunch can you try my PR #16 please, it fix the issue in my case.

[kingcrunch]

#16 seems to be the better (because more) stable solution.