Upgrade lesspipe to 1.71

[royliu@…]

The title says it all. See patch archive attached.

[royliu@…]

The patch archive.

[royliu@…]

I might add that the new version finds the correct version of 'tar' for archive viewing, so this request is slightly more than an upgrade.

[IngmarStein@…]

Well then, could someone apply the patch please?

[jmr@…]

Sure, done in r67534. Luckily I was trawling through the list of update tickets; normally nobody but the reporter, the owner and those in cc will see ticket comments. If you, as a port's maintainer, would like something committed, the most effective place to ask is on the macports-dev list or on IRC.

Note to the reporter; attaching the patch in an archive like this makes it harder to review. Including whitespace changes for a bunch of lines where nothing functional has changed makes it even harder. The patch also incorrectly removes the Id line.

[royliu@…]

Sorry about that, my workflow is heavily automated to the point where I just submit whatever my scripts have post-processed. For future reference, should I leave the Id field unchanged? I was always confused about how SVN versions and diffs files with keyword replacements like $Id$.

[jmr@…]

Replying to royliu@…:

Sorry about that, my workflow is heavily automated to the point where I just submit whatever my scripts have post-processed. For future reference, should I leave the Id field unchanged? I was always confused about how SVN versions and diffs files with keyword replacements like $Id$.

Yes, just leave the Id line unchanged. It is stored in the repository as "# $Id$" and is expanded by the client on checkout. If you can't avoid making whitespace changes in the first place, try using diff -b.