fix spelling in tools

[bipinct]

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

Affected core subsystem(s)

[gireeshpunathil]

The changes look good, I am not 100% sure whether these files come from upstream google or we maintain it ourselves. thoughts?

[joyeecheung]

@gireeshpunathil Judging from the git log I think we maintain our own versions of gyp and icu, should be OK to modify them. CC @refack @srl295 @bnoordhuis

[gibfahn]

Same comments as nodejs/node-gyp#1315 (comment) regarding commit username and email.

[refack]

Both tools have an upstream "home" and we strive to minimize the number of "floating" patches we maintain on top of those:

• GYP - https://gyp.gsrc.io/docs/Hacking.md
• ICU - http://site.icu-project.org/processes/contribute

[gibfahn]

Both tools have an upstream "home" and we strive to minimize the number of "floating" patches we maintain on top of those:

Ahh, good to know. In that case @Bipin000 could you upstream this to https://gyp.gsrc.io/ first ?

[gibfahn]

Making what I said in #16633 (comment) explicit (before someone lands this 😁 ).

This should be upstreamed to GYP first.

[BridgeAR]

As this is only in the comments I do not see the necessity to land this. Instead I think there should be PRs upstream to fix this and we should just depend on that and not worry otherwise.

@Bipin000 that does not mean I do not appreciate your work! Thanks a lot for fixing it, I just think it is the wrong spot to do so.

[bipinct]

@BridgeAR Thanks!!! I have pushed changes to GYP .