diff options
| author | Andrew Gregory <andrew.gregory.8@gmail.com> | 2014-10-11 10:08:20 +0200 |
|---|---|---|
| committer | Allan McRae <allan@archlinux.org> | 2014-10-13 06:28:29 +0200 |
| commit | c1d6cec2e277eea7affa43601086c523311b58c0 (patch) | |
| tree | ad2ef11358f4c67c7dd2a4d488b600616d91f0e3 /lib/libalpm/trans.c | |
| parent | 42c859e4cc38f91a1236cd93eae9e9f9a1bc1e62 (diff) | |
| download | pacman-c1d6cec2e277eea7affa43601086c523311b58c0.tar.gz pacman-c1d6cec2e277eea7affa43601086c523311b58c0.tar.xz | |
sync.c: download packages before starting commit
download_files sets the transaction state to STATE_DOWNLOADING.
Modifying the state after it has already been set to STATE_COMMITTING
created a brief window where SIGINT would fail to interrupt the process
and caused interrupted downloads to result in a 'transaction started'
message in the log with no matching transaction end.
Signed-off-by: Andrew Gregory <andrew.gregory.8@gmail.com>
Diffstat (limited to 'lib/libalpm/trans.c')
| -rw-r--r-- | lib/libalpm/trans.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/lib/libalpm/trans.c b/lib/libalpm/trans.c index aebd8dd2..7cdb096f 100644 --- a/lib/libalpm/trans.c +++ b/lib/libalpm/trans.c @@ -175,6 +175,16 @@ int SYMEXPORT alpm_trans_commit(alpm_handle_t *handle, alpm_list_t **data) return 0; } + if(trans->add) { + if(_alpm_sync_load(handle, data) != 0) { + /* pm_errno is set by _alpm_sync_load() */ + return -1; + } + if(trans->flags & ALPM_TRANS_FLAG_DOWNLOADONLY) { + return 0; + } + } + trans->state = STATE_COMMITING; alpm_logaction(handle, ALPM_CALLER_PREFIX, "transaction started\n"); |