[Libvir] make syntax-check fails with bzr checkouts

Jim Meyering jim at meyering.net
Wed Apr 30 17:24:42 UTC 2008


Soren Hansen <soren at 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.




More information about the libvir-list mailing list