1.1 --- a/main.c	Sun Apr 06 23:57:13 2008 -0400
     1.2 +++ b/main.c	Mon Apr 07 00:06:22 2008 -0400
     1.3 @@ -351,8 +351,7 @@
     1.4  	if (errors)
     1.5  		return 1;
     1.6  
     1.7 -	set = razor_transaction_run(trans);
     1.8 -	razor_transaction_destroy(trans);
     1.9 +	set = razor_transaction_finish(trans);
    1.10  	razor_set_write(set, updated_repo_filename);
    1.11  	razor_set_destroy(set);
    1.12  	razor_set_destroy(upstream);
    1.13 @@ -376,8 +375,7 @@
    1.14  	if (errors)
    1.15  		return 1;
    1.16  
    1.17 -	set = razor_transaction_run(trans);
    1.18 -	razor_transaction_destroy(trans);
    1.19 +	set = razor_transaction_finish(trans);
    1.20  	razor_set_write(set, updated_repo_filename);
    1.21  	razor_set_destroy(set);
    1.22  	printf("wrote system-updated.repo\n");
    1.23 @@ -546,13 +544,7 @@
    1.24  	if (errors)
    1.25  		return 1;
    1.26  
    1.27 -	/* FIXME: Use _finish() convention here?  That is, a function
    1.28 -	 * that starts the computation and returns the result while
    1.29 -	 * destroying the transaction.  Nice for transient objects
    1.30 -	 * such as the merger and the importer.  Should we do that for
    1.31 -	 * transactions too, that is, razor_transaction_finish()? */
    1.32 -	next = razor_transaction_run(trans);
    1.33 -	razor_transaction_destroy(trans);
    1.34 +	next = razor_transaction_finish(trans);
    1.35  
    1.36  	/* FIXME: Need razor_set_write_to_fd() so we can open it excl
    1.37  	 * up front here or fail if it already exists. */