{"thread":{"id":"4464","subject":"[PATCH 0/3] another series of cg-admin-rewritehist -r fixes","startedAt":"2006-06-11T10:12:39Z","lastAt":"2006-06-11T10:12:39Z","messageCount":1,"participants":["Yann Dirson"],"isPatch":true,"patchVersion":1,"patchTotal":3},"messages":[{"id":"21589","messageId":"20060611101239.11876.42498.stgit@gandelf.nowhere.earth","threadId":"4464","inReplyTo":null,"subject":"[PATCH 0/3] another series of cg-admin-rewritehist -r fixes","fromName":"Yann Dirson","fromEmail":"ydirson@altern.org","sentAt":"2006-06-11T10:12:39Z","receivedAt":"2006-06-11T10:12:39Z","isPatch":true,"sender":{"key":"ydirson@altern.org","avatar":"https://avatars.githubusercontent.com/u/1190950?v=4"},"body":"The -r flag is a bit confusing, in that it expects the 1st revision de rewrite, and I\ncaught myself feeding it the last revision not to rewrite instead.  Patch #2 catches\nthis early.  Although Patch #2 should take care of most problems, a non-zero status\nreturned by a command not-last in a pipe is not caught, even under \"set -e\", so Patch #1\nadds an additional safeguard.\n\nPatch #3 corrects seeding of the rewrite map from -r arguments.\n\n-- \nYann Dirson    <ydirson@altern.org> |\nDebian-related: <dirson@debian.org> |   Support Debian GNU/Linux:\n                                    |  Freedom, Power, Stability, Gratis\n     http://ydirson.free.fr/        | Check <http://www.debian.org/>\n"}]}