← Back to team overview

kicad-developers team mailing list archive

Re: Rebase the tree?

 

Perhaps the git commit hook can include a check for file size and
complain if larger than some-arbitrary-threshold?

On Tue, Jan 21, 2020 at 12:43 AM Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
>
> No problem.  Hopefully it wont cause too many issues for other devs who
> have to rebase any local changes.
>
> Cheers,
>
> Wayne
>
> On 1/20/20 6:15 PM, Jeff Young wrote:
> > Thanks, Wayne!
> >
> > Sorry for causing such a mess.
> >
> > Cheers,
> > Jeff.
> >
> >
> >> On 20 Jan 2020, at 22:32, Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
> >>
> >> I found it.  I had to create a new protection for the master branch.  I
> >> pushed the changes and enabled the GitLab protection for the master
> >> branch so we should be good to resume normal development.  Thank you for
> >> the help and patience to work through this.
> >>
> >> Cheers,
> >>
> >> Wayne
> >>
> >> On 1/20/20 5:27 PM, Nick Østergaard wrote:
> >>> It should be there, I am sure you are just confused by the gitlab webui.
> >>>
> >>> It should be something along the lines of:
> >>> Settings -> Repository -> Protected branches, click expand. Set the
> >>> proper settings in the "Protect a branch" section.
> >>>
> >>> On Mon, 20 Jan 2020 at 23:26, Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
> >>>>
> >>>> Well this is a kick in the teeth.  I just unprotected it and I don't see
> >>>> an option to re-enable the protection after I force the changes.
> >>>>
> >>>> On 1/20/20 5:18 PM, Nick Østergaard wrote:
> >>>>> There is a "protected branches" section in the settings of the repo.
> >>>>>
> >>>>> On Mon, 20 Jan 2020 at 23:18, Nick Østergaard <oe.nick@xxxxxxxxx> wrote:
> >>>>>>
> >>>>>> You probably need to disable the option to disable force pushing on the repo.
> >>>>>>
> >>>>>> On Mon, 20 Jan 2020 at 23:20, Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
> >>>>>>>
> >>>>>>> GitLab rejected the forced push using this method.  Anyone else have any
> >>>>>>> ideas.  Until we get this resolved, please do not push any commits to
> >>>>>>> the master branch.
> >>>>>>>
> >>>>>>> Thanks,
> >>>>>>>
> >>>>>>> Wayne
> >>>>>>>
> >>>>>>> On 1/20/20 5:13 PM, Wayne Stambaugh wrote:
> >>>>>>>> Please do not push anything to the master branch or perform any merge
> >>>>>>>> requests until I push the rebase the master branch to prevent any commit
> >>>>>>>> losses because forcing a push will wipe out any changes.  I ran the git
> >>>>>>>> command suggested by Simon and it seems to have the desired results but
> >>>>>>>> I have no idea how this is going to play out so I'm making a backup
> >>>>>>>> clone of master just in case things go sideways.  I'll ping everyone
> >>>>>>>> once I have pushed the rebase.  Thank you for your cooperation and I
> >>>>>>>> apologize for whatever pain and agony this causes.  Hopefully we will
> >>>>>>>> never make this mistake again.
> >>>>>>>>
> >>>>>>>> Cheers,
> >>>>>>>>
> >>>>>>>> Wayne
> >>>>>>>>
> >>>>>>>> On 1/18/20 7:29 AM, Simon Richter wrote:
> >>>>>>>>> Hi,
> >>>>>>>>>
> >>>>>>>>> On 17.01.20 19:14, Simon Richter wrote:
> >>>>>>>>>
> >>>>>>>>>> 1. "git rebase -i origin/master"
> >>>>>>>>>> 2. in the editor, if they are present, remove the lines
> >>>>>>>>>>
> >>>>>>>>>>    pick ea31730b4 Handle error returns from lstat.
> >>>>>>>>>>    pick e83420f19 Remove file accidentally commited in ea31730b4
> >>>>>>>>>>    pick e27e6ee16 Also catch null dereference in case wxASSERT was skipped.
> >>>>>>>>>>    pick e1925b89c Remove file accidentally added in e27e6ee1
> >>>>>>>>>>
> >>>>>>>>>> 3. save and exit
> >>>>>>>>>
> >>>>>>>>> Even less interactive:
> >>>>>>>>>
> >>>>>>>>>    git filter-branch \
> >>>>>>>>>        --prune-empty \
> >>>>>>>>>        --index-filter \
> >>>>>>>>>            'git rm --cached --ignore-unmatch common/libcommon.a.*' \
> >>>>>>>>>        9df2cfb32..HEAD
> >>>>>>>>>
> >>>>>>>>> This rewrites the current branch to a state where the files were never
> >>>>>>>>> added, and removes the now-empty correction commits. The SHA1 sums in
> >>>>>>>>> the new branch are different, but as the commit contents are identical,
> >>>>>>>>> rebasing feature branches then goes smoothly even from the gitlab GUI.
> >>>>>>>>>
> >>>>>>>>> If you rebased a branch containing the offending commits on top of a
> >>>>>>>>> cleaned one, this generates four commits adding and removing the files
> >>>>>>>>> with no further changes, and the filter-branch commit above then reduces
> >>>>>>>>> these to no-ops and removes the commits.
> >>>>>>>>>
> >>>>>>>>> A simple test in gitlab *merge request is descended from e1925b89c"
> >>>>>>>>> could identify merge requests that would need to be rewritten.
> >>>>>>>>>
> >>>>>>>>>   Simon
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> _______________________________________________
> >>>>>>>>> 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
> >>>>>>>>>
> >>>>>>>
> >>>>>>> _______________________________________________
> >>>>>>> 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
> >>
> >> _______________________________________________
> >> 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
> >
>
> _______________________________________________
> 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



-- 
/Jonatan
http://kymatica.com


References