OSGeo / libgeotiff

Official repository of the libgeotiff project
182 stars 70 forks source link

Fixing wrong eol #14

Closed tigerfoot closed 4 years ago

tigerfoot commented 5 years ago

In openSUSE packaging we used to have this bash part to fix utf-8 and wrong eol in source code.

for f in `find . -type f` ; do
        if file $f | grep -q ISO-8859 ; then
                echo "Fix encoding for $f"
                set -x
                iconv -f ISO-8859-1 -t UTF-8 $f > ${f}.tmp && \
                        mv -f ${f}.tmp $f
                set +x
        fi
        if file $f | grep -q CRLF ; then
                echo "Fix line ends for $f"
                set -x
                sed -i -e 's|\r||g' $f
                set +x
        fi
done

While checking 1.5.0 future release I picked still some files that should be fixed

Fix line ends for ./cmake/FindGeoTIFF.cmake
+ sed -i -e 's|\r||g' ./cmake/FindGeoTIFF.cmake

Fix line ends for ./cmake/FindPROJ.cmake
+ sed -i -e 's|\r||g' ./cmake/FindPROJ.cmake

Fix line ends for ./cmake/geo_config.h.in
+ sed -i -e 's|\r||g' ./cmake/geo_config.h.in

Fix line ends for ./bin/CMakeLists.txt
+ sed -i -e 's|\r||g' ./bin/CMakeLists.txt

Fix line ends for ./libxtiff/CMakeLists.txt
+ sed -i -e 's|\r||g' ./libxtiff/CMakeLists.txt

If PR are allowed now here directly I should be able to send it.

rouault commented 5 years ago

PR welcome