Re: Another patch

Gabor Z. Papp ([email protected]) wrote:
: [email protected] wrote:

: | A very silly bug, and some log level butchering.

: So summary there are 2 diffs after 0.3.1?

Basically yes.

-harold

diff -u /tmp/sn-0.3.1/CHANGES ./CHANGES
--- /tmp/sn-0.3.1/CHANGES	Fri Oct 29 11:24:26 1999
+++ ./CHANGES	Tue Dec  7 10:46:44 1999
@@ -1,3 +1,7 @@
+Bug in command parsing causes slrn grief
+More log levels tweaked
+Format bugs in snsend.
+--
 0.3.1 Fri Oct 29 11:24:12 SGT 1999
 Added -v option to snexpire.
 Bad logic when checking to map in newsgroup.c.
diff -u /tmp/sn-0.3.1/Makefile ./Makefile
--- /tmp/sn-0.3.1/Makefile	Fri Oct 29 11:19:26 1999
+++ ./Makefile	Fri Nov  5 10:32:33 1999
@@ -141,7 +141,7 @@
 
 _dep: Makefile cc-flags
 	(sed '/^## DO NOT REMOVE ##/q' Makefile; echo; for i in $(OBJS); \
-	do cc -MM -MG `cat cc-flags` `basename $$i .o`.c; done) >.dep
+	do $(CC) -MM -MG `cat cc-flags` `basename $$i .o`.c; done) >.dep
 	mv .dep Makefile
 
 ## DO NOT REMOVE ##
diff -u /tmp/sn-0.3.1/README ./README
--- /tmp/sn-0.3.1/README	Tue Aug 10 12:00:42 1999
+++ ./README	Sun Dec  5 12:01:36 1999
@@ -82,6 +82,10 @@
 previous version.  Otherwise read file INSTALL, then INSTALL.run,
 then INSTALL.notes.
 
[email protected] (Patrik Radman) has set up a mailing list for
+sn, [email protected], administered by [email protected].
+Thanks Patrik.
+
 License
 -------
 Distribution of sn is covered by the GPL.  See file COPYING.
Common subdirectories: /tmp/sn-0.3.1/lib and ./lib
diff -u /tmp/sn-0.3.1/list.c ./list.c
--- /tmp/sn-0.3.1/list.c	Tue Aug 10 12:11:08 1999
+++ ./list.c	Tue Nov 16 23:55:12 1999
@@ -22,7 +22,7 @@
 #include <out.h>
 #include <wildmat.h>
 
-static const char rcsid[] = "$Id: list.c,v 1.19 1999/08/10 04:11:08 harold Exp $";
+static const char rcsid[] = "$Id: list.c,v 1.20 1999/11/16 15:55:12 harold Exp $";
 
 /* Same as LIST but may have an optional wildmat */
 
@@ -38,7 +38,7 @@
 
   for_each_key(i, j, kp){
     group = KEY(kp);
-    if( ! args[2] || wildmat(group, args[2]) )
+    if( ! args[1] || ! args[2] || wildmat(group, args[2]) )
       if( 0 == group_info(group, &g) )
         args_write(1, "%s %d %d %s\r\n",
          group, g.last, g.first, g.nopost?"n":"y");
diff -u /tmp/sn-0.3.1/snsend.c ./snsend.c
--- /tmp/sn-0.3.1/snsend.c	Mon Oct 25 01:03:47 1999
+++ ./snsend.c	Tue Dec  7 10:35:47 1999
@@ -110,7 +110,7 @@
     strcpy(path2, path1);
     *strrchr(path2, '+') = '$';
     if( -1 == link(path1, path2) && EEXIST != errno ){
-      er = -1; log("store_global:link(%s,%s):%m"); }
+      er = -1; log("store_global:link(%s,%s):%m", path1, path2); }
   }else
     log("store_global:write(%s):%m", path1);
 
@@ -175,11 +175,11 @@
   if( WIFEXITED(s) ){
     if( 0 == WEXITSTATUS(s) )return(0);
     s = WEXITSTATUS(s);
-    fail(s, "%s/.outgoing exited with %d", s);
+    fail(s, "%s/.outgoing exited with %d", s, s);
   }
   if( WIFSIGNALED(s) )
     fail(WTERMSIG(s)+128, "%s/.outgoing caught signal %d", WTERMSIG(s));
-  fail(9, "%s/.outgoing unknown wait status %d", s);
+  fail(9, "%s/.outgoing unknown wait status %d", group, s);
   _exit(1); /* dumb compiler */
 }
 static int store_special(char * group) { return(special(group, 0)); }