On Thu, Jul 24, 2014 at 01:30:11PM +0200, Marc-André Lureau wrote: > ping > > > On Tue, Nov 12, 2013 at 3:27 AM, Marc-André Lureau < > marcandre.lureau@xxxxxxxxx> wrote: > > > It turned out that not only the current MSI broke the "component rule", > > but also that our files are not versionized correctly. Windows Installer > > applies some file versioning rules before replacing a file > > http://msdn.microsoft.com/en-us/library/aa368599%28v=vs.85%29.aspx > > > > Since msitools doesn't extract version from files and populate the Version > > field of the File table, it "usually" keep the current file installed. > > > > It's practically impossible to rely on version information from > > files (from a quick look, only 5% of the files are versionized and even > > less correctly, libgcrypt seems to do non-monotonic buildid for example) > > > > So the rule that applies when files are not versionized is to check the > > file hash, and the modified date. File hash was added recently in > > msitools, but doesn't apply when the installed file itself has a > > version. > > > > In order to solve the above problems, it's simpler to just have a > > different installation prefix. Windows Installer will see files with > > different component guid, and won't be checking any file update rule. I > > have verified the upgrade is working, not leaving any file behind and > > updating registry correctly with this solution. Until the files are > > correctly versionized, it looks like the only sensible thing to > > do. Furthermore, this make it simpler to have several versions installed > > in parallel later on (when we change productid) > > --- > > data/virt-viewer.wxs.in | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/data/virt-viewer.wxs.in b/data/virt-viewer.wxs.in > > index d5c2522..a004de4 100644 > > --- a/data/virt-viewer.wxs.in > > +++ b/data/virt-viewer.wxs.in > > @@ -65,7 +65,7 @@ > > > > <Directory Id="TARGETDIR" Name="SourceDir"> > > <Directory Id="$(var.ArchProgramFilesFolder)"> > > - <Directory Id="INSTALLDIR" Name="VirtViewer"> > > + <Directory Id="INSTALLDIR" Name="VirtViewer v@VERSION@@BUILDID@"> > > <Component Id="CDepsFile" Guid="*"> > > <File Id="filA1E799D196006E6DF67DACE15B8C6193" KeyPath="yes" > > Source="deps.txt"/> > > </Component> ACK Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
