Code review comment for lp:~openerp-community/openerp/skitzotek_trunk_symlinks

Revision history for this message
David Bowers (skitzotek) wrote :

Hi,

My Diff needs to be rejected I think.
Now I've had a look at the release notes and the new Makefile I realise that we should not be making symbolic links for 6.1+ in the server as addons also come from web and including the directories in the addons path seems to be a better solution.

However the Makefile is a pain to use under Windows and doesn't seem quite so flexible, so a .py would still be much better. To make it easier, bzr_set.py, Makefile and Makefile_helper.py should be combined into one Python script, bzr_openerp.py that is kept up to date from the server. Another script like bzr_settings.py should hold localised settings that do not get overwritten when bzr_openerp.py is auto-updated from the server. It could also list the various bugs and features that are being branched, committed and pushed.

Just an idea ...

« Back to merge proposal