Unable to compile from putty-0.60-cygterm-20090811.patch.gz; windres error on puttycyg.ico about puttycyg

[GoogleCodeExporter]

The puttycyg.ico from the repository looks OK.  
(http://puttycyg.googlecode.com/svn/trunk/putty-0.60/windows/puttycyg.ico)

This is a bug in the way I've created the patch file (the -a flag to diff).  
Please use the 
above icon file and I will fix the patch file soon.

Original comment by medgar123 on 27 Dec 2009 at 8:05

• Changed state: Accepted

[GoogleCodeExporter]

Sounds great.  Already did pull down the icon from SVN.  Thanks!

Original comment by [email protected] on 27 Dec 2009 at 8:08

[GoogleCodeExporter]

It's not a problem with the patch itself, but with the ability to execute the 
patch.  It 
seems bash doesn't like NUL bytes appearing in a << here doc.  This problem 
goes away 
if you use zsh:

zcat putty-0.60-cygterm-20090811.tar.gz | zsh

I don't have a good fix for this which preserves the ability to use the file 
with patch(1) as 
well as executing it directly without also bringing in another dependency (like 
uudecode).  I suppose since perl is required to build PuTTY, I can assume it is 
available...

Original comment by medgar123 on 28 Dec 2009 at 4:57

• Changed state: Started

[GoogleCodeExporter]

zsh should also be considered as an additional dependency, as it is not 
included in
the default Cygwin install, not required by the PuTTY build, and otherwise just 
not
generally necessary.  That said, I think either approach (zsh or uudecode, 
etc.) is
valid, as long as the documentation is updated accordingly.  I guess I would 
prefer
something more along the lines of the uudecode approach, since it eliminates a
dependency on a particular shell.

Thanks!

Original comment by [email protected] on 28 Dec 2009 at 6:31

[GoogleCodeExporter]

The patch is now a thin sheen of perl which wraps the shell script.  I've 
updated the 
http://code.google.com/p/puttycyg/wiki/Building page.

Original comment by medgar123 on 29 Dec 2009 at 6:17

• Changed state: Fixed