Skip to content
This repository was archived by the owner on Sep 16, 2024. It is now read-only.

Conversation

@moylop260
Copy link
Contributor

We have the following patch: gurneyalex/odoo-extra@92b578a
But runbot_travis2docker is not working with this one

Fix #105


<record id="runbot_repo_demo1" model="runbot.repo">
<field name="name">github.com/vauxoo-dev/fast-travis-test.git</field>
<field name="name">https://github.com/vauxoo-dev/fast-travis-test.git</field>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think the code is working if this data didn't work. The repo name didn't start with https:// or git@, so shouldn't the https have been appended automatically?

Copy link
Contributor Author

@moylop260 moylop260 Nov 10, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The error is because travis is using the original repo of odoo/odoo-extra without the following patch: odoo/odoo-extra#74 used on OCA Runbot server

Copy link
Contributor

@lasley lasley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @moylop260

@moylop260
Copy link
Contributor Author

moylop260 commented Nov 11, 2016

FYI I just ran a t2d of runbot but using:

  • This PR
  • Data demo without https
  • Patched odoo/odoo-extra repository

screen shot 2016-11-10 at 6 34 26 pm

@pedrobaeza pedrobaeza merged commit 028cc61 into OCA:9.0 Nov 11, 2016
@moylop260 moylop260 deleted the 9.0-oca-https-moy branch March 24, 2018 07:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants