Bug 725160

Summary: file command doesn't recognize xdelta patch files
Product: [Fedora] Fedora Reporter: Andre Robatino <robatino>
Component: fileAssignee: Jan Kaluža <jkaluza>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: jkaluza
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-10-12 10:19:25 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Andre Robatino 2011-07-23 13:09:31 UTC
Description of problem:

xdelta patch files not recognized in Rawhide or F15 (haven't checked F14).

$ xdelta -s a b c
$ file c
c: data
$

Version-Release number of selected component (if applicable):
file-5.07-5.fc16

Additional info:
Works in RHEL 6 (note that the older syntax is different):

[robatino@secondary01 ~]$ xdelta delta a b c
xdelta_el6: warning: no matches found in from file, patch will apply without it
[robatino@secondary01 ~]$ file c
c: XDelta binary patch file 1.1
[robatino@secondary01 ~]$

Comment 1 Andre Robatino 2011-07-23 13:12:30 UTC
Should explain that the "xdelta_el6" is due to this being a CentOS 6 executable I'm using since the xdelta package was not installed on the RHEL 6 machine.

Comment 2 Andre Robatino 2011-07-23 13:23:24 UTC
Works on F14 (xdelta-1.1.4-9.fc13 and file-5.04-17.fc14) where the old xdelta syntax is used:

[andre@localhost ~]$ xdelta delta a b c
xdelta: warning: no matches found in from file, patch will apply without it
[andre@localhost ~]$ file c
c: XDelta binary patch file 1.1
[andre@localhost ~]$

Does not work on F15 (xdelta-3.0.0-2.fc15 and file-5.07-4.fc15) with the new xdelta syntax. The file command probably only recognizes the old xdelta format.

Comment 3 Jan Kaluža 2011-07-25 10:49:03 UTC
If I understand current situation well, xdelta uses VCDIFF format to store binary diff, so it's not possible to show "XDelta binary patch file x.y" for new format, because it can be generated with more tools.

I will probably use "VCDIFF binary diff".

Comment 4 Jan Kaluža 2011-10-12 10:19:25 UTC
Fixed in file-5.09 update.