Code review comment for lp:~liuyq0307/lava-dispatcher/fix-1195536

Revision history for this message
Yongqin Liu (liuyq0307) wrote :

One thing I am not sure is that if I should change the usage of _copy_needed_files_from_partition method in deploy_android method of lava_dispatcher/device/fastmodel.py file.

it is called as such in the deploy_linaro method:
self._copy_needed_files_from_directory(odir)
self._copy_needed_files_from_partition(self.config.boot_part, 'rtsm')
self._copy_needed_files_from_partition(self.config.root_part, 'boot')

but called like this in the deploy_android method:
self._copy_needed_files_from_partition(self.config.boot_part, '')

if we add following line to the deploy_android method, I guess it will work too.
self._copy_needed_files_from_partition(self.config.boot_part, 'rtsm')

but do we really need to specify the second parameter?

« Back to merge proposal