On Fri, May 29, 2015 at 09:18:11PM -0500, Robert Dailey wrote: > So I am working on trying to setup my environment (VM through Virtual Box) > to do some testing on this. You all have encouraged me to try the mailing > list review model. So I won't give up yet.
I am not sure you need a VM or Linux environment. Of course it will be helpful in case your tests do no pass on Linux (which they sometimes do due to some differences between the OSes). But until we actually run into that problem I do not see anything wrong developing your change purely on Windows. Since it seems you are more familiar with that platform I would even encourage you to do so. That reduces the toolset friction which you might experience in a new environment. Even if you run into the problem, that your tests do not pass on Linux, we might be able to solve that on the list. Have you seen the github pull request -> mailing list proxy thing[1]? If that helps you maybe you can test it and use it for your patch submission. I think nobody will be annoyed if we get some strange emails on the list during that testing phase since that might help more contributors to contribute. > In the meantime I'd like to ask, do we even need to add an option for this? > What if we just make `diff.submodule log` not use --first-parent? This seems > like a backward compatible change in of itself. And it's simpler to > implement. I can't think of a good justification to add more settings to an > already hugely complex configuration scheme for such a minor difference in > behavior. > > Thoughts? This behavior has been with --first-parent for a long time. Even though it seems like a minor change in my experience there will be complaints from people that have got used to it and will now get big differences. You never know how people use it. Since the extra value (full-log) (AFAIR) has reached a consensus on the list and it allows us to satisfy both long log and short log users I would prefer to go that route. Cheers Heiko [1] http://article.gmane.org/gmane.comp.version-control.git/269699 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
