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

Auto Recompile ErlyDtl #28

Open
Dryymoon opened this issue Mar 22, 2014 · 7 comments
Open

Auto Recompile ErlyDtl #28

Dryymoon opened this issue Mar 22, 2014 · 7 comments
Milestone

Comments

@Dryymoon
Copy link

I will update to last version of Sync 22/03/2014, but ErlyDtl templates don`t recompiles after changes.

@choptastic
Copy link
Collaborator

Oh boy. Right, the latest ErlyDTL Api changes. I'll have a look at this,
thanks!

Jesse Gumm
Owner, Sigma Star Systems
414.940.4866 || sigma-star.com || @jessegumm
On Mar 21, 2014 8:19 PM, "Dryymoon" [email protected] wrote:

I will update to last version of Sync 22/03/2014, but ErlyDtl templates
don`t recompiles after changes.

Reply to this email directly or view it on GitHubhttps://github.com//issues/28
.

@Dryymoon
Copy link
Author

Thanks a lot, for fast reply.

@kaos
Copy link

kaos commented Apr 1, 2014

Hmm.. I thought I'd kept the API backwards compatible.. I'd be interested to know if this was not the case.

@choptastic
Copy link
Collaborator

Thanks Andreas, it's possible that given some of the crash issues from the
pre 0.9.3 releases are related to this.

I'll let you know as soon as I discover what's going on here.

On Tue, Apr 1, 2014 at 4:13 AM, Andreas Stenius [email protected]:

Hmm.. I thought I'd kept the API backwards compatible.. I'd be interested
to know if this was not the case.

Reply to this email directly or view it on GitHubhttps://github.com//issues/28#issuecomment-39185044
.

Jesse Gumm
Owner, Sigma Star Systems
414.940.4866 || sigma-star.com || @jessegumm

@kaos
Copy link

kaos commented Apr 2, 2014

Cool. I also realized that there actually was a API breaking change going
from 0.9.0 to 0.9.1 in the result terms.
But looking at the code, it looks like this should was already taken care
of, as the erlydtl compile call is paired with a call to compile:file.

2014-04-02 3:17 GMT+02:00 Jesse Gumm [email protected]:

Thanks Andreas, it's possible that given some of the crash issues from the
pre 0.9.3 releases are related to this.

I'll let you know as soon as I discover what's going on here.

On Tue, Apr 1, 2014 at 4:13 AM, Andreas Stenius <[email protected]

wrote:

Hmm.. I thought I'd kept the API backwards compatible.. I'd be interested
to know if this was not the case.

Reply to this email directly or view it on GitHub<
https://github.com/rustyio/sync/issues/28#issuecomment-39185044>
.

Jesse Gumm
Owner, Sigma Star Systems
414.940.4866 || sigma-star.com || @jessegumm

Reply to this email directly or view it on GitHubhttps://github.com//issues/28#issuecomment-39279766
.

@mdg
Copy link

mdg commented Oct 5, 2014

Likewise, I'm not seeing sync recognize changes to my .dtl files either. If I shutdown my app, make and restart, the changes show. If I make w/o shutting down, everything gets picked up as a change and the new template shows.

@choptastic choptastic added this to the 1.0.0 milestone Oct 12, 2014
@ten0s
Copy link
Contributor

ten0s commented Mar 18, 2016

sync doesn't reload modules built without this erlydtl_opts's {module_ext, "_dtl"} option.

because of:
https://github.com/rustyio/sync/blob/master/src/sync_scanner.erl#L441
https://github.com/rustyio/sync/blob/master/src/sync_scanner.erl#L484

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

5 participants