Bug #102150 for Config-AutoConf: [PATCH] Fix some typographical errors.

Mon Feb 16 07:39:07 2015 roam [...] ringlet.net - Ticket created

From:	roam [...] ringlet.net
Subject:	[PATCH] Fix some typographical errors.

Hi, First of all, thanks for writing and maintaining Config-AutoConf! While working on updating the Debian version of Config-AutoConf, I thought I'd let you know about some typographical errors in various comments and messages in the module. What do you think about the attached patch? I'm actually working on a bit larger patch that will reword some of the more unwieldy grammatical constructs, too, but these here are the typos that jumped out at me immediately. Description: Fix some typographical errors. Author: Peter Pentchev <roam@ringlet.net> Last-Update: 2015-02-16 The patch is tracked in our Git repository at https://anonscm.debian.org/cgit/pkg-perl/packages/libconfig-autoconf-perl.git/plain/debian/patches/01-typos.patch Thanks for considering, and keep up the good work! Peter Pentchev, Debian Perl Group

Mon Feb 16 07:39:10 2015 roam [...] ringlet.net - Correspondence added

Here's the patch.

Message body is not shown because sender requested not to inline it.

Mon Feb 16 07:39:10 2015 The RT System itself - Status changed from 'new' to 'open'

Thu Feb 19 07:00:19 2015 REHSACK [...] cpan.org - Correspondence added

Show quoted text

> While working on updating the Debian version of Config-AutoConf, > I thought I'd let you know about some typographical errors in > various comments and messages in the module. What do you think > about the attached patch?

There were several thoughts crossing my mind: 1) I'm surprised that updating a package in Debian causes you to review the documentation in that depth. I appreciate that and being thankful. 2) Thanks for the kick-ass to add a spell-checker to the author tests. Even if it doesn't find suitable tools on my Mac, on my Debian box it will and complain. Might not affect every release, but the bigger ones ;) Show quoted text

> I'm actually working on a bit larger patch that will reword some of > the more unwieldy grammatical constructs, too, but these here are > the typos that jumped out at me immediately.

If you're quick on it, I'll hold the release (see https://github.com/ambs/Config-AutoConf to patch against most recent version). Otherwise I'll do a separate release later - just let me know. Cheers, Jens

Thu Feb 19 07:00:21 2015 REHSACK [...] cpan.org - Status changed from 'open' to 'patched'

Mon Feb 23 04:41:04 2015 roam [...] ringlet.net - Correspondence added

На четв фев 19 07:00:19 2015, REHSACK написа: Show quoted text

> > While working on updating the Debian version of Config-AutoConf, > > I thought I'd let you know about some typographical errors in > > various comments and messages in the module. What do you think > > about the attached patch?

> > There were several thoughts crossing my mind: > > 1) I'm surprised that updating a package in Debian causes you to > review the documentation in that depth. I appreciate that and being > thankful.

Well, it's not really a hard-and-fast rule ;) We try to do it when we can, both for Perl modules and for other software that crosses our path. Show quoted text

> 2) Thanks for the kick-ass to add a spell-checker to the author tests. > Even if it doesn't find suitable tools on my Mac, on my Debian box it > will and complain. Might not affect every release, but the bigger ones > ;)

Ah, yeah, I have to do this for some of my projects, too :) Show quoted text

> > I'm actually working on a bit larger patch that will reword some of > > the more unwieldy grammatical constructs, too, but these here are > > the typos that jumped out at me immediately.

> > If you're quick on it, I'll hold the release (see > https://github.com/ambs/Config-AutoConf to patch against most recent > version). Otherwise I'll do a separate release later - just let me > know.

Sorry for the delayed reply :( Unfortunately, it turns out that I may not be able to do it in the next couple of days, either, so if you want to release with the current changes, go ahead, I'll catch up later. And thanks again for your time and your work! G'luck, Peter

Thu Apr 07 05:16:11 2016 REHSACK [...] cpan.org - Correspondence added

Oops - was resolved in 0.311 already, forgot to mark resolved. Sorry

Thu Apr 07 05:16:12 2016 REHSACK [...] cpan.org - Status changed from 'patched' to 'resolved'

Thu Apr 07 05:16:15 2016 REHSACK [...] cpan.org - Fixed in 0.311 added