Re: [PATCH v2] treewide: remove CONFIG_DEBUG_INFO from defconfigs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 20, 2019 at 10:54:21AM +0200, Sam Ravnborg wrote:
> Hi Antony
> 
> On Thu, Jun 20, 2019 at 11:25:09AM +0300, Antony Pavlov wrote:
> > The commit b917f7864115a35 ("remove CONFIG_DEBUG_INFO")
> > has dropped Kconfig DEBUG_INFO option however we
> > still have very many DEBUG_INFO mentions in defconfig
> > files. Drop them using sed:
> > 
> >   find -iname '*defconfig' -type f -exec \
> >       sed -i "/CONFIG_DEBUG_INFO=y/d" {} ';'
> Another approach could be to regenerate all defconfigs.
> A little scripting around savedefconfig should do it.
> 
> Then we would get rid of all obsolete symbols in one go - for all
> defconfigs.
> 
> The generated defconfig would be the one that people would be using
> anyway, so there should be only a little risk to introduce new issues
> doing it this way.
> 
> Willing to try this approcah - maybe across all architectures?

I once thought the same and came up with this little thing:

#!/bin/bash

for a in arch/*; do
        arch=$(basename $a)
        for c in $a/configs/*; do
                config=$(basename $c)
		export ARCH=$arch
		make $config && make savedefconfig && mv defconfig $c
        done
done

Just tested, still works. Maybe I should just run this from time to time
and commit the result. What do you think?

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox



[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux