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

Fix travis (and local) builds #211

Closed
wants to merge 5 commits into from
Closed

Fix travis (and local) builds #211

wants to merge 5 commits into from

Conversation

macie
Copy link

@macie macie commented Dec 5, 2017

This branch fixes #207 by:

  • updating pillar download scripts (to most recent)
  • updating travis config to run automatic builds.

What is not working (yet):

  • compiling book - something is wrong when I try to compile book. How did you make it before?

@Ducasse
Copy link
Member

Ducasse commented Dec 6, 2017

Hi
Thanks for this contribution. Now I do not know what you are doing. Because this book should be migrated to use the same configuration and version than for example the SmaccBooklet or the LearningObjectOriented book.

@macie
Copy link
Author

macie commented Dec 6, 2017

My primary motivation was to read about Pharo on my e-book reader (#209). To do that, I wanted to compile book on my computer (to one-file HTML document, and than convert to EPUB). But it was impossible.

So I updated pillar download script and (by the way) simplify travis configuration. But now, after reading your configuration examples I get stuck. Documentation on compilation is unclear. I feel overwhelmed by this procedure (travis with jenkins with docker with makefiles - probably created by package Pillar-MakefileGeneration - with bintray) - it's very complicated to create a readable book.

I'd like to help with this project, but unfortunately I miss direction. It would be great, if you can provide me some guidelines.

@Ducasse
Copy link
Member

Ducasse commented Dec 6, 2017

If it works for you then use it.
Now travis is simple this is just to compile each time someone without latex installation does a commit.
Docker is for people that do not have latex installed.
So if you have latex installed you do not need docker. Docker is for the travis configuration.
I do not get why you need to touch the travis configuration because this is for the validation on travis and you do not care about it.
Now I do not have the time to use the latest version of Pillar (pillar 50) the one that it is used in the booklet for example or the learning OOP book.
Basically we should copy the support folder and the makefile and update the pillar.conf but I'm super super busy right now because it may not work the first time.

@Ducasse
Copy link
Member

Ducasse commented Sep 8, 2018

We moved to the pillar 7 dev branch (soon to be released).
Thanks again for your time. We will check the Epub support because we have it in the booklet on physche or a reflective core.

@Ducasse Ducasse closed this Sep 8, 2018
@macie macie deleted the mz_fix_build branch September 9, 2018 17:26
@iNode
Copy link

iNode commented Jun 17, 2019

We moved to the pillar 7 dev branch (soon to be released).
Thanks again for your time. We will check the Epub support because we have it in the booklet on physche or a reflective core.

Any progress with actual fix? Build still seems to be broken.

@Ducasse
Copy link
Member

Ducasse commented Jun 17, 2019

Hi

I do not have brain ressources for this issue now. I will try to have a look at it after mid of july.

S.

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

Successfully merging this pull request may close these issues.

3 participants