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 | |
| 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>
| -rw-r--r-- | lib/libalpm/sync.c | 11 | ||||
| -rw-r--r-- | lib/libalpm/sync.h | 1 | ||||
| -rw-r--r-- | lib/libalpm/trans.c | 10 |
3 files changed, 19 insertions, 3 deletions
diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c index 9fe4eff5..c513b6b5 100644 --- a/lib/libalpm/sync.c +++ b/lib/libalpm/sync.c @@ -1263,13 +1263,12 @@ static int load_packages(alpm_handle_t *handle, alpm_list_t **data, return 0; } -int _alpm_sync_commit(alpm_handle_t *handle, alpm_list_t **data) +int _alpm_sync_load(alpm_handle_t *handle, alpm_list_t **data) { alpm_list_t *i, *deltas = NULL; size_t total = 0; uint64_t total_bytes = 0; alpm_trans_t *trans = handle->trans; - alpm_event_t event; if(download_files(handle, &deltas)) { alpm_list_free(deltas); @@ -1318,7 +1317,13 @@ int _alpm_sync_commit(alpm_handle_t *handle, alpm_list_t **data) return -1; } - trans->state = STATE_COMMITING; + return 0; +} + +int _alpm_sync_commit(alpm_handle_t *handle, alpm_list_t **data) +{ + alpm_trans_t *trans = handle->trans; + alpm_event_t event; /* fileconflict check */ if(!(trans->flags & ALPM_TRANS_FLAG_DBONLY)) { diff --git a/lib/libalpm/sync.h b/lib/libalpm/sync.h index f25e450b..c15d348f 100644 --- a/lib/libalpm/sync.h +++ b/lib/libalpm/sync.h @@ -25,6 +25,7 @@ #include "alpm.h" int _alpm_sync_prepare(alpm_handle_t *handle, alpm_list_t **data); +int _alpm_sync_load(alpm_handle_t *handle, alpm_list_t **data); int _alpm_sync_commit(alpm_handle_t *handle, alpm_list_t **data); #endif /* _ALPM_SYNC_H */ 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"); |