@n8felton oh, sorry! If I had known it'd be that quick, I would have held off... @sschuberth could you have a look when you have time?

I really don't have time for a thorough review currently. Just from a very quick glance I see a lot of indentation and style issues. I'd prefer to not merge this as-is.

b518955 has the proper whitespaces; I converted the TABs to Spaces. Sorry about that.

Thanks for the update. I've commented on a few lines that still have style issues, but there are more lines. In general, our currently style does not use whitespaces before / after special characters.

I see more what you mean now by style. I'll make these changes and resubmit.

Alright, give d30295f a look and let me know what you think.

Besides all my style comments, I assume that you have actually build the installer and verified that it works as expected, correct?

Yes, after each style correction suggestion, I am running the release.sh script to build the installer and I am testing the functionality.

@n8felton oh, I thought you had tested it already. For my curiosity, could you paste a .inf file in a comment to this ticket (I'd like to see how it looks like to get a feel what one can do with it)?

Example INF after running a /saveinf argument with my preferred settings.

[Setup]
Lang=default
Dir=C:\Program Files (x86)\Git
Group=Git
NoIcons=0
SetupType=default
Components=assoc,assoc_sh
Tasks=
PathOption=BashOnly
SSHOption=OpenSSH
CRLFOption=CRLFCommitAsIs

Check out f5f991e. Hopefully I have all (if not most) of the style errors fixed.

b53b404 is up. Let me know what you think.

Looks good to me now. @dscho Please merge it you're OK with it, too.

Thanks both!