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/sync.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/sync.c')
-rw-r--r-- | lib/libalpm/sync.c | 11 |
1 files changed, 8 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)) { |