Contributors mailing list archives

contributors@odoo-community.org

Re: OCA Addons Packaging next step

by
Savoir-faire Linux, Maxime Chambreuil
- 14/03/2016 15:58:44
Stéphane,

Would it be possible to have the setup.py file in the module directory, next to __openerp__.py and README.rst?

Thanks.
--
Maxime Chambreuil
Ring ID: 239e01f6262afbf3c7d276d2c661173ddddc0340
Tel: +1 (514) 276-5468 #126

----- Le 14 Mar 16, à 7:38, <Bidoul@pad.odoo-community.org> a écrit :

Hello Daniel,
Thanks for looking in to this. My comments inline below.

On Fri, Mar 11, 2016 at 7:23 PM, Daniel Reis <dgreis@sapo.pt> wrote:

a) I would prefer to have a ".setup" directory name instead of "setup".  Is this viable?

It would work I think. But I prefer setup because
* even if it is autogenerated most of the time, sometimes the setup.py can and will be adapted manually to cover special cases, or express explicit version dependencies
* if you want to install from git source, you can do pip install -e "git+https://github.com/acsone/account-financial-reporting.git#egg=mis_builder&subdirectory=setup/mis_builder", so there too I think the dot would complicate things

b) I also couldn't find the setup.py file in the  acsone/account-financial-reporting repo. Did i miss something?

They are on the 8.0-with-setup branch. See for instance https://github.com/acsone/account-financial-reporting/tree/8.0-with-setup/setup/ .
There is one setup.py for each addon.

c) How will the "setup" directory be kept up to date? An additional task for the OCA bot?

At first I plan to do it with the OCA bot, yes. The setuptools-odoo-makedefault script (part of setuptools-odoo) does most of the required work already. 
In the future, I hope authors will include the setup.py manually (it's simple enough) and the autogeneration will progressively become unncessary.

Best regards,

-sbi

_______________________________________________
Mailing-List: http://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: http://odoo-community.org/groups?unsubscribe