[Tilesathome] Client 10225 not uploading everything

[Maarten Deen]

spaetz wrote:
> On Wed, Aug 27, 2008 at 08:39:14PM +0200, Maarten Deen wrote:
>> I haven't found the actual point where you make the commandline for the 
>> compress. Where is it and can you put a debug message that prints the 
>> commandline (in the unstable branch)?
> 
> Compress.pm    Line 174  SVN-10243
>  is where the 7zip is constructed and called.
> I have now added a DEBUG statement there that should print out the full command that is being called.

Jep, found the error. This is what Compress.pm instructs 7zip to do:
C:\ProgramFiles\TilesAtHome\zip\7za.exe a -tzip 
C:\temp\TaH\uploadable\_58045_260984_tile_tileset.zip 
C:\temp\TaH\tile_12_2653_1259.dir\*.png
(put it all on one line)

And this is what it should do:
C:\ProgramFiles\TilesAtHome\zip\7za.exe a -tzip 
C:\temp\TaH\uploadable\_58045_260984_tile_tileset.zip 
C:\temp\TaH\12_2653_1259_0P0WH\tile_12_2653_1259.dir\*.png

You're missing the temp directory (12_2653_1259_0P0WH) in the list of files you 
want to zip (Compress.pm lines 176 and onward).

On top of that, I think it would be wise (for Windows at least) to put all 
strings between "". The command will fail also if there is a space in the arguments.

Maarten