fstream-npm: do not rename *.gyp twice

[indutny]

trafficstars

Do not rename multiple *.gyp files to binding.gyp . As a side-effect, no .gyp files will be renamed if binding.gyp is present.

Fix: https://github.com/npm/npm/issues/8411

[indutny]

cc @othiym23 I'm not exactly sure if this kind of sort function is acceptable. Other solution may be to set this _renameGyp flag from the sort, if it will see binding.gyp. However, it seems strange too.

Please let me know what you think about it.

[kenany]

Does this accomplish the same thing as https://github.com/npm/fstream-npm/pull/16 does?

[indutny]

No, @KenanY. It is a slightly more compatible solution, IMO. It will still rename .gyp files, but only if there are no binding.gyp, and only the first one of .gyp in any case.

[kenany]

Ah okay :)

[mikemorris]

Dupe of https://github.com/npm/fstream-npm/pull/16?

[indutny]

@mikemorris they are different, see https://github.com/npm/fstream-npm/pull/18#issuecomment-189067408