[PATCH] Update - Setup.exe property sheet patch, properly diffed.

Robert Collins robert.collins@itdomain.com.au
Wed Nov 7 07:42:00 GMT 2001


Ok, I'll get nitty-gritty now.

ChangeLog presentation:
:The changelog is formatted wrongly - you have extra lines.
:When you have
(foo): Did bar.
(foo): Did barf.
write it as
(foo): Did bar.
Did barf.
:in the main entry you have a tab halfway through the line

Regarding the patch:
:Please remove the #if 0'd items. If its a mistake to remove them, then
we can get them back from CVS.
:Please remove your package_meta.h (sdesc) workaround. It's not the
right answer. (We can discuss what is instead if you like).
:lets assume that chooser will subsume choose, can you please make the
changes direct to choose. (I don't really want a short lived migration
file - it seems pointless). (remember - CVS is smart). If you think
there really will be two classes for this (other than the working inside
classes) then leave it as is.

Cheers,
Rob



More information about the Cygwin-patches mailing list