[Git][security-tracker-team/security-tracker][master] 3 commits: Improve check-new-issues for new users

[Salvatore Bonaccorso (@carnil)]

Salvatore Bonaccorso pushed to branch master at Debian Security Tracker / security-tracker


Commits:
600a9d35 by Neil Williams at 2021-11-19T10:56:07+00:00
Improve check-new-issues for new users

Print the supported commands before entering interactive mode.
Skip if only listing the CVEs

- - - - -
b49b16f9 by Neil Williams at 2021-11-19T12:47:28+00:00
check-new-issues: implement review comments

Move the print statements inside the conditional.

- - - - -
246d8aed by Salvatore Bonaccorso at 2021-11-19T12:57:19+00:00
Merge branch 'newissues' into 'master'

Improve check-new-issues for new users

See merge request security-tracker-team/security-tracker!94
- - - - -


1 changed file:

- bin/check-new-issues


Changes:

=====================================
bin/check-new-issues
=====================================
@@ -12,13 +12,14 @@ getopts('ln:fhi:t:Tca:e:uUsDb', \%opts);
 sub print_commands {
     print <<'EOF';
   * blank line to skip to next issue
+  * .h to repeat this help output of the list of commands
   * .fname to do "apt-file search name"
   * .cname to do "apt-cache search name"
   * .wname to look up name in wnpp
   * .mpackage to search data/embedded-code-copies for "package"
   * .rpackage to launch an editor with a report of the issue against "package"
   * .gissue to go to the given issue, even if it's not a todo
-  * d to display again the issue information
+  * d to display the issue information again
   * !command to execute a command with system() without any escaping
   * v or e to launch an editor with the current item
   * - package-entry to add an entry for "package" and launch an editor (e.g. - poppler <unfixed>)
@@ -30,7 +31,7 @@ EOF
 
 if ($opts{h}) {
     print <<'EOF';
-downloads allitems.txt from cve.mitre.org and shows full decription for each
+downloads allitems.txt from cve.mitre.org and shows full description for each
 "TODO: check" item (2003 and newer). Then
 
 - tries to guess product name and php filename and does 
@@ -164,7 +165,7 @@ foreach my $entry (@{$entries}) {
 		$name=$1;
 	}
 	else {
-		die "invlid entry:\n$entry";
+		die "invalid entry:\n$entry";
 	}
 	if (!$opts{l} && $entry =~ /^\s+-\s+([^\s]+)/m ) {
 	    my $pkg = $1;
@@ -198,6 +199,12 @@ print ", $num_todo todos matching /$todo_regexp/" if $num_todo;
 print ", $num_missing_bug entries with missing bug reference" if $num_missing_bug;
 print "\n";
 
+if ((! $opts{l}) and (! $opts{b})) {
+    print "\nCommands:\n";
+    print_commands;
+    print "\n";
+}
+
 if ($opts{l}) {
 	#list only
 	foreach my $todo (sort {$b <=> $a} @todos) {



View it on GitLab: https://salsa.debian.org/security-tracker-team/security-tracker/-/compare/5504511792f40b57b9995ffa0d3ecde7650bd31e...246d8aedb9175ce3715eab0ad8afef74fe21df31

-- 
View it on GitLab: https://salsa.debian.org/security-tracker-team/security-tracker/-/compare/5504511792f40b57b9995ffa0d3ecde7650bd31e...246d8aedb9175ce3715eab0ad8afef74fe21df31
You're receiving this email because of your account on salsa.debian.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/debian-security-tracker-commits/attachments/20211119/a42b4db7/attachment-0001.htm>