Hi! On Mon, Dec 16, 2019 at 03:13:49PM -0700, Jeff Law wrote: > On Mon, 2019-12-16 at 15:59 -0600, Segher Boessenkool wrote: > > In particular, we should look carefully at the commit > > > attributions in both conversions and Maxim's may well give ideas for > > > improving the reposurgeon changelogs command (Richard came up with three > > > ideas recently, which I've just filed in the reposurgeon issue tracker). > > > But I also think: > > > > > > * reposurgeon is a safer approach than ad hoc scripts, provided we get > > > clean verification of basic properties such as branch tip contents. > > > > I totally do not agree. Black boxes are not safe. *New* black boxes > > are even worse. > > > > I trust scripts that have low internal complexity much better. > Perhaps. But there's also limits to what scripts can do.
Absolutely. Just this trust in complicated things is very misplaced, in my opinion. > > There is absolutely no reason to trust a system that supposedly was > > already very mature, but that required lots of complex modifications, > > and even a complete rewrite in a different language, that even has its > > own bug tracker, to work without problems (although we all have *seen* > > some of its many problems over the last years), and at the same time > > bad-mouthing simple scripts that simply work, and have simple problems. > I'd disagree. THe experience and testsuites from that system are a > significant benefit. That isn't what I said. I said that freshly constructed complex software will have more and deeper errors than stupid simple scripts do (or I implied that at least, maybe it wasn't clear). And I only say this because the opposite was claimed, which is laughable imnsho. > > > * Richard's improvements to commit messages are a great improvement to > > > the > > > resulting repository (and it's OK if a small percentage end up misleading > > > because someone used the wrong PR number, sometimes people use the wrong > > > commit message or commit changes they didn't mean to and so having some > > > misleading messages is unavoidable). > > > > As long as the original commit message is kept, verbatim, and you only > > add a new summary line, all is fine. If not -> nope, not okay. > Sorry, have to disagree here. I think what Richard has done is a > significant step forward. We talked about it for days, and as far as I understand it Richard agreed. But, there is no way I can verify this yet, or is there? Is there a repo we can look at? Something close to final. > > > * As we're part of the free software community as a whole rather than > > > something in isolation, choosing to make a general-purpose tool work for > > > our conversion is somewhat preferable to choosing an ad hoc approach > > > because it contributes something of value for other repository > > > conversions > > > by other projects in future. > > > > This, I don't agree with at all either: having some lean-and-mean > > scripts that worked for the GCC conversion will be at least as helpful > > for another conversion as a "general" tool that first requires you to > > build a custom machine before you can use it properly, would be. > > > > > > Anyway: yes, please verify all conversion candidates for your criteria. > > Thanks! > And if they're the same, then I'm still going to prefer reposurgeon. But they aren't the same, so you will have to make an actual choice, based on actual data ;-) > So unless there's something Maxim's scripts are getting right that > aren't by reposurgeon, then reposurgeon is the right choice. ... in your opinion. Anyway, I'd love too hear Richard's input too, but we will have to wait for that a few days. The quality of the conversion should be judged by the output of it, not by anything else. You do not want to see how sausage is made, as they say. The GCC community has nothing to gain from a generic conversion tool: what we want and need is *one* conversion, and it should be a good one. So let us compare *that*! Segher