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

[REF] install: Use git instead of tar to download odoo #540

Merged
merged 1 commit into from
Apr 17, 2018

Conversation

moylop260
Copy link
Contributor

@moylop260 moylop260 commented Apr 16, 2018

  • Advantages using git clone instead of the tar file:
  • Running MQT script from your local machine or docker you can re-use the odoo repository to upgrade it (git pull)
  • Using git depth=50 like as travis-ci uses we will have a faster download.
  • We can know the odoo sha of the build if there is an error you can reproduce it using the same sha.
  • We can verify a custom change applied to odoo repository if there is a sed executed or patch using git diff and return to original state.
  • Remove flaky error using tar files:
  • We can use a odoo's PR to test it

More info about:
#539 (comment)

* Advantages using git clone instead of the tar file:
 - Running MQT script from your local machine or docker you can re-use the odoo repository to upgrade it (`git pull`)
 - Using `git depth=50` like as travis-ci uses we will have a faster download.
 - We can know the odoo sha of the build if there is an error you can reproduce it using the same sha.
 - We can verify a custom change applied to odoo repository if there is a `sed` executed or `patch` using `git diff` and return to original state.
 - Remove flaky error using tar files:
    - OCA#539
 - We can use a odoo's PR to test it
@moylop260
Copy link
Contributor Author

@pedrobaeza
It's done!

@pedrobaeza
Copy link
Member

I think we can go on with this technical PR, as no more reviews are expected.

@pedrobaeza pedrobaeza merged commit fd8fa63 into OCA:master Apr 17, 2018
@moylop260 moylop260 deleted the master-oca-git-clone-moy branch April 17, 2018 19:11
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.

None yet

3 participants