Mailinglist Archive: zypp-devel (115 mails)
| < Previous | Next > |
[zypp-devel] Re: download area at commit
- From: Ladislav Slezak <lslezak@xxxxxxx>
- Date: Thu, 26 Jul 2007 15:08:43 +0200
- Message-id: <46A89CDB.9030105@xxxxxxx>
Duncan Mac-Vicar Prett wrote:
[...]
> It means:
> - as we dont open the medias till commit, and MoveDownloadArea() in bindings
> is called just before commit, it should be safe to use just
Some files are downloaded before commit (e.g. license), they are downloaded
by internal MediaAccess stored in pkg-bindings and assigned to that repository.
So I think it's OK.
All files downloaded by yast are small so it should be OK to use the ramdisk
even later when the target is accessible (so we do not need to change the
already existing MediaAccess objects in SourceMoveDownloadArea()).
> MediaManager::setAttachPrefix in the MoveDownloadArea() func.
> - MoveDownLoad area gets /instdir + /var/tmp as parameter. I am not sure if
> MediaManager::setAttachPrefix includes the tmpdir again.
It should not add any prefix (or suffix) to the passed path, it should use the
path as is.
--
Best Regards
Ladislav Slezák
Yast Developer
------------------------------------------------------------------------
SUSE LINUX, s.r.o. e-mail: lslezak@xxxxxxx
Lihovarská 1060/12 tel: +420 284 028 960
190 00 Prague 9 fax: +420 284 028 951
Czech Republic http://www.suse.cz/
--
To unsubscribe, e-mail: zypp-devel+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: zypp-devel+help@xxxxxxxxxxxx
[...]
> It means:
> - as we dont open the medias till commit, and MoveDownloadArea() in bindings
> is called just before commit, it should be safe to use just
Some files are downloaded before commit (e.g. license), they are downloaded
by internal MediaAccess stored in pkg-bindings and assigned to that repository.
So I think it's OK.
All files downloaded by yast are small so it should be OK to use the ramdisk
even later when the target is accessible (so we do not need to change the
already existing MediaAccess objects in SourceMoveDownloadArea()).
> MediaManager::setAttachPrefix in the MoveDownloadArea() func.
> - MoveDownLoad area gets /instdir + /var/tmp as parameter. I am not sure if
> MediaManager::setAttachPrefix includes the tmpdir again.
It should not add any prefix (or suffix) to the passed path, it should use the
path as is.
--
Best Regards
Ladislav Slezák
Yast Developer
------------------------------------------------------------------------
SUSE LINUX, s.r.o. e-mail: lslezak@xxxxxxx
Lihovarská 1060/12 tel: +420 284 028 960
190 00 Prague 9 fax: +420 284 028 951
Czech Republic http://www.suse.cz/
--
To unsubscribe, e-mail: zypp-devel+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: zypp-devel+help@xxxxxxxxxxxx
| < Previous | Next > |