When multi_progress_bar finishes print new line automatically

The last line of `multi_progress_bar` can look like:
`[2/2] Total           100% |   0.0   B/s |  20.0   B |  00m00s`
or
`http://localhost:43223/api_3/rpmrepo 100% |  72.0 KiB/s | 295.0   B |  00m00s`
I don't think any client will ever want to append to that.

Instead of it needing to be printed by the client the
`multi_progress_bar` automatically ends with a new line.

This new line was missing on some places. For example in output of:
`dnf copr enable rpmsoftwaremanagement/dnf-nightly `
or
`dnf5 rq --repofrompath=test,https://www.not-available-repo.com/ --repo test --setopt=skip_if_unavailable=0`
There were also excessive new lines like in:
`dnf5 remove htop -y &> out`

Closes: #1792

dnf5/commands/download/download.cpp

@@ -304,7 +304,6 @@ void DownloadCommand::run() {
 
     std::cout << "Downloading Packages:" << std::endl;
     downloader.download();
-    std::cout << std::endl;
 }
 
 

dnf5/context.cpp

@@ -456,7 +456,6 @@ void Context::Impl::download_and_run(libdnf5::base::Transaction & transaction) {
     if (base.get_config().get_downloadonly_option().get_value()) {
         return;
     }
-    print_info("");
 
     if (should_store_offline) {
         store_offline(transaction);
@@ -493,7 +492,6 @@ void Context::Impl::download_and_run(libdnf5::base::Transaction & transaction) {
     }
 
     auto result = transaction.run();
-    print_info("");
     if (result != libdnf5::base::Transaction::TransactionRunResult::SUCCESS) {
         print_error(libdnf5::utils::sformat(
             _("Transaction failed: {}"), libdnf5::base::Transaction::transaction_result_to_string(result)));

dnf5/download_callbacks.cpp

@@ -104,7 +104,6 @@ int DownloadCallbacks::mirror_failure(void * user_cb_data, const char * msg, con
 void DownloadCallbacks::reset_progress_bar() {
     multi_progress_bar.reset();
     if (printed) {
-        std::cerr << std::endl;
         printed = false;
     }
 }

dnf5daemon-client/callbacks.cpp

@@ -89,7 +89,6 @@ void DownloadCB::print() {
 void DownloadCB::reset_progress_bar() {
     multi_progress_bar.reset();
     if (printed) {
-        std::cerr << std::endl;
         printed = false;
     }
 }

libdnf5-cli/progressbar/multi_progress_bar.cpp

@@ -124,11 +124,8 @@ std::ostream & operator<<(std::ostream & stream, MultiProgressBar & mbar) {
         }
         bar->set_number(numbers.back());
         numbers.pop_back();
-        if (mbar.line_printed) {
-            stream << std::endl;
-        }
         stream << *bar;
-        mbar.line_printed = true;
+        stream << std::endl;
         mbar.bars_done.push_back(bar);
         // TODO(dmach): use iterator
         mbar.bars_todo.erase(mbar.bars_todo.begin() + static_cast<int>(i));
@@ -200,7 +197,8 @@ std::ostream & operator<<(std::ostream & stream, MultiProgressBar & mbar) {
         }
 
         stream << mbar.total;
-        mbar.num_of_lines_to_clear += 2;
+        stream << std::endl;
+        mbar.num_of_lines_to_clear += 3;
     }
 
     return stream;