threads / patch / 4464

patch, 3 partsanother series of cg-admin-rewritehist -r fixes

Subject: [PATCH 0/3] another series of cg-admin-rewritehist -r fixes

## tl;dr

One message between Jun 11, 2006 and Jun 11, 2006. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Yann Dirson· Jun 11, 2006, 10:12 UTC · lore

The -r flag is a bit confusing, in that it expects the 1st revision de rewrite, and I caught myself feeding it the last revision not to rewrite instead. Patch #2 catches this early. Although Patch #2 should take care of most problems, a non-zero status returned by a command not-last in a pipe is not caught, even under "set -e", so Patch #1 adds an additional safeguard.

Patch #3 corrects seeding of the rewrite map from -r arguments.
-- 
Yann Dirson    <ydirson@altern.org> |
Debian-related: <dirson@debian.org> |   Support Debian GNU/Linux:
                                    |  Freedom, Power, Stability, Gratis
     http://ydirson.free.fr/        | Check <http://www.debian.org/>

← back to recent threads