Re: [PATCH virt-viewer] msi: Do not escape '$' for wixl-heat

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

 



On Mon, Jul 24, 2017 at 10:10:02AM +0200, Pavel Grunt wrote:
> Hi,
> 
> On Mon, 2017-07-24 at 08:54 +0100, Christophe Fergeau wrote:
> > Hey,
> > 
> > On Fri, Jul 21, 2017 at 07:27:45PM +0200, Pavel Grunt wrote:
> > > wixl-heat does it itself since its commit 9273514c95e343340d18067b4f1
> > > 
> > > otherwise msi generation fails with:
> > > Couldn't find file $/tmp/tmp.1MYrxTLd1U/usr/x86_64-w64-mingw32/sys-
> > > root/mingw/bin/debug-helper.exe
> > 
> > Looks like either the git version, or all msitools releases are going to
> > be broken by this change?
> 
> Yes. All the future releases are going to be broken/fixed [1] (do you know why
> it was escaped?) since that change.
> 
> >  Should we do something conditional on wixl
> > --version output? (wixl-heat does not have --version).
> 
> imo overkill. The msi release always happen using the latest released fedora.
> And due to changes in rpms, you need the updated version of msitools anyway...

Well, at the moment this change is not in a release, the .spec file
BuildRequires were not changed to reflect that, ... If we already need a
very new msitools version, I'd mention this in the commit log so that
it's clear this should not cause too much breakage.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list

[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux