Re: [PATCH] scripts: Translate profile2linkerlist.

From	Michal Marek <mmarek@suse.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] scripts: Translate profile2linkerlist.
Date	2016-08-07 22:20 +0200
Message-ID	<s3CI9-8tY-3@gated-at.bofh.it> (permalink)
References	<s3AZH-7jt-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Dne 7.8.2016 v 20:33 Jorge Natz napsal(a):
> -while (<>) {
> -  my $line = $_;
> -
> -  $_ =~ /\W*[0-9]+\W*([a-zA-Z\_0-9]+)\W*[0-9]+/;
> -
> -  print "*(.text.$1)\n"
[...]
> +	LINE=$( echo $LINE | sed -e s"/[0-9.][0-9.]*//" -e s"/[0-9.][0-9.]*$//" | xargs)
> +	case $LINE in *unknown*|*total* ) continue
> +	;;
> +	esac
> +	echo '*(.text.'$LINE')'
> +done

The two regular expressions are not equivalent and the shell version
does not take into account the locale-dependent decimal separator. It
also does not handle localized "total", but that's an issue of the perl
version as well. Since this script is not used during kernel build, its
dependency on perl is less of an issue and it can be left as is.

Michal

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] scripts: Translate profile2linkerlist. Jorge Natz <jorgenatzdev@gmail.com> - 2016-08-07 20:30 +0200
  Re: [PATCH] scripts: Translate profile2linkerlist. Michal Marek <mmarek@suse.com> - 2016-08-07 22:20 +0200

csiph-web