I found a typo in the rebuild function name. Going to try rolling a patch to fix it & call to it (my second time around on patching evar!)

Comments

hongpong’s picture

Status: Active » Needs review
StatusFileSize
new2.63 KB

Alright they're kicking us out of code sprint now, but I think this covers what needs to be done. I forgot one ref... but yeah it looks like the 2nd patch has all of em. my 2nd & 3rd patches respectively. thnx for yr review

hongpong’s picture

StatusFileSize
new1.87 KB

this was the first one. not needed i think. (it's missing the last fix).

dawehner’s picture

Status: Needs review » Fixed

Thanks again!

Commited, added a changelog entry and added you to the commit message.

git commit --amend really rocks :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

jeffschuler’s picture

Status: Closed (fixed) » Needs review
StatusFileSize
new459 bytes

Looks like the patch from #2 was committed instead of the complete solution in #1, because one of these typos still remains.

This patch fixes that one.

hefox’s picture

Status: Needs review » Reviewed & tested by the community

Works and fixes the issue

dawehner’s picture

Status: Reviewed & tested by the community » Fixed

Oh totally forgot about this patch, thanks for it!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.