1.1 --- a/main.c	Sun Apr 06 00:29:47 2008 -0400
     1.2 +++ b/main.c	Sun Apr 06 19:34:58 2008 -0400
     1.3 @@ -338,14 +338,15 @@
     1.4  {
     1.5  	struct razor_set *set, *upstream;
     1.6  	struct razor_transaction *trans;
     1.7 +	int errors;
     1.8  
     1.9  	set = razor_set_open(repo_filename);
    1.10  	upstream = razor_set_open(rawhide_repo_filename);
    1.11  	if (set == NULL || upstream == NULL)
    1.12  		return 1;
    1.13  	trans = razor_transaction_create(set, upstream, argc, argv, 0, NULL);
    1.14 -	razor_transaction_describe(trans);
    1.15 -	if (trans->errors)
    1.16 +	errors = razor_transaction_describe(trans);
    1.17 +	if (errors)
    1.18  		return 1;
    1.19  
    1.20  	set = razor_transaction_run(trans);
    1.21 @@ -363,13 +364,14 @@
    1.22  {
    1.23  	struct razor_set *set;
    1.24  	struct razor_transaction *trans;
    1.25 +	int errors;
    1.26  
    1.27  	set = razor_set_open(repo_filename);
    1.28  	if (set == NULL)
    1.29  		return 1;
    1.30  	trans = razor_transaction_create(set, NULL, 0, NULL, argc, argv);
    1.31 -	razor_transaction_describe(trans);
    1.32 -	if (trans->errors)
    1.33 +	errors = razor_transaction_describe(trans);
    1.34 +	if (errors)
    1.35  		return 1;
    1.36  
    1.37  	set = razor_transaction_run(trans);
    1.38 @@ -519,7 +521,7 @@
    1.39  	struct razor_rpm *rpm;
    1.40  	const char *filename;
    1.41  	char path[PATH_MAX], new_path[PATH_MAX], **packages;
    1.42 -	int i;
    1.43 +	int errors, i;
    1.44  
    1.45  	upstream = create_set_from_rpms(argc, argv);
    1.46  	snprintf(path, sizeof path,
    1.47 @@ -535,8 +537,8 @@
    1.48  					 argc, (const char **)packages,
    1.49  					 0, NULL);
    1.50  	free(packages);
    1.51 -	razor_transaction_describe(trans);
    1.52 -	if (trans->errors)
    1.53 +	errors = razor_transaction_describe(trans);
    1.54 +	if (errors)
    1.55  		return 1;
    1.56  
    1.57  	/* FIXME: Use _finish() convention here?  That is, a function