If that's the case then it may make the most sense to squash everything into a single commit.
On 3/31/19 2:17 PM, Jon Evans wrote: > That one was very late and would be easy to squash. However, some of the > very early commits in the branch were split up for review purposes and > not intended to be built on their own. > > On Sun, Mar 31, 2019, 14:16 Wayne Stambaugh <stambau...@gmail.com > <mailto:stambau...@gmail.com>> wrote: > > When was the code that did not build on msvc introduced? If was early > then it might make sense to squash everything. Otherwise, it may be > worthwhile squashing from the commit where the build error was > introduced to and including the commit where the build error was fixed. > I don't have a preference one way or the other. I'm open to > suggestion. > > On 3/31/19 11:14 AM, Jon Evans wrote: > > Seth, I split up the initial rebased branch to make review a bit > > easier. I could squash everything into one huge commit if you'd > prefer. > > > > On Sun, Mar 31, 2019 at 9:56 AM Seth Hillbrand <s...@hillbrand.org > <mailto:s...@hillbrand.org> > > <mailto:s...@hillbrand.org <mailto:s...@hillbrand.org>>> wrote: > > > > Am 2019-03-30 17:33, schrieb Simon Richter: > > > Hi Wayne, > > > > > > On 30.03.19 21:30, Wayne Stambaugh wrote: > > > > > >> Is this the last of it? If so, I will attempt to get this > merged > > >> tomorrow. > > > > > > Compiles fine on msys2 and msvc. Not all intermediate > commits compile, > > > but I'm not sure anyone does git-bisect anyway. > > > > I use git bisect frequently. Please do not push commits that > do not > > compile. This has been done previously and it added hours to > fixing a > > single bug in 5.0.1. > > > > -S > > > > _______________________________________________ > > Mailing list: https://launchpad.net/~kicad-developers > > Post to : kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net> > > <mailto:kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net>> > > Unsubscribe : https://launchpad.net/~kicad-developers > > More help : https://help.launchpad.net/ListHelp > > > > > > _______________________________________________ > > Mailing list: https://launchpad.net/~kicad-developers > > Post to : kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net> > > Unsubscribe : https://launchpad.net/~kicad-developers > > More help : https://help.launchpad.net/ListHelp > > > > _______________________________________________ > Mailing list: https://launchpad.net/~kicad-developers > Post to : kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net> > Unsubscribe : https://launchpad.net/~kicad-developers > More help : https://help.launchpad.net/ListHelp > _______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : kicad-developers@lists.launchpad.net Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp