1 |
I've been using both repoman _and_ pkgcheck becasue I was not sure which |
2 |
one covers all the checks I need to make. In fact [Pull Requests |
3 |
wiki](https://wiki.gentoo.org/wiki/Project:GitHub/Pull_requests) has a |
4 |
big red warning box with the following message, and I quote: |
5 |
|
6 |
"CI is not an excuse not to run repoman full, at least for the packages |
7 |
being committed." |
8 |
|
9 |
[Package maintainer's responsibilities |
10 |
wiki](https://wiki.gentoo.org/wiki/Package_maintainer%27s_responsibilities) |
11 |
says "When committing changes, repoman commit **or** pkgdev commit (from the |
12 |
dev-util/pkgdev package) can be used.", emphasis mine. |
13 |
|
14 |
I use documentation from time to time to remind myself of details, as I |
15 |
want to avoid bugging people on IRC or the mailing list, and to provide |
16 |
up to date and clean PRs. Per said documentation repoman is (was?) |
17 |
a cornerstone of the workflow, as mentioned by others too. Developers |
18 |
should not be frowned upon for using Repoman, as they were doing |
19 |
exactly what was asked of them. |
20 |
|
21 |
Additionally based on the existing documentation it was not evident |
22 |
that, pkgcheck covers all checks done by repoman and more, or that it's |
23 |
faster, or that it has advanced features, like scanning only commits since |
24 |
HEAD/origin (I myself was only ever using it in the ebuild's directory, |
25 |
because Repoman is slow when running over more than one ebuild). I've |
26 |
had to read most of this thread to obtain all these details and to learn |
27 |
about the proper replacements for all the steps in my workflow. |
28 |
|
29 |
Today I've unmerged repoman and I've used only pkgdev, pkgcheck, and git |
30 |
to create a new PR to force myself off Repoman. So far I think the |
31 |
experience was superior to the Repoman workflow. I like that I can run |
32 |
pkgcheck at the end before the push only on the commits and I don't have |
33 |
to worry about Repoman trying to scan the entire tree. I think `pkgdev |
34 |
commit` is quite sufficient, not sure why would one need to use |
35 |
pkgcommit. |
36 |
|
37 |
I found that mgorny's dev scripts package has a lot of irrelevant stuff |
38 |
I don't need on my system (maybe not even applicable to anyone outside |
39 |
of mgorny), I'd prefer we don't make it part of the mandated workflow as |
40 |
is. Maybe if the generally useful scripts are separated out to some |
41 |
gentoo devtoolkit package or similar, then it's okay. |
42 |
|
43 |
Summa summarum, I think deprecating repoman is the right direction, |
44 |
but documentation needs to be updated comprehensively before it's done. |
45 |
|
46 |
Just a proxy maintainer's 2¢. |
47 |
|
48 |
Zoltan |
49 |
|
50 |
P.S.: I'm with Joshua about the snarkiness of the tooling, I liked that |
51 |
a lot and it's a bit sad that the new tools are a bit sterile. |
52 |
|
53 |
On Wed, Mar 09, 2022 at 04:25:17PM -0500, Ionen Wolkens wrote: |
54 |
> On Wed, Mar 09, 2022 at 01:00:37PM -0800, Matt Turner wrote: |
55 |
> > I'd like to deprecate and ultimately remove repoman. I believe that |
56 |
> > dev-util/pkgcheck and pkgcommit (from app-portage/mgorny-dev-scripts) |
57 |
> |
58 |
> There's also dev-util/pkgdev as an alternative to pkgcommit with more |
59 |
> features for those that want them that will feel similar to repoman |
60 |
> - pkgdev commit -c/-b bugnum (repoman commit -c/-b bugnum) |
61 |
> - pkgdev manifest (repoman manifest) |
62 |
> - pkgdev push (runs pkgcheck scan --commits before pushing) |
63 |
> - updates copyright year / manifest on commit |
64 |
> - basic commit message templates, like auto "cat/pkg: add <version>" on |
65 |
> bumps without manual editing -- prepend cat/pkg: like other tools |
66 |
> if nothing specific |
67 |
> |
68 |
> And other handy features like pkgdev mask --rites days |
69 |
> |
70 |
> > are far superior replacements, and it makes sense to have people using |
71 |
> > the same tool and seeing the same warnings as in the CI. |
72 |
> > |
73 |
> > Are there any useful checks or behaviors of repoman that are missing |
74 |
> > from pkgcheck and pkgcommit? |
75 |
> > |
76 |
> > Thanks, |
77 |
> > Matt |
78 |
> > |
79 |
> |
80 |
> -- |
81 |
> ionen |