Install missing dependencies when using --all

After running:

trytond-admin -v -c trytond.conf -d test_trytond --all

I get this error:

12159 140645614442304 [2016-07-07 22:06:31,642] INFO trytond.pool init pool for "test_trytond"
Traceback (most recent call last):
File "/home/labData/SourceCode/trytond/try4/env/bin/trytond-admin", line 21, in <module>
admin.run(options)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/admin.py", line 48, in run
Pool(db_name).init(update=options.update, lang=lang)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/pool.py", line 155, in init
lang=lang)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/modules/__init__.py", line 429, in load_modules
_load_modules()
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/modules/__init__.py", line 396, in _load_modules
graph = create_graph(module_list)[0]
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/modules/__init__.py", line 191, in create_graph
- set((p[0] for p in packages))))
Exception: Missing dependencies: [u'purchase_request']


Purchase request is installed

trytond (4.0.2)
trytond-purchase (4.0.2)
trytond-purchase-request (4.0.1)

This is a new module for which you are depending now. So you must activate it when updating with "-u purchase_request".

added 1 deleted label

closed

Ok I tried:

trytond-admin -v -c trytond.conf -d test_trytond -u purchase_request

but then I got this error:

13603 140183954634560 [2016-07-07 22:43:57,034] ERROR trytond.convert Error while parsing xml file:
In tag record: model ir.action.act_window with id act_purchase_request_form.
Traceback (most recent call last):
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/convert.py", line 447, in parse_xmlstream
self.sax_parser.parse(source)
File "/usr/lib64/python2.7/xml/sax/expatreader.py", line 107, in parse
xmlreader.IncrementalParser.parse(self, source)
File "/usr/lib64/python2.7/xml/sax/xmlreader.py", line 123, in parse
self.feed(buffer)
File "/usr/lib64/python2.7/xml/sax/expatreader.py", line 210, in feed
self._parser.Parse(data, isFinal)
File "/usr/lib64/python2.7/xml/sax/expatreader.py", line 307, in end_element
self._cont_handler.endElement(name)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/convert.py", line 507, in endElement
self.taghandler = self.taghandler.endElement(name)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/convert.py", line 289, in endElement
self.model.__name__, self.values, self.xml_id)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/convert.py", line 693, in import_record
self.create_records(model, [values], [fs_id])
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/convert.py", line 699, in create_records
records = Model.create(vlist)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/ir/action.py", line 324, in create
record, = super(ActionMixin, cls).create([values])
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/model/modelsql.py", line 595, in create
field.set(cls, fname, *fargs)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/model/fields/function.py", line 109, in set
setter(Model.browse(ids), name, value)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/ir/action.py", line 285, in set_action
Action.write([r.action for r in records], {
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/model/fields/field.py", line 234, in __get__
return inst.__getattr__(self.name)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/model/modelstorage.py", line 1352, in __getattr__
read_data = self.read(list(ids), ffields.keys())
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/model/modelsql.py", line 692, in read
order_by=history_order, limit=history_limit))
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/psycopg2cffi/_impl/cursor.py", line 30, in check_closed_
return func(self, *args, **kwargs)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/psycopg2cffi/_impl/cursor.py", line 263, in execute
self._pq_execute(self._query, conn._async)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/psycopg2cffi/_impl/cursor.py", line 696, in _pq_execute
self._pq_fetch()
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/psycopg2cffi/_impl/cursor.py", line 757, in _pq_fetch
raise self._conn._create_exception(cursor=self)
ProgrammingError: column a.context_model does not exist
LINE 1: ...te" AS "create_date", "a"."context" AS "context", "a"."conte...
^

Traceback (most recent call last):
File "/home/labData/SourceCode/trytond/try4/env/bin/trytond-admin", line 21, in <module>
admin.run(options)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/admin.py", line 48, in run
Pool(db_name).init(update=options.update, lang=lang)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/pool.py", line 155, in init
lang=lang)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/modules/__init__.py", line 429, in load_modules
_load_modules()
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/modules/__init__.py", line 398, in _load_modules
load_module_graph(graph, pool, update, lang)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/modules/__init__.py", line 251, in load_module_graph
tryton_parser.parse_xmlstream(fp)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/convert.py", line 447, in parse_xmlstream
self.sax_parser.parse(source)
File "/usr/lib64/python2.7/xml/sax/expatreader.py", line 107, in parse
xmlreader.IncrementalParser.parse(self, source)
File "/usr/lib64/python2.7/xml/sax/xmlreader.py", line 123, in parse
self.feed(buffer)
File "/usr/lib64/python2.7/xml/sax/expatreader.py", line 210, in feed
self._parser.Parse(data, isFinal)
File "/usr/lib64/python2.7/xml/sax/expatreader.py", line 307, in end_element
self._cont_handler.endElement(name)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/convert.py", line 507, in endElement
self.taghandler = self.taghandler.endElement(name)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/convert.py", line 289, in endElement
self.model.__name__, self.values, self.xml_id)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/convert.py", line 693, in import_record
self.create_records(model, [values], [fs_id])
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/convert.py", line 699, in create_records
records = Model.create(vlist)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/ir/action.py", line 324, in create
record, = super(ActionMixin, cls).create([values])
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/model/modelsql.py", line 595, in create
field.set(cls, fname, *fargs)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/model/fields/function.py", line 109, in set
setter(Model.browse(ids), name, value)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/ir/action.py", line 285, in set_action
Action.write([r.action for r in records], {
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/model/fields/field.py", line 234, in __get__
return inst.__getattr__(self.name)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/model/modelstorage.py", line 1352, in __getattr__
read_data = self.read(list(ids), ffields.keys())
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/trytond/model/modelsql.py", line 692, in read
order_by=history_order, limit=history_limit))
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/psycopg2cffi/_impl/cursor.py", line 30, in check_closed_
return func(self, *args, **kwargs)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/psycopg2cffi/_impl/cursor.py", line 263, in execute
self._pq_execute(self._query, conn._async)
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/psycopg2cffi/_impl/cursor.py", line 696, in _pq_execute
self._pq_fetch()
File "/home/labData/SourceCode/trytond/try4/env/lib/python2.7/site-packages/psycopg2cffi/_impl/cursor.py", line 757, in _pq_fetch
raise self._conn._create_exception(cursor=self)
psycopg2cffi._impl.exceptions.ProgrammingError: column a.context_model does not exist
LINE 1: ...te" AS "create_date", "a"."context" AS "context", "a"."conte...

added 1 deleted label and removed 1 deleted label

reopened

But you also have to upgrade the other modules.

PS: This is not the right place to get support. The bug tracker is only for bugs.

added 1 deleted label and removed 1 deleted label

closed

Yes I understand that. I'm wasn't trying to get support, but trying to upgrade all the modules gave an error and upgrading one gave an error, uninstalling the module and then doing the upgrade gives the same error. Which is why I felt this was a bug.

added 1 deleted label and removed 1 deleted label

reopened

There is no bug if you make an upgrade of everything plus the new modules.

added 1 deleted label and removed 1 deleted label

closed

I will like to convert this to a feature, so when using --all the missing dependencies are also installed.

added trytond type::feature + 1 deleted label and removed 1 deleted label

reopened

I don't think it is a good feature. Explicit is better than implicit.

I see your point, but I'm thinking in a command that will always work when updating to new versions installing new dependencies. So maybe we should add a new option like debian: update vs dist-update [1]

[1] http://askubuntu.com/questions/194651/why-use-apt-get-upgrade-instead-of-apt-get-dist-upgrade#226213