← Back to team overview

kicad-developers team mailing list archive

Re: Bus upgrades merge

 

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 <seth@xxxxxxxxxxxxx
> <mailto:seth@xxxxxxxxxxxxx>> 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@xxxxxxxxxxxxxxxxxxx
>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>     Unsubscribe : https://launchpad.net/~kicad-developers
>     More help   : https://help.launchpad.net/ListHelp
> 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
> 


Follow ups

References