[LinuxCNC/linuxcnc PR#477] configure.in: require `yapps` executable

#1 – SebKuzminsky 于 2018-07-31

The commit message for bf985533a99e1 is wrong (talks about intltool), but the patch is right (checks for yapps). Please fix the commit message and I’ll merge it.

#2 – SebKuzminsky 于 2018-07-31

I merged #476 into 2.7 and merged 2.7 into master, so if you rebase bf98553 onto master you won’t need the merge in this PR, simplifying it some.

#3 – zultron 于 2018-07-31

D’oh! I’ve been making a bunch of embarrassing copy and paste mistakes lately.

I see that #475 was merged, but I don’t see it in master.

Anyway, the PR is fixed. Thanks, @SebKuzminsky.

#4 – SebKuzminsky 于 2018-07-31

Oops, you’re right, I didn’t merge #476 into master, it’s just in 2.7. Derp. I’ll fix it.

#5 – zultron 于 2018-08-03

Jeezus H. Beezus, @SebKuzminsky you caught the commit with the wrong name, but missed the PR misnamed “configure.in: require yaml2 executable”! I can’t use the “copy’n’paste” excuse for that one.

Fixed PR title.