[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#809534: jessie-pu: package ttylog/0.26-1~deb8u1



Control: retitle -1 jessie-pu: package ttylog/0.26-1~deb8u1

Adam,

On Thursday, December 31, 2015 08:09:28 PM Adam D. Barratt wrote:
> 
> I see that you've already uploaded a package for this. I'm afraid that
> I'm going to have to reject that upload. For future reference, this is
> why the preferred process is for the diff to be approved /before/ the
> upload, rather than the discussion occurring afterwards.
> ....

  Very much appreciate you taking the time to explain all of that!

 
> The rest of the diff looks okay from a quick view. If you'd like to
> prepare and test a package that reverts the debhelper changes and is
> versioned as 0.26-1~deb8u1, please attach a new debdiff to this bug and
> I'd be happy to look at reviewing that.

Please see the attached debdiff-deb8u1.txt for the new debdiff  between the 
existing version ttylog_0.25-1 in jessie and the proposed update package 
version 0.26-1~deb8u1.




Regards,
  RJ Clay


diff -Nru ttylog-0.25/ChangeLog ttylog-0.26/ChangeLog
--- ttylog-0.25/ChangeLog	2013-02-14 15:25:52.000000000 -0500
+++ ttylog-0.26/ChangeLog	2015-06-03 13:32:14.000000000 -0400
@@ -1,3 +1,24 @@
+2015-6-03  Robert James Clay <jame@rocasa.us>  0.26
+
+    * ttylog.c: Correct truncating of modem_device string, implementing ttylog
+      patch #3 (thanks to Daniel Flinkmann). Closes Debian Bug #731086.
+    * ttylog.c: Rewrite the "-h|--help" option code, including the expansion
+      of the "Usage:" line.
+    * ttylog.c: Rewrite the '-v|--version' option code, including the addition
+      of the License and Copyright information.
+    * ChangeLog: Correct bad date entry for v0.1.b-3.
+
+
+2014-7-29  Robert James Clay <jame@rocasa.us>
+
+    * ttylog.8: Rewrite the '-f --flush' option description in ttylog.8.
+
+
+2013-6-19  Robert James Clay <jame@rocasa.us>
+
+    * README.md: Add initial version of the README.md file.
+
+
 2013-2-14  Robert James Clay <jame@rocasa.us>  0.25
 
     * ttylog.c: Corrections to the code adding a NULL to modem_device variable.
@@ -77,7 +98,7 @@
     * Fix nostrip build issue. (Closes: bug#438226)
 
 
-2008-11-2002  Tibor Koleszar <oldw@debian.org>  0.1.b-3
+2002-11-04  Tibor Koleszar <oldw@debian.org>  0.1.b-3
 
     * Using optional output flush - Closes: Debian bug#164778
 
diff -Nru ttylog-0.25/COPYRIGHT ttylog-0.26/COPYRIGHT
--- ttylog-0.25/COPYRIGHT	2013-02-14 15:25:52.000000000 -0500
+++ ttylog-0.26/COPYRIGHT	2015-06-03 13:32:14.000000000 -0400
@@ -1,7 +1,7 @@
 
  ttylog - TTY logger
  Copyright (C) 1999-2002 Tibor Koleszar <oldw@debian.org>
- Copyright (C) 2008-2013 Robert James Clay <jame@rocasa.us>
+ Copyright (C) 2008-2015 Robert James Clay <jame@rocasa.us>
 
  This program is free software; you can redistribute it and/or
  modify it under the terms of the GNU General Public License
diff -Nru ttylog-0.25/debian/changelog ttylog-0.26/debian/changelog
--- ttylog-0.25/debian/changelog	2013-06-20 15:47:56.000000000 -0400
+++ ttylog-0.26/debian/changelog	2015-12-31 22:46:54.000000000 -0500
@@ -1,3 +1,22 @@
+ttylog (0.26-1~deb8u1) stable; urgency=medium
+
+  * Resolve the issue in 'jessie' with the truncating of the modem_device string
+    during the normal operation of ttylog.
+  * Revert Debhelper Compatibility and Build-Depends to version 8 for 'jessie'.
+
+ -- Robert James Clay <jame@rocasa.us>  Thu, 31 Dec 2015 22:38:47 -0500
+
+ttylog (0.26-1) unstable; urgency=medium
+
+  * New upstream release:
+    - Correct truncating of 'modem_device' string. (Closes: #731086).
+  * Reformat the long descriptions in debian/control.
+  * Set Debhelper Compatibility and Build-Depends to version 9.
+  * Update copyright years for Robert James Clay in debian/copyright.
+  * Set Standards Version in debian/control to 3.9.6, no changes required.
+
+ -- Robert James Clay <jame@rocasa.us>  Sun, 21 Jun 2015 09:46:01 -0400
+
 ttylog (0.25-1) unstable; urgency=low
 
   * New upstream release, including:
@@ -35,14 +54,14 @@
   * New upstream release.
   * Set debhelper Build-Depends to version 7.
   * Explicitly set Debian package format as 1.0.
-  * Changed debian/watch to point to Sourceforge. 
+  * Changed debian/watch to point to Sourceforge.
   * Change Maintainer email address to jame@rocasa.us.
   * Set Standards Version to 3.9.1; no changes necessary.
   * Detailed the Debian packaging copyright in debian/copyright.
   * Change homepage in debian/control to Sourceforge project site.
   * Add Vcs-Git & Vcs-Browser to debian/control. (Closes: #566938)
   * Changed to using dh_prep in the install target of debian/rules.
- 
+
  -- Robert James Clay <jame@rocasa.us>  Sat, 26 Mar 2011 14:15:39 -0400
 
 ttylog (0.1.c-1) unstable; urgency=low
@@ -74,7 +93,7 @@
 ttylog (0.1.b-3) unstable; urgency=low
 
   * Using optional output flush - Closes: bug#164778
-   
+
  -- Tibor Koleszar <oldw@debian.org>  Sat, 04 Nov 2002 15:18:12 +0200
 
 ttylog (0.1.b-2) unstable; urgency=low
@@ -84,7 +103,7 @@
   * Added COPYRIGHT and LICENCE files - Closes: bug#127499
   * Using debhelper instead of debmake - Closes: bug#119600 
   * Its not a debian native package - Closes: bug#127498 
-   
+
  -- Tibor Koleszar <oldw@debian.org>  Sat, 12 Jan 2002 13:21:27 +0200
 
 ttylog (0.1.b-1) unstable; urgency=low
@@ -93,7 +112,7 @@
   * Removed binary files from upstream - Closes: bug#127496
   * Added COPYRIGHT and LICENCE files - Closes: bug#127499
   * Using debhelper instead of debmake - Closes: bug#119600 
-   
+
  -- Tibor Koleszar <oldw@debian.org>  Sat, 12 Jan 2002 13:21:27 +0200
 
 ttylog (0.1.a-3) unstable; urgency=low
diff -Nru ttylog-0.25/debian/control ttylog-0.26/debian/control
--- ttylog-0.25/debian/control	2013-06-20 15:47:56.000000000 -0400
+++ ttylog-0.26/debian/control	2015-12-31 22:46:54.000000000 -0500
@@ -3,7 +3,7 @@
 Priority: extra
 Maintainer: Robert James Clay <jame@rocasa.us>
 Build-Depends: debhelper (>= 8), dpkg-dev (>= 1.16.1~), cmake
-Standards-Version: 3.9.4
+Standards-Version: 3.9.6
 Vcs-Git: git://git.code.sf.net/p/ttylog/code -b debian
 Vcs-Browser: http://sourceforge.net/p/ttylog/code/
 Homepage: http://ttylog.sourceforge.net
@@ -12,18 +12,15 @@
 Architecture: any
 Depends: ${shlibs:Depends}, ${misc:Depends}
 Description: serial port logger
- Print everything to stdout that comes from a
- serial device. You can specify the device and
- the baud rate.
+ Print everything to stdout that comes from a serial device. You can specify
+ the device and the baud rate.
 
 Package: ttylog-dbg
 Architecture: any
 Section: debug
 Depends: ttylog (= ${binary:Version}), ${misc:Depends}
 Description: debugging symbols for ttylog
- Print everything to stdout that comes from a
- serial device. You can specify the device and
- the baud rate.
+ Print everything to stdout that comes from a serial device. You can specify
+ the device and the baud rate.
  .
- This package contains the debugging symbols for
- ttylog.
+ This package contains the debugging symbols for ttylog.
diff -Nru ttylog-0.25/debian/copyright ttylog-0.26/debian/copyright
--- ttylog-0.25/debian/copyright	2013-06-20 15:47:56.000000000 -0400
+++ ttylog-0.26/debian/copyright	2015-12-31 22:46:54.000000000 -0500
@@ -5,14 +5,14 @@
 
 Files: *
 Copyright: 1999-2002, Tibor Koleszar <oldw@debian.org>
-           2008-2013, Robert James Clay <jame@rocasa.us>
+           2008-2015, Robert James Clay <jame@rocasa.us>
 License: GPL-2+
 
 Files: debian/*
 Comment: Copyright and License for the debian directory files.
 Copyright: 1999-2002, Tibor Koleszar <oldw@debian.org>
               2006,   Andrew Pollock <apollock@debian.org>
-           2008-2013, Robert James Clay <jame@rocasa.us>
+           2008-2015, Robert James Clay <jame@rocasa.us>
 License: GPL-2+
 
 License: GPL-2+
diff -Nru ttylog-0.25/README.md ttylog-0.26/README.md
--- ttylog-0.25/README.md	1969-12-31 19:00:00.000000000 -0500
+++ ttylog-0.26/README.md	2015-06-03 13:32:14.000000000 -0400
@@ -0,0 +1,7 @@
+ttylog
+======
+
+Print everything to stdout that comes from a serial device. You can specify
+device and baud rate of the device.
+
+Information about how to build ttylog can be found in the INSTALL file.
diff -Nru ttylog-0.25/ttylog.8 ttylog-0.26/ttylog.8
--- ttylog-0.25/ttylog.8	2013-02-14 15:25:52.000000000 -0500
+++ ttylog-0.26/ttylog.8	2015-06-03 13:32:14.000000000 -0400
@@ -1,4 +1,4 @@
-.TH TTYLOG 8 "2012-09-25" "" ""
+.TH TTYLOG 8 "2014-07-29" "" ""
 .SH NAME
 ttylog \- serial device logger
 .SH DESCRIPTION
@@ -22,7 +22,7 @@
 The serial device. For example /dev/ttyS1
 .TP
 .B -f --flush
-Flush output
+Output buffers are flushed after every write.
 .SH AUTHOR
 This manual page was originally written by Tibor Koleszar <t.koleszar@somogy.hu>,
 for the Debian GNU/Linux system.  Modifications and updates written by
diff -Nru ttylog-0.25/ttylog.c ttylog-0.26/ttylog.c
--- ttylog-0.25/ttylog.c	2013-02-14 15:25:52.000000000 -0500
+++ ttylog-0.26/ttylog.c	2015-06-03 13:32:14.000000000 -0400
@@ -1,6 +1,6 @@
 /* ttylog - serial port logger
  Copyright (C) 1999-2002  Tibor Koleszar
- Copyright (C) 2008-2013  Robert James Clay
+ Copyright (C) 2008-2015  Robert James Clay
 
  This program is free software; you can redistribute it and/or
  modify it under the terms of the GNU General Public License
@@ -25,7 +25,7 @@
 #include <sys/types.h>
 #include <fcntl.h>
 
-#define VERSION "0.25"
+#define VERSION "0.26"
 #define BAUDN 9
 
 char flush = 0;
@@ -59,15 +59,22 @@
 
       if (!strcmp (argv[i], "-h") || !strcmp (argv[i], "--help"))
         {
-          printf ("ttylog version %s. Programmed by Tibor Koleszar <oldw@debian.org>\n", VERSION);
-          printf ("Usage: \n -h, --help	This help\n -v, --version	Version number\n");
-          printf (" -b, --baud	Baud rate\n -d, --device	Serial device (eg. /dev/ttyS1)\n -f, --flush	Flush output\n");
+          printf ("ttylog version %s\n", VERSION);
+          printf ("Usage:  ttylog [-b|--baud] [-d|--device] [-f|--flush] > /path/to/logfile\n");
+          printf (" -h, --help	This help\n -v, --version	Version number\n -b, --baud	Baud rate\n");
+          printf (" -d, --device	Serial device (eg. /dev/ttyS1)\n -f, --flush	Flush output\n");
+          printf ("ttylog home page: <http://ttylog.sourceforge.net/>\n\n");
           exit (0);
         }
 
       if (!strcmp (argv[i], "-v") || !strcmp (argv[i], "--version"))
         {
-          printf ("ttylog version %s.\n", VERSION);
+          printf ("ttylog version %s\n", VERSION);
+          printf ("Copyright (C) 2015 Robert James Clay <jame@rocasa.us>\n");
+          printf ("Copyright (C) 2002 Tibor Koleszar <oldw@debian.org>\n");
+          printf ("License GPLv2+: <http://www.gnu.org/licenses/old-licenses/gpl-2.0.html>\n");
+          printf ("This is free software: you are free to change and redistribute it.\n");
+          printf ("There is NO WARRANTY, to the extent permitted by law.\n");
           exit (0);
         }
 
@@ -97,7 +104,7 @@
           {
             memset (modem_device, '\0', sizeof(modem_device));
             strncpy (modem_device, argv[i + 1], sizeof(modem_device)-1);
-            modem_device[i + 1] = '\0';
+            modem_device[(sizeof(modem_device)-1)] = '\0';
           }
         else
           {

Reply to: