Re: [Powerpc-utils-devel] [PATCH v2] Powerpc-utils: Restrict the usage of snap on RHEL 7 onwards

[Nathan F. <nf...@li...>]

Applied.

-Nathan

On 02/25/2015 08:52 AM, Henish Patel wrote:
> snap is not supported on the RHEL 7 onwards, because we have integrated snap
> functionality in sosreport. So this patch displays a suitable message.
> RHEL 7+ users can use sosreport to collect log data.
> 
> Signed-off-by: Henish Patel <hp...@li...>
> ---
>  scripts/snap |   14 +++++++++++++-
>  1 file changed, 13 insertions(+), 1 deletion(-)
> 
> diff --git a/scripts/snap b/scripts/snap
> index a3eb8c7..dc8abe1 100755
> --- a/scripts/snap
> +++ b/scripts/snap
> @@ -34,7 +34,19 @@ my $outfile = "snap.tar.gz";		# in the working dir.
>  my $cmddir = "snap_commands";		# cmd output dir.
>  my $cmdoutdir = "$outdir/$cmddir";	# in outdir dir.
>  my $rsxx_exists = 0;			# Does an IBM Flash Adapter exist?
> -my $distro_file = "/etc/issue"
> +my $distro_file = "/etc/issue";
> +my $redhat_release_file = "/etc/redhat-release";
> +
> +if (-e $redhat_release_file) {
> +	open(RELEASE, "< $redhat_release_file") or die "open: $!\n";
> +	$_ = <RELEASE>;
> +	my $redhat_version = (split / /, $_)[6];
> +	if ($redhat_version >= 7.0) {
> +		print "snap is not supported on the RHEL 7 onwards..!\n";
> +		print "Please use sosreport to collect log data..!! \n";
> +		exit 1;
> +	}
> +}
> 
>  our($opt_a, $opt_d, $opt_h, $opt_o, $opt_t, $opt_v);
> 
> 
> 
> ------------------------------------------------------------------------------
> Dive into the World of Parallel Programming The Go Parallel Website, sponsored
> by Intel and developed in partnership with Slashdot Media, is your hub for all
> things parallel software development, from weekly thought leadership blogs to
> news, videos, case studies, tutorials and more. Take a look and join the 
> conversation now. http://goparallel.sourceforge.net/
> _______________________________________________
> Powerpc-utils-devel mailing list
> Pow...@li...
> https://lists.sourceforge.net/lists/listinfo/powerpc-utils-devel
>