Re: make syntax-check fails with bzr checkouts

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]