Skip to content

Commit

Permalink
SWUTILS-918: Improve the error output message when the directory lack…
Browse files Browse the repository at this point in the history 
…s write permissions.
  • Loading branch information
@KalepuAMD
KalepuAMD committed Oct 3, 2024
1 parent 4b49868 commit 1f4d955
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion sfreport.pl
View file
Original file line number Diff line number Diff line change
Expand Up @@ -3088,7 +3088,7 @@ sub check_x3_debug_info {
$out_file = *STDOUT{IO};
$out_format = format_text;
} else {
$out_file = new FileHandle($out_path, 'w') or die "open: $!";
$out_file = new FileHandle($out_path, 'w') or die "No write permissions on output directory.\nopen: $!";
if ($out_path =~ /\.html?$/) {
$out_format = format_html;
} else {
Expand Down

1 comment on commit 1f4d955

@KalepuAMD
Copy link
Contributor Author

@KalepuAMD KalepuAMD commented on 1f4d955 Oct 3, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The current output is:

server: sfreport$ sudo perl sfreport.pl 
AMD Solarflare system report (version 4.16.1)
No write permissions on output directory.
open: Permission denied at sfreport.pl line 3091.

Please sign in to comment.