Bug#859778: RFS: xtrs/4.9d-2

2017-04-20 Thread Sean Whitton
Hello Branden,

On Thu, Apr 20, 2017 at 04:58:03PM -0400, G. Branden Robinson wrote:
> Sure am.  In fact I have xtrs 4.9d-3 pretty much ready to go except for
> the official versioning and tagging.  I've also been in touch with
> upstream; he's been working on an xtrs 5.0 for quite some time.  :)
> 
> See attachments.  I've also pushed my work to alioth.

Please remove the moreinfo tag from the bug when it's ready to be
uploaded.

-- 
Sean Whitton


signature.asc
Description: PGP signature


Bug#859778: RFS: xtrs/4.9d-2

2017-04-20 Thread G. Branden Robinson
At 2017-04-20T15:58:04-0700, Sean Whitton wrote:
> Please remove the moreinfo tag from the bug when it's ready to be
> uploaded.

Will do!

Regards,
Branden


signature.asc
Description: PGP signature


Bug#859778: RFS: xtrs/4.9d-2

2017-04-20 Thread G. Branden Robinson
At 2017-04-20T09:45:13-0700, Sean Whitton wrote:
> Hello Branden,

Hi Sean!

> On Wed, Apr 12, 2017 at 11:09:45PM -0400, G. Branden Robinson wrote:
> > Yeah, I didn't understand that at the time.  Subsequently I changed all
> > the 4.9d changelog entries to target experimental.
> > 
> > > Thus, it'd be better to target the new version at experimental instead
> > > (or refrain from updates for now -- but that goes against "release
> > > early, release often" and makes debdiffs far harder to read).
> > 
> > It sounds like I should maybe just forget about -2 and upload -3.  I had
> > one more thing I wanted to do for it but it's a big task, rather
> > involved, and not required.
> 
> What is the status of this RFS?  Are you still working on the upload to
> experimental?

Sure am.  In fact I have xtrs 4.9d-3 pretty much ready to go except for
the official versioning and tagging.  I've also been in touch with
upstream; he's been working on an xtrs 5.0 for quite some time.  :)

See attachments.  I've also pushed my work to alioth.

-- 
Regards,
Branden
Format: 3.0 (quilt)
Source: xtrs
Binary: xtrs
Architecture: any
Version: 4.9d-3~~unreleased
Maintainer: G. Branden Robinson 
Homepage: http://www.tim-mann.org/xtrs.html
Standards-Version: 3.9.8
Build-Depends: bsdmainutils, debhelper (>= 9), groff-base, html2text, 
libreadline-dev, libx11-dev, po-debconf
Package-List:
 xtrs deb contrib/otherosfs extra arch=any
Checksums-Sha1:
 42b1fc90246901456d29071421e838b545f39f0f 99 xtrs_4.9d.orig.tar.gz
 a346107dc33d28e66e440590edeb63c1e35fe679 100156 
xtrs_4.9d-3~~unreleased.debian.tar.xz
Checksums-Sha256:
 3985f2331e76198dfc027bc2afcd09a158d2bcad0348aeb4a4958a8fb99cf5c4 99 
xtrs_4.9d.orig.tar.gz
 702129c437c486a6b98d75b54ff1561aff16b1e59b96189e44c6c9a31a873163 100156 
xtrs_4.9d-3~~unreleased.debian.tar.xz
Files:
 93868bed769c038bfae907375316bb2d 99 xtrs_4.9d.orig.tar.gz
 f89963f7dc0155dd23eeb65f9d4d3f23 100156 xtrs_4.9d-3~~unreleased.debian.tar.xz
 dpkg-buildpackage -rfakeroot -us -uc
dpkg-buildpackage: info: source package xtrs
dpkg-buildpackage: info: source version 4.9d-3~~unreleased
dpkg-buildpackage: info: source distribution experimental
dpkg-buildpackage: info: source changed by G. Branden Robinson 

 dpkg-source --before-build xtrs
dpkg-buildpackage: info: host architecture amd64
dpkg-source: info: applying prep-makefiles-for-debian.patch
dpkg-source: info: applying fix-compiler-warnings.patch
dpkg-source: info: applying ignore-alt-key-events.patch
dpkg-source: info: applying add-ifdef-guards-around-setuid.patch
dpkg-source: info: applying stop-ignoring-result-from-fread.patch
dpkg-source: info: applying make-plain-text-docs-from-html.patch
dpkg-source: info: applying debian-stop-clobbering-cflags-and-ldflags.patch
dpkg-source: info: applying mkdisk-document-d-option-in-usage-message.patch
dpkg-source: info: applying stop-mkdisk-from-overflowing-buffers.patch
dpkg-source: info: applying mkdisk-check-fopen-return-with-dmk-images.patch
dpkg-source: info: applying mkdisk-protect-against-overwrites.patch
dpkg-source: info: applying this-one-goes-to-c11.patch
dpkg-source: info: applying hex2cmd-idiomatize-manpage.patch
dpkg-source: info: applying cmddump-idiomatize-manpage.patch
dpkg-source: info: applying cassette-idiomatize-manpage.patch
dpkg-source: info: applying debian-cassette-manpage-del-paragraph.patch
dpkg-source: info: applying mkdisk-idiomatize-manpage.patch
dpkg-source: info: applying xtrs-idiomatize-manpage.patch
dpkg-source: info: applying makefile-generate-pdf-manpages.patch
dpkg-source: info: applying emtsafe-flag-on-by-default.patch
dpkg-source: info: applying write-online-help-to-stderr-if-small-window.patch
dpkg-source: info: applying kill-last-fprintf-stderr-stragglers.patch
 fakeroot debian/rules clean
dh clean
   dh_testdir
   dh_auto_clean
make -j1 clean
make[1]: Entering directory '/home/branden/git/debian/xtrs'
rm -f z80.o main.o load_cmd.o load_hex.o trs_memory.o trs_keyboard.o error.o 
debug.o dis.o trs_io.o trs_cassette.o trs_xinterface.o trs_chars.o 
trs_printer.o trs_rom1.o trs_rom3.o trs_rom4p.o trs_disk.o trs_interrupt.o 
trs_imp_exp.o trs_hard.o trs_uart.o mkdisk.o compile_rom.o error.o load_cmd.o 
load_hex.o cmd.o error.o load_hex.o hex2cmd.o \
cmddump.o load_cmd.o trs_rom*.c *~ \
xtrs mkdisk hex2cmd cmddump compile_rom \
cpmutil.txt dskspec.txt
make[1]: Leaving directory '/home/branden/git/debian/xtrs'
   dh_clean
rm -f debian/debhelper-build-stamp
rm -f debian/xtrs.substvars
rm -f debian/xtrs.*.debhelper
rm -rf debian/xtrs/
rm -rf debian/.debhelper/
rm -f debian/*.debhelper.log
rm -f debian/files
rm -f -- debian/copyright-info.actual debian/no-copyright-info.actual
find .  \( \( \
\( -path .\*/.git -o -path .\*/.svn -o -path .\*/.bzr -o -path 
.\*/.hg -o -path .\*/CVS \) -prune -o -type f -a \
\( -name '#*#' -o -name 

Bug#859778: RFS: xtrs/4.9d-2

2017-04-20 Thread Sean Whitton
Hello Branden,

On Wed, Apr 12, 2017 at 11:09:45PM -0400, G. Branden Robinson wrote:
> Yeah, I didn't understand that at the time.  Subsequently I changed all
> the 4.9d changelog entries to target experimental.
> 
> > Thus, it'd be better to target the new version at experimental instead
> > (or refrain from updates for now -- but that goes against "release
> > early, release often" and makes debdiffs far harder to read).
> 
> It sounds like I should maybe just forget about -2 and upload -3.  I had
> one more thing I wanted to do for it but it's a big task, rather
> involved, and not required.

What is the status of this RFS?  Are you still working on the upload to
experimental?

Thanks!

-- 
Sean Whitton


signature.asc
Description: PGP signature


Bug#859778: RFS: xtrs/4.9d-2

2017-04-12 Thread G. Branden Robinson
At 2017-04-12T22:49:51+0200, Adam Borowski wrote:
> On Wed, Apr 12, 2017 at 12:28:11PM -0400, G. Branden Robinson wrote:
> > I think I got it fixed.
> > 
> > The URL of my package is:
> > https://mentors.debian.net/package/xtrs
> > 
> > The respective dsc file can be found at:
> > https://mentors.debian.net/debian/pool/contrib/x/xtrs/xtrs_4.9d-2.dsc
> 
> Cool, got it.
> 
> The package looks good to me, albeit I did only a quite cursory
> review.
> 
> From the technical point of view, I've so far noticed only one issue:
> the changelog lost the entry for 4.9c-4; even though changes done
> there have been reimplemented, it would confuse BTS version tracking
> somewhat.  Could you please insert that entry back?

It's already merged in in what will become 4.9d-3.  See attached
changelog.

> However, you try to put into unstable a lot of changes not targetted at
> Stretch.  That's a pretty bad idea even for a leaf package -- in case
> there's a need for an update (not unlikely after an upload a mere week ago),
> using testing-proposed-updates is burdensome and doesn't work well.

Yeah, I didn't understand that at the time.  Subsequently I changed all
the 4.9d changelog entries to target experimental.

> Thus, it'd be better to target the new version at experimental instead
> (or refrain from updates for now -- but that goes against "release
> early, release often" and makes debdiffs far harder to read).

It sounds like I should maybe just forget about -2 and upload -3.  I had
one more thing I wanted to do for it but it's a big task, rather
involved, and not required.

-- 
Regards,
Branden
xtrs (4.9d-3~~unreleased) experimental; urgency=medium

  * mkdisk.c: Fix buffer overflow when given filename >8 characters.  Truncate
filename by default when copying to hard disk image.  Add -S ("spill")
flag to partially simulate old behavior.  Exit with error if filename
argument would overflow even the subsequent structure member historically
used by xtrs to store extra filename characters.
+ mkdisk.man: Document -S flag and related issues.
+ test-mkdisk.sh: Add tests for overflow and new filename truncation and
  spillage logic.
+ Makefile: Add "check" target to run the foregoing test.  Nothing
  upstream calls this target automatically.
  * mkdisk.c: Check return value of fopen() when creating DMK disk image file.
  * mkdisk.c: Refuse to clobber files by default.  Add -f ("force") flag to
override this behavior.
+ mkdisk.man: Document new behavior and -f flag.
+ test-mkdisk.sh: Test default no-clobber and -f flag behavior.
  * mkdisk.c: Document the -d option for hard disk images in usage message.
  * Makefile: Observe LDFLAGS when building internal "compile_rom" tool.
Thanks to Graham Inggs for the discussion!  (Closes: #859751)
  * Port to C11 and build with -std=c11.

 -- G. Branden Robinson   Sun, 09 Apr 2017 
15:34:01 -0400

xtrs (4.9d-2) experimental; urgency=medium

  * Export Debian build flags to environment.  Executables are now hardened
per < https://wiki.debian.org/Hardening >.
  * Add Turkish debconf template translations; thanks, Mert Dirik!
(Closes: #757864)
  * Add Dutch debconf template translations; thanks, Frans Spiesschaert!
(Closes: #767488)
  * Add Indonesian debconf template translations; thanks, Izharul Haq!
(Closes: #835622)

 -- G. Branden Robinson   Sun, 26 Mar 2017 
22:17:39 -0400

xtrs (4.9d-1) experimental; urgency=medium

  * Merge new upstream release.
+ "Deleted all SIGIO code.  The code was a kludge to begin with and it no
  longer worked with current X libraries and Linux kernels, causing xtrs
  to hang.  It was also reported to cause hangs when xtrs was compiled for
  Windows using Cygwin.  Thanks to Howard Pepper, Dennis Lovelady, Arumin
  Nueckel, Christopher Currie, and Joe Peterson for bug reports."
  (Closes: #511645)
  * Update README.Debian to refresh URLs and reflect developments in the
TRS-80 retrocomputing enthusiast community over the past several years.
  * Implement debian/compare-copyright script.
+ Add "check-source" target to debian/rules to call the script.
+ Add debian/{no-,}copyright-info.expected files.
  * Migrate former contents of debian/checklist to debian/README.source.
  * Rewrite debian/copyright using machine-readable copyright info.
  * Migrate to new (to me) quilt-based Debian source format 3.0.
+ Migrate former contents of debian/patches to debian/patch/*; dropping
  patches now merged upstream.
  * Migrate former contents of debian/README.contrib-only to Disclaimer field
of debian/copyright, and update discussion.
  * Stop shipping Tim Mann's TRS-80 FAQ document.  It's great, but strictly
speaking, it doesn't carry a license, I don't want to pester him to put
one on it, and in any event it updates much more frequently than the xtrs
software itself.  Finally, I trust people to do web searches, and
archive.org to stick around

Bug#859778: RFS: xtrs/4.9d-2

2017-04-12 Thread Adam Borowski
On Wed, Apr 12, 2017 at 12:28:11PM -0400, G. Branden Robinson wrote:
> I think I got it fixed.
> 
> The URL of my package is:
> https://mentors.debian.net/package/xtrs
> 
> The respective dsc file can be found at:
> https://mentors.debian.net/debian/pool/contrib/x/xtrs/xtrs_4.9d-2.dsc

Cool, got it.


The package looks good to me, albeit I did only a quite cursory review.

>From the technical point of view, I've so far noticed only one issue: the
changelog lost the entry for 4.9c-4; even though changes done there have
been reimplemented, it would confuse BTS version tracking somewhat.  Could
you please insert that entry back?


However, you try to put into unstable a lot of changes not targetted at
Stretch.  That's a pretty bad idea even for a leaf package -- in case
there's a need for an update (not unlikely after an upload a mere week ago),
using testing-proposed-updates is burdensome and doesn't work well.  Thus,
it'd be better to target the new version at experimental instead (or refrain
from updates for now -- but that goes against "release early, release often"
and makes debdiffs far harder to read).


ᛗᛖᛟᚹ!
-- 
⢀⣴⠾⠻⢶⣦⠀ Meow!
⣾⠁⢠⠒⠀⣿⡁
⢿⡄⠘⠷⠚⠋⠀ Collisions shmolisions, let's see them find a collision or second
⠈⠳⣄ preimage for double rot13!



Bug#859778: RFS: xtrs/4.9d-2

2017-04-12 Thread G. Branden Robinson
I think I got it fixed.

The URL of my package is:
https://mentors.debian.net/package/xtrs

The respective dsc file can be found at:
https://mentors.debian.net/debian/pool/contrib/x/xtrs/xtrs_4.9d-2.dsc

##lftp mentors.debian.net:/debian/pool/contrib/x/xtrs> ls
drwxr-xr-x  --  ..
-rw-r--r-- 433K  2017-04-01 02:48  xtrs_4.9c.orig.tar.gz
-rw-r--r--  30K  2017-04-12 16:19  xtrs_4.9d-2.debian.tar.xz
-rw-r--r-- 1.8K  2017-04-12 16:19  xtrs_4.9d-2.dsc
-rw-r--r-- 434K  2017-04-12 16:19  xtrs_4.9d.orig.tar.gz

At 2017-04-12T17:39:54+0200, Adam Borowski wrote:
> On Wed, Apr 12, 2017 at 10:20:55AM -0400, G. Branden Robinson wrote:
> > Hrm #1: GMail helpfully categorized your mail as spam.
> 
> Not nice, let's try again; if you don't respond in a day I'll send
> from some other place.

It's fixed.  I told GMail you're not a spammer.  :)

-- 
Regards,
Branden


signature.asc
Description: PGP signature


Bug#859778: RFS: xtrs/4.9d-2

2017-04-12 Thread Adam Borowski
On Wed, Apr 12, 2017 at 10:20:55AM -0400, G. Branden Robinson wrote:
> At 2017-04-07T15:23:19+0200, Adam Borowski wrote:
> > On Fri, Apr 07, 2017 at 07:41:51AM -0400, G. Branden Robinson wrote:
> > > * Package name: xtrs
> > >   Version : 4.9d-2
> > 
> > > https://mentors.debian.net/package/xtrs
> > > dget -x 
> > > https://mentors.debian.net/debian/pool/contrib/x/xtrs/xtrs_4.9d-2.dsc
> > 
> > I'm afraid these URLs are 404-compliant.  Am I missing something?
> 
> Hrm #1: GMail helpfully categorized your mail as spam.

Not nice, let's try again; if you don't respond in a day I'll send from some
other place.

> Hrm #2: Apparently I forgot the correct sponsorship-request procedure.

Not sure if learning it is really worth your time :p

This used to work for me, I'm a DD only since Dec 2015 so it's probably
still valid:

.--[ ~/.dupload.conf
package config;

$cfg{'mentors'} =
{
   fqdn => 'mentors.debian.net',
   incoming => '/',
   dinstall_runs => 1,
   passive => 1,
};

1;
`

> $ dput mentors xtrs_4.9d-2_amd64.changes
> Package has already been uploaded to mentors on mentors.debian.net
> Nothing more to do for xtrs_4.9d-2_amd64.changes
> 
> lftp mentors.debian.net and then "ls" returns no results.
> 
> How do I find my upload after I've uploaded it?

You should get a mail shortly after the upload, if you didn't receive one,
the upload has either been found invalid or hasn't been processed yet.

Like with the main archive, bad/unknown signatures usually cause no
response.


If you get angry about mentors.debian.net not cooperating, any other place
to stick the packages is almost as good: be it reprepro/your favourite
private archive tool, or just manually dumping .dsc + .debian.tar.?z + .orig
onto a random http/https/ftp/gopher server.

-- 
⢀⣴⠾⠻⢶⣦⠀ Meow!
⣾⠁⢠⠒⠀⣿⡁
⢿⡄⠘⠷⠚⠋⠀ Collisions shmolisions, let's see them find a collision or second
⠈⠳⣄ preimage for double rot13!



Bug#859778: RFS: xtrs/4.9d-2

2017-04-12 Thread G. Branden Robinson
At 2017-04-07T15:23:19+0200, Adam Borowski wrote:
> On Fri, Apr 07, 2017 at 07:41:51AM -0400, G. Branden Robinson wrote:
> > * Package name: xtrs
> >   Version : 4.9d-2
> 
> > https://mentors.debian.net/package/xtrs
> > dget -x 
> > https://mentors.debian.net/debian/pool/contrib/x/xtrs/xtrs_4.9d-2.dsc
> 
> I'm afraid these URLs are 404-compliant.  Am I missing something?

Hrm #1: GMail helpfully categorized your mail as spam.

Hrm #2: Apparently I forgot the correct sponsorship-request procedure.

$ dput mentors xtrs_4.9d-2_amd64.changes
Package has already been uploaded to mentors on mentors.debian.net
Nothing more to do for xtrs_4.9d-2_amd64.changes

lftp mentors.debian.net and then "ls" returns no results.

How do I find my upload after I've uploaded it?

Regards,
Branden


signature.asc
Description: PGP signature


Bug#859778: RFS: xtrs/4.9d-2

2017-04-07 Thread Adam Borowski
On Fri, Apr 07, 2017 at 07:41:51AM -0400, G. Branden Robinson wrote:
> * Package name: xtrs
>   Version : 4.9d-2

> https://mentors.debian.net/package/xtrs
> dget -x 
> https://mentors.debian.net/debian/pool/contrib/x/xtrs/xtrs_4.9d-2.dsc

I'm afraid these URLs are 404-compliant.  Am I missing something?

-- 
⢀⣴⠾⠻⢶⣦⠀ Meow!
⣾⠁⢠⠒⠀⣿⡁
⢿⡄⠘⠷⠚⠋⠀ Collisions shmolisions, let's see them find a collision or second
⠈⠳⣄ preimage for double rot13!



Bug#859778: RFS: xtrs/4.9d-2

2017-04-07 Thread G. Branden Robinson
Package: sponsorship-requests
Severity: normal

Dear mentors,

I seek a sponsor for my package "xtrs".

* Package name: xtrs
  Version : 4.9d-2
  Upstream Author : Tim Mann
* URL : http://www.tim-mann.org/xtrs.html
* License : 2 different custom permissive licenses
  Section : otherosfs

It builds these binary packages:

xtrs  - emulator for TRS-80 Model I/III/4/4P computers

To access further information about this package, please visit the
following URL:

https://mentors.debian.net/package/xtrs

Alternatively, one can download the package with dget using this
command:

dget -x 
https://mentors.debian.net/debian/pool/contrib/x/xtrs/xtrs_4.9d-2.dsc

Changes since the last upload:

  * Export Debian build flags to environment.  Executables are now hardened
per < https://wiki.debian.org/Hardening >.
  * Add Turkish debconf template translations; thanks, Mert Dirik!
(Closes: #757864)
  * Add Dutch debconf template translations; thanks, Frans Spiesschaert!
(Closes: #767488)
  * Add Indonesian debconf template translations; thanks, Izharul Haq!
(Closes: #835622)

Regards,
Branden

-- System Information:
Debian Release: 9.0
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 4.9.0-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=C, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)