Return exit status 1 for additional cases

The combination of -e and package name patterns was not possible
before, therefore there is no precedent whether to return 0 or 1
if there is no match. If -e alone was used, the exit status was 0
even if there was no match, and this behavior is retained.

For the new situation that the package names matched but the
condition failed for all matches, 1 is now returned to indicate
that no output is generated in a query for specific packages.

src/query.c

@@ -1024,7 +1024,7 @@ exec_query(int argc, char **argv)
 		i++;
 	} while (i < argc);
 
-	if (nprinted == 0 && condition_sql == NULL && retcode == EXIT_SUCCESS) {
+	if (nprinted == 0 && match != MATCH_ALL && retcode == EXIT_SUCCESS) {
 		/* ensure to return a non-zero status when no package
 		 were found. */
 		retcode = EXIT_FAILURE;

tests/frontend/query.sh

@@ -81,7 +81,7 @@ EOF
 	atf_check \
 		-o empty \
 		-e empty \
-		-s exit:0 \
+		-s exit:1 \
 		pkg query -e "%#r>0" "%n: %rn %rv %ro" plop
 
 	atf_check \