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

Sitemap does not generated #84

Open
dmitrys99 opened this issue Jul 22, 2015 · 5 comments
Open

Sitemap does not generated #84

dmitrys99 opened this issue Jul 22, 2015 · 5 comments

Comments

@dmitrys99
Copy link

When I include (sitemap) to plugins setting nothing happens after call to (coleslaw:main ...) while other files generated successfully.
Seems that sitemap plugin does not even get called.
Latest master branch, sbcl 1.2.13, FreeBSD.

@kingcons
Copy link
Collaborator

Thanks for the bug report! I'll get a fix for this written this weekend.

@PuercoPop
Copy link
Collaborator

It makes sense that it doesn't get called.

The sitemap plugin 'hooks' itself by defining a publish method to a class that is neither a subclass of content or index, so #'compile-blog will never call it. However that appears to be a deliberate decision 78f1fc9

Although the problem is that sitemap needs to be called last, I can't quite grasp what was the intended solution.

The least hacky solution I can think of, but hacky nonetheless, is to define an *after-publish-hook* in #'compile-blog and let the sitemap plugin hook to it.

@dmitrys99
Copy link
Author

Maybe it will be good to have some other hooks exported? Before publish, for example?
Or maybe make plugin have method execute to be called when appropriate?

@kingcons
Copy link
Collaborator

Indeed, the previous "intended solution" was clearly not thought through on my part. This is a big part of why my next big Coleslaw task during my vacation the next 2 weeks will be to get a solid test suite written and Travis CI set up.

I think from a design perspective, turning compile-blog (and maybe also deploy) into a Generic Function that takes a Config object is an interesting idea. Then there could be an :around method defined on compile-blog to do the directory handling stuff and run (call-next-method) inside the with-staging-directory stuff. Anyway, I will have a look at it as soon as I can. This week at work is particularly crazy so it will be a few days.

@ehouse
Copy link

ehouse commented Mar 2, 2016

Have there been any updates on this? I noticed it still isn't getting generated on master.

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

4 participants