Re: check-news target fails if NEWS file starts with a copyright header

2010-06-07 Thread Jim Meyering
Peter Simons wrote: > Karl Berry writes: > > However, I also see no harm in looking at the first 20 lines instead > > of the first 10 lines to match the version. > > yes, I agree with Karl here. The current implementation enforces a > policy that feels rather odd. A NEWS file that features the re

Re: check-news target fails if NEWS file starts with a copyright header

2010-06-07 Thread Peter Simons
Jim Meyering wrote: > The penalty for an inadequate test would be to release without updating > NEWS. so, according to your logic, expecting any of the top 10 lines of NEWS to match a particular regular expression is "adequate", but expecting any of the top, say 11 lines, of NEWS to match that

Re: check-news target fails if NEWS file starts with a copyright header

2010-06-07 Thread Jim Meyering
Peter Simons wrote: > Jim Meyering wrote: > > The penalty for an inadequate test would be to release without updating > > NEWS. > > so, according to your logic, expecting any of the top 10 lines of NEWS to > match a particular regular expression is "adequate", but expecting any of > the top, say

Re: check-news target fails if NEWS file starts with a copyright header

2010-06-07 Thread Peter Simons
Jim Meyering wrote: > My wanting to minimize risk is now "irrational fears"? No, of course not. The desire to minimize risks is perfectly reasonable. However, your perception of the risks involved in this change strikes me as somewhat exaggerated. > Is there some reason you'd rather not move

Re: check-news target fails if NEWS file starts with a copyright header

2010-06-07 Thread Jim Meyering
Peter Simons wrote: ... > > If you're motivated, propose a patch. > > My suggested solution is attached. Thanks, but that does not allow us to select ranges like "just line 3" or "lines 20-22". How about removing the use of "head" and using this as the default: sed -n 1,10p

[PATCH] describe --local-dir option in documentation

2010-06-07 Thread Ben Pfaff
I couldn't find a description of how or why to use --local-dir, so I wrote this up. OK to push it? Thanks, Ben. --8<--cut here-->8-- >From e5c8b4f4b93bef75db1680d4473958337ebfbd07 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Mon, 7 Jun 2010 22

relocwrapper interaction with replacement functions

2010-06-07 Thread Ben Pfaff
On OpenBSD 4.7, GNU PSPP from current "master" configured with --enable-relocatable fails linking relocwrapper.c at "make install" time: /tmp//ccy8gNb6.o(.text+0x64): In function `add_dotbin': gl/relocwrapper.c:105: undefined reference to `rpl_fprintf' /tmp//ccy8gNb6.o(.text+0x172): In