agile-france / propile

Community Conference Program Compiler (for Agile France)
2 stars 8 forks source link

Configurer Travis CI #6

Closed thbar closed 10 years ago

thbar commented 11 years ago

L'intégration continue va nous aider.

dboissier commented 10 years ago

Configuration for Travis CI was written but some errors appear when building.

oelmekki commented 10 years ago

This is fixed by #24.

Travis problem was that database was not properly migrated. Actually, no migration was run at all, since rake db:migrate crashed, with the error : "host required".

It appears it was vlad library that was redefining the "role" method.

thenrio commented 10 years ago

haha, vlad was impaled!

dboissier commented 10 years ago

PDF tests fail :(

oelmekki commented 10 years ago

This works perfectly well on local, it seems to be travis specific.

Given where it happens (when trying to create the file, not to read it) and provided the error (no such file or directory), I suspect the problem is that path is relative : "tmp/sessioncard#{@session.id}.pdf"

It seems to mean that "tmp/" does not exist where command is issued. I'm gonna give a try to setting an absolute path, but we won't know if it works until travis run it, so please to merge the PR right away when it comes.

oelmekki commented 10 years ago

Queued for build

thbar commented 10 years ago

I suspect you'll have to create the tmp folder unless it already exist...

folder = Rails.root.join('tmp')
FileUtils.mkdir(folder) unless File.exists?(folder)
oelmekki commented 10 years ago

PR didn't fixed it.

I suspect you'll have to create the tmp folder unless it already exist...

Indeed, you're right : I just realized that tmp folder is ignored altogether.

Modifying pull request.

oelmekki commented 10 years ago

Ok, fixed and all green : #25

thbar commented 10 years ago

:+1: