Re: [Libvir] make syntax-check fails with bzr checkouts

From: Jim Meyering <jim meyering net>

To: libvir-list redhat com

Subject: Re: [Libvir] make syntax-check fails with bzr checkouts

Date: Wed, 30 Apr 2008 19:24:42 +0200

Soren Hansen <soren ubuntu com> wrote:
> On Wed, Apr 30, 2008 at 12:19:51PM +0200, Jim Meyering wrote:
>> Unfortunately, the above no longer applies, due to upstream (gnulib)
>> changes to deal with non-srcdir (aka VPATH) builds. I updated libvirt
>> from gnulib just yesterday, and will again, later today.
>>
>> Can you adapt your patch to make bzr work with the newer version?
>
> I'll do it when you're done updating gnulib.
I'm done for now, and pulled the new version into libvirt.
...
> Well, if anyone insists on not changing something that only works with
> bash to something that works with any shell (${*:-*} vs. ${*-*}) , but
> also refuses to put /bin/bash explicitly as the interpreter is just
> being pointlessly difficult, IMO.
Um... maybe you didn't notice that I removed the
problematic syntax altogether?
...
>> Here's the patch I've pushed to gnulib:
>>
>> http://git.sv.gnu.org/gitweb/?p=gnulib.git;a=commitdiff;h=173a9f0c48a16c3507f8
>
> That patch will make the cvs case fail, too. It will prepend a ./ to all
> path names which will screw with the comparison, AFAICS.
It handled one case. The two subsequent patches addressed the others.
>> If you care about the CVS-oriented cases, there are two more patches.