Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When multi_progress_bar finishes print new line automatically #1805

Merged
merged 2 commits into from
Nov 1, 2024

Conversation

kontura
Copy link
Contributor

@kontura kontura commented Oct 29, 2024

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

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: rpm-software-management#1792
Mostly to test new lines after `multi_progress_bar`.
@kontura
Copy link
Contributor Author

kontura commented Oct 29, 2024

CI test fix: rpm-software-management/ci-dnf-stack#1582

@m-blaha m-blaha self-assigned this Oct 30, 2024
Copy link
Member

@m-blaha m-blaha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@m-blaha m-blaha added this pull request to the merge queue Nov 1, 2024
Merged via the queue into rpm-software-management:main with commit 1c5f28c Nov 1, 2024
16 of 20 checks passed
kontura added a commit to kontura/dnf5 that referenced this pull request Dec 4, 2024
Previously the new line was always printed resulting in excessive empty
line which was visible for example during the downloading packages phase
of an update:
```
[  1/109] kernel-modules-0:6.13.0-0.rc1.20241202gite70140ba0d2b.14.fc42.x86_64                                                                         7% [=                 ] | 781.3 KiB/s |   4.9 MiB | -01m18s
[  2/109] kernel-core-0:6.13.0-0.rc1.20241202gite70140ba0d2b.14.fc42.x86_64                                                                            8% [==                ] | 381.1 KiB/s |   1.5 MiB | -00m45s
[  3/109] kernel-modules-core-0:6.13.0-0.rc1.20241202gite70140ba0d2b.14.fc42.x86_64                                                                    3% [=                 ] | 355.8 KiB/s |   1.2 MiB | -01m47s
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
[  0/109] Total                                                                                                                                        3% [=                 ] |   1.5 MiB/s |   7.6 MiB | -02m05s

```
(last line is empty)

This was introduced recently in PR rpm-software-management#1805
github-merge-queue bot pushed a commit that referenced this pull request Dec 5, 2024
Previously the new line was always printed resulting in excessive empty
line which was visible for example during the downloading packages phase
of an update:
```
[  1/109] kernel-modules-0:6.13.0-0.rc1.20241202gite70140ba0d2b.14.fc42.x86_64                                                                         7% [=                 ] | 781.3 KiB/s |   4.9 MiB | -01m18s
[  2/109] kernel-core-0:6.13.0-0.rc1.20241202gite70140ba0d2b.14.fc42.x86_64                                                                            8% [==                ] | 381.1 KiB/s |   1.5 MiB | -00m45s
[  3/109] kernel-modules-core-0:6.13.0-0.rc1.20241202gite70140ba0d2b.14.fc42.x86_64                                                                    3% [=                 ] | 355.8 KiB/s |   1.2 MiB | -01m47s
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
[  0/109] Total                                                                                                                                        3% [=                 ] |   1.5 MiB/s |   7.6 MiB | -02m05s

```
(last line is empty)

This was introduced recently in PR #1805
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Problem with reposync stderr formatting with skip_if_unavailable
2 participants