Bug#691389: [checkbashisms] multiple bug fixes and new checks

Benjamin Drung bdrung at debian.org
Sat Nov 10 23:08:00 UTC 2012


Am Donnerstag, den 01.11.2012, 14:27 -0600 schrieb Raphael Geissert:
> On Thursday 25 October 2012 16:44:03 Benjamin Drung wrote:
> > Am Mittwoch, den 24.10.2012, 21:15 -0500 schrieb Raphael Geissert:
> > > Attached is an mbox with a bunch of bug fixes and new checks, #687450
> > > included.
> > 
> > Thanks. I have applied your patches 1 up to 14. I grabbed the working
> > test cases from your git repository and added them to devscripts. After
> > applying a patch, I added the newly passing test cases. Feel free to add
> > more test cases to devscripts or to improve/simplify the shunit2 tests.
> 
> Hope you don't mind me re-using this bug report.

Nope. Reusing a general patch bug is fine.

> Attached is an mbox with 7 more patches.

Thanks. I have applied them all.

I prefer to have these patches as normal attachments, because otherwise
the filenames get lost and I have to reconstruct the order.

> I don't know yet when I'm going to start another archive-wide 
> run for this patchset, but I don't expect regressions.
> 
> I have updated my test suite accordingly.

Thanks for that. I have integrated the new ones in our shunit2 test framework as
you can see in the devscripts git repository.

Can you send me a patch that adds the changelog entries for all the
(25) applied patches?

-- 
Benjamin Drung
Debian & Ubuntu Developer



More information about the devscripts-devel mailing list