Fix: Watch Pattern

[gpedro]

No description provided.

[gpedro]

#109

[zenorocha]

Just started reviewing :)

Sent from GH.

[zenorocha]

Hey @gpedro, I tried changing a value in src/layout/default.html and changes were not updated.

[gpedro]

@zenorocha, it appears that changed, but we need a trigger to rebuild.

[obetomuniz]

@gpedro @zenorocha The link below show another implementation using path module

https://github.com/braziljs/conf-boilerplate/blob/migration/docpad2metalsmith/metalsmith.js

[zenorocha]

I noticed that too @gpedro, could you send a fix?

[gpedro]

path.join LGTM. i'm trying understand how to trigger build on watch

[gpedro]

well, the problem is caused by

metalsmith.source('src/documents')

the rebuilder is working, but the source path fucks my life. If I change the layout/default and save, it creates the layouts folder in the root directory.
this happens with both branches migration/docpad2metalsmith as metalsmith.

we may have to create an intermediate folder (.tmp) to separate what will not go to the out folder

cc @zenorocha @obetomuniz

[zenorocha]

I see... well, we could try another solution such as metalsmith-simplewatch.

That way we don't need to create another build step to filter what is generated and what isn't.

[zenorocha]

Could you check it out @gpedro?

[gpedro]

@zenorocha: I will test this.

[gpedro]

@zenorocha, @obetomuniz
there are better ways to solve this problem, but gulp LGTM
https://github.com/gpedro/conf-boilerplate/commits/feature/metalsmith-gulp

[zenorocha]

I love gulp but introducing another build tool doesn't look good to me.

[zenorocha]

Maybe a standalone solution like simple-recursive-watch may work.

[gpedro]

@zenorocha,
simple-recursive-watch works! but the metalsmith does not work. When he will rebuild, rather than loading the file name, it loads the content in the name. WTF?

look: http://i.imgur.com/qnyC7yf.png

[zenorocha]

wait, what? hahaha

can you send your code so we can take a look?

[gpedro]

@zenorocha https://github.com/gpedro/conf-boilerplate/tree/fix/metalsmith-watch

[zenorocha]

Tried several stuff here and still couldn't make it work. @obetomuniz, could you take a look?

[gpedro]

@zenorocha, I will create an issue at segmentio/metalsmith reporting that.

---edit
segmentio/metalsmith/#134

[diogomoretti]

Any solution to the watch? The metalsmith watch it is now in version 1.0.1. Or we will use gulp method?

[gpedro]

@diogomoretti i will try with the 1.0.1

[diogomoretti]

The same error for me in your gif :/

[gpedro]

why we choose metalsmith? cc @zenorocha

[zenorocha]

Problem fixed.

[zenorocha]

I chose metalsmith after evaluating all the other JavaScript-based static generators.