ZOSOpenTools / autoconfport

Delta files to build autoconf for z/OS
Apache License 2.0
1 stars 3 forks source link

checking tags across autoconf generated files reveals many as EBCDIC #9

Open MikeFultonDev opened 2 years ago

MikeFultonDev commented 2 years ago

Ideally, everything should be ASCII - this likely is highlighting a problem:

find . -name "*" | xargs chtag -p 2>/dev/null | grep IBM-1047
t IBM-1047    T=on  ./autoconf-2.71/bin/autoreconf
t IBM-1047    T=on  ./autoconf-2.71/bin/autom4te
t IBM-1047    T=on  ./autoconf-2.71/bin/autoconf
t IBM-1047    T=on  ./autoconf-2.71/bin/autoheader
t IBM-1047    T=on  ./autoconf-2.71/bin/autoupdate
t IBM-1047    T=on  ./autoconf-2.71/bin/autoscan
t IBM-1047    T=on  ./autoconf-2.71/bin/ifnames
t IBM-1047    T=on  ./autoconf-2.71/build-aux/config.guess
t IBM-1047    T=on  ./autoconf-2.71/lib/autoconf/general.m4
t IBM-1047    T=on  ./autoconf-2.71/lib/autoconf/status.m4
t IBM-1047    T=on  ./autoconf-2.71/lib/autoscan/autoscan.list
t IBM-1047    T=on  ./autoconf-2.71/lib/m4sugar/version.m4
t IBM-1047    T=on  ./autoconf-2.71/lib/autom4te.cfg
t IBM-1047    T=on  ./autoconf-2.71/tests/local.at
t IBM-1047    T=on  ./autoconf-2.71/tests/atlocal
t IBM-1047    T=on  ./autoconf-2.71/tests/atconfig
t IBM-1047    T=on  ./autoconf-2.71/tests/autoconf
t IBM-1047    T=on  ./autoconf-2.71/tests/autoheader
t IBM-1047    T=on  ./autoconf-2.71/tests/autom4te
t IBM-1047    T=on  ./autoconf-2.71/tests/autoreconf
t IBM-1047    T=on  ./autoconf-2.71/tests/autoscan
t IBM-1047    T=on  ./autoconf-2.71/tests/autoupdate
t IBM-1047    T=on  ./autoconf-2.71/tests/ifnames
t IBM-1047    T=on  ./autoconf-2.71/configure
t IBM-1047    T=on  ./autoconf-2.71/config.log
t IBM-1047    T=on  ./autoconf-2.71/config.status
t IBM-1047    T=on  ./autoconf-2.71/Makefile
MikeFultonDev commented 2 years ago

I think this stems from the fact that 'patch' will create an IBM-1047 tagged file even if the input files are ISO8859-1 and so that IBM-1047 tag ends up getting propagated. With the shift to use 'git diff' for patch management, this should 'fix itself'. @IgorTodorovskiIBM fyi...