msys2 / msys2-pacman

A friendly fork of https://gitlab.archlinux.org/pacman/pacman
GNU General Public License v2.0
21 stars 12 forks source link

makepkg doesn't detect reference to (`cygpath -m /`|`cygpath -w /`) in *.cmake files. #14

Closed MehdiChinoune closed 2 years ago

jeremyd2019 commented 2 years ago

Please attach an example of such a file that was not detected. The only thing I can figure is that grep thinks it's binary. Maybe the files are encoded in UTF-16 or something.

MehdiChinoune commented 2 years ago

I have encountered this issue with mingw-w64-itk package after trying to comment the part where it fixes *.cmake files. https://github.com/MehdiChinoune/MINGW-packages/commit/281ba3f9dbb1f9dbe1c878cf665525fbce30319e https://github.com/MehdiChinoune/MINGW-packages/runs/5673705864?check_suite_focus=true#step:9:6974

MehdiChinoune commented 2 years ago
$ file -i ITK/*.cmake
ITK/ITKConfig.cmake:          text/plain; charset=us-ascii
ITK/ITKConfigVersion.cmake:   text/plain; charset=us-ascii
ITK/ITKModuleAPI.cmake:       text/plain; charset=us-ascii
ITK/ITKTargets-release.cmake: text/plain; charset=us-ascii
ITK/ITKTargets.cmake:         text/plain; charset=us-ascii
ITK/UseITK.cmake:             text/plain; charset=us-ascii