Hi all,
I branched yesterday otopi-1.8 to be used for ovirt-4.3, and forgot to
patch stdci accordingly. Now pushed the patch [1] (to both master and
otopi-1.8 - I realize I do not need it in master, but hopefully this
will help me remember patching it in 4.4). What can/should I do to
include the otopi-1.8.1 build [2] in the 4.3 change queue?
And, related to this, some ideas about how to make stdci more useful
for mere developers:
1. Make the change-queue post comments in gerrit patches once it has a
result (patch passed change queue X, failed, something else?). I (as
always) prefer a bit more information (e.g. a few more comments that
might not be that interesting) than not enough (current). We can
always refine later. IIRC this was already discussed in the past - any
update?
2. Make the current comment "This change was successfully submitted to
the change queue(s) for system testing." more informative - include
names of change queues and links to relevant pages (builds? not sure,
I do not know enough CI).
3. As a side note, I realize that the option names were made
case-insensitive and ignore whitespace/dashes/underscores in order to
not impose a certain style on the many different projects/maintainers
and let them use what they find best for their project. I personally
think this was a mistake. If you have to make a choice when naming
something, and have a feeling that your opinion might cause
noise/objections/etc. in the future, make a quick poll asking what
people prefer, and then pick a single name. Having several (many)
different names for things makes it much harder later on to _find_
them. It might be too late now to change, because people might already
picked different names in practice and we do not want to break their
stuff, and that's a pity. But next time, be opinionated, or ask
beforehand - not keep the choice. Most computer languages and
libraries have single unique names for things, for a reason.
[1]
https://gerrit.ovirt.org/#/q/I794426ec9212e0a023c3e5f158d0a88fc8e6842c
[2]
https://gerrit.ovirt.org/98408
--
Didi