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

Mention oca-port in migration procedure wiki page #589

Open
sbidoul opened this issue Nov 13, 2023 · 6 comments
Open

Mention oca-port in migration procedure wiki page #589

sbidoul opened this issue Nov 13, 2023 · 6 comments

Comments

@sbidoul
Copy link
Member

sbidoul commented Nov 13, 2023

I'm under the impression that oca-port is not now mature enough to recommend in the migration procedure wiki page.

What do people think?

@sebalix would you be able to propose that (while keeping the manual procedure in the document for now)?

@pedrobaeza
Copy link
Member

In any case, it should be odoo-module-migrator instead.

@sbidoul
Copy link
Member Author

sbidoul commented Nov 13, 2023

Why "instead"? In my understanding they are complementary tools, with oca-port caring about the branch creation and cherry-picking, while odoo-module-migrator is to transform the source code. That's why I created a separate issue.

@pedrobaeza
Copy link
Member

I think odoo-module-migrator does also the commit history fetching.

@sebalix
Copy link

sebalix commented Nov 13, 2023

Yes odoo-module-migrator does the git format-patch as well, that's the only common point I think, otherwise oca-port is only about playing with git repositories/commits, while odoo-module-migrator deals with code transformation in respect to the targeted Odoo version. In fact it could be called after oca-port process during a migration.

@sbidoul agree, oca-port is not mature enough, and we aim to change a bit the input parameters so I would not recommend this tool yet in the migration guide.

@sbidoul
Copy link
Member Author

sbidoul commented Nov 13, 2023

agree, oca-port is not mature enough, and we aim to change a bit the input parameters so I would not recommend this tool yet in the migration guide.

Ow, I had mistyped "not" where I meant "now mature" enough. But, good to now there are still changes upcoming in the CLI. Thanks for the update.

@sebalix
Copy link

sebalix commented Nov 13, 2023

That said, odoo-module-migrator exists since a long time so it deserves to be mentioned in the doc.
Once I'm confident with oca-port stability, and if it doesn't break any workflow with other tools I'll be glad to mention it as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants