| 1 |
Mike Gilbert <floppym@g.o> writes: |
| 2 |
|
| 3 |
> This looks a lot safer than yesterday's patch since there are no |
| 4 |
> ebuild removals here. |
| 5 |
|
| 6 |
Thank you Mike. |
| 7 |
|
| 8 |
> If/when you do want to remove old ebuilds, I suggest creating a github |
| 9 |
> PR, and let the CI bot check reverse dependencies. |
| 10 |
|
| 11 |
Yeah, that would have been a much safer way to remove ebuilds. |
| 12 |
|
| 13 |
> This was actually done for the change that was reverted yesterday, but |
| 14 |
> it seems like the CI results were ignored and the commit was pushed |
| 15 |
> regardless. |
| 16 |
|
| 17 |
Yesterday the original pull requests by Mo did not remove ebuilds. It |
| 18 |
was only when I started to adopt the PR that I saw |
| 19 |
|
| 20 |
> RepoMan scours the neighborhood... |
| 21 |
> repo.eapi-deprecated 1 |
| 22 |
> virtual/cblas/cblas-1.0.ebuild: 5 |
| 23 |
|
| 24 |
after which I impulsively killed it. |
| 25 |
|
| 26 |
|
| 27 |
I should take this lesson and be more careful removing ebuilds. |
| 28 |
|
| 29 |
Yours, |
| 30 |
Benda |
Archives