{"thread":{"id":"15770","subject":"[FYI][PATCH] Customizing the WinGit installer","startedAt":"2008-10-03T12:27:27Z","lastAt":"2008-10-08T07:38:42Z","messageCount":23,"participants":["Petr Baudis","Johannes Schindelin","Jakub Narebski","Linus Torvalds","Bruce Stephens","Robin Burchell","Heikki Orsila","Abdelrazak Younes","Dmitry Potapov","Avery Pennarun","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"92244","messageId":"20081003122727.GE10360@machine.or.cz","threadId":"15770","inReplyTo":null,"subject":"[FYI][PATCH] Customizing the WinGit installer","fromName":"Petr Baudis","fromEmail":"pasky@suse.cz","sentAt":"2008-10-03T12:27:27Z","receivedAt":"2008-10-03T12:27:27Z","isPatch":true,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":"This patch is intended just as a FYI example on customizing the WinGit\ninstaller for e.g. corporate deployment or similar needs, as a starting\npoint in case anyone needs to do that.\n\nThis patch removes some steps redundant for our usage scenario (choose\nof the $PATH mode, showing release notes), slightly customizes the\ndefault Git configuration and changes the quickstart icon from Git Bash\nto Git GUI (this one might be worth considering for upstream too,\nespecially after stock git-gui gains the remotes management support).\n\nSigned-off-by: Petr Baudis \n\n---\n\ndiff --git a/etc/gitconfig b/etc/gitconfig\nindex 251088c..d0e8e53 100644\n--- a/etc/gitconfig\n+++ b/etc/gitconfig\n@@ -5,3 +5,12 @@\n \tdiff = auto\n [pack]\n \tpackSizeLimit = 2g\n+[locator \"Server\"]\n+\ttemplate = M:/public_git/%s\n+[locator \"Library\"]\n+\ttemplate = git://lib.example.com/%s\n+[gui]\n+\tremotelocator = Server\n+\tpulllocator = Library\n+\tpushlocator = Server\n+\tautoexplore = true\ndiff --git a/share/WinGit/install.iss b/share/WinGit/install.iss\nindex 0f0514d..05fdae6 100644\n--- a/share/WinGit/install.iss\n+++ b/share/WinGit/install.iss\n@@ -20,7 +20,7 @@ ChangesEnvironment=yes\n DefaultDirName={pf}\\{#emit APP_NAME}\n DefaultGroupName={#emit APP_NAME}\n DisableReadyPage=yes\n-InfoBeforeFile=gpl-2.0.rtf\n+;InfoBeforeFile=gpl-2.0.rtf\n PrivilegesRequired=none\n UninstallDisplayIcon=etc\\git.ico\n \n@@ -36,13 +36,13 @@ Name: guiextension; Description: \"Add \"\"Git &GUI Here\"\"\"; GroupDescription: \"Win\n \n [Files]\n Source: \"*\"; DestDir: \"{app}\"; Excludes: \"\\*.bmp, gpl-2.0.rtf, \\install.*, \\tmp.*, \\bin\\*install*\"; Flags: recursesubdirs\n-Source: ReleaseNotes.rtf; DestDir: \"{app}\"; Flags: isreadme\n+;Source: ReleaseNotes.rtf; DestDir: \"{app}\"; Flags: isreadme\n \n [Icons]\n Name: \"{group}\\Git GUI\"; Filename: \"{app}\\bin\\wish.exe\"; Parameters: \"\"\"{app}\\bin\\git-gui\"\"\"; WorkingDir: \"%USERPROFILE%\"; IconFilename: \"{app}\\etc\\git.ico\"\n Name: \"{group}\\Git Bash\"; Filename: \"{syswow64}\\cmd.exe\"; Parameters: \"/c \"\"\"\"{app}\\bin\\sh.exe\"\" --login -i\"\"\"; WorkingDir: \"%USERPROFILE%\"; IconFilename: \"{app}\\etc\\git.ico\"\n Name: \"{group}\\Uninstall Git\"; Filename: \"{uninstallexe}\"\n-Name: \"{userappdata}\\Microsoft\\Internet Explorer\\Quick Launch\\Git Bash\"; Filename: \"{syswow64}\\cmd.exe\"; Parameters: \"/c \"\"\"\"{app}\\bin\\sh.exe\"\" --login -i\"\"\"; WorkingDir: \"%USERPROFILE%\"; IconFilename: \"{app}\\etc\\git.ico\"; Tasks: quicklaunchicon\n+Name: \"{userappdata}\\Microsoft\\Internet Explorer\\Quick Launch\\Git GUI\"; Filename: \"{app}\\bin\\wish.exe\"; Parameters: \"\"\"{app}\\bin\\git-gui\"\"\"; WorkingDir: \"%USERPROFILE%\"; IconFilename: \"{app}\\etc\\git.ico\"; Tasks: quicklaunchicon\n Name: \"{code:GetShellFolder|desktop}\\Git Bash\"; Filename: \"{syswow64}\\cmd.exe\"; Parameters: \"/c \"\"\"\"{app}\\bin\\sh.exe\"\" --login -i\"\"\"; WorkingDir: \"%USERPROFILE%\"; IconFilename: \"{app}\\etc\\git.ico\"; Tasks: desktopicon\n \n [Messages]\n@@ -171,8 +171,7 @@ begin\n end;\n \n var\n- EnvPage,PuTTYPage:TWizardPage;\n- RdbGitBash,RdbGitCmd,RdbGitCmdTools:TRadioButton;\n+ PuTTYPage:TWizardPage;\n RdbOpenSSH,RdbPLink:TRadioButton;\n EdtPLink:TEdit;\n \n@@ -197,105 +196,12 @@ end;\n \n procedure InitializeWizard;\n var\n- LblGitBash,LblGitCmd,LblGitCmdTools,LblGitCmdToolsWarn:TLabel;\n LblOpenSSH,LblPLink:TLabel;\n BtnPLink:TButton;\n begin\n- // Create a custom page for modifying the environment.\n- EnvPage:=CreateCustomPage(\n- wpSelectTasks,\n- 'Adjusting your PATH environment',\n- 'How would you like to use Git from the command line?'\n- );\n-\n- // 1st choice\n- RdbGitBash:=TRadioButton.Create(EnvPage);\n- with RdbGitBash do begin\n- Parent:=EnvPage.Surface;\n- Caption:='Use Git Bash only';\n- Left:=ScaleX(4);\n- Top:=ScaleY(8);\n- Width:=ScaleX(129);\n- Height:=ScaleY(17);\n- Font.Style:=[fsBold];\n- TabOrder:=0;\n- Checked:=True;\n- end;\n- LblGitBash:=TLabel.Create(EnvPage);\n- with LblGitBash do begin\n- Parent:=EnvPage.Surface;\n- Caption:=\n- 'This is the most conservative choice if you are concerned about the stability' + #13 +\n- 'of your system. Your PATH will not be modified.';\n- Left:=ScaleX(28);\n- Top:=ScaleY(32);\n- Width:=ScaleX(405);\n- Height:=ScaleY(26);\n- end;\n-\n- // 2nd choice\n- RdbGitCmd:=TRadioButton.Create(EnvPage);\n- with RdbGitCmd do begin\n- Parent:=EnvPage.Surface;\n- Caption:='Run Git from the Windows Command Prompt';\n- Left:=ScaleX(4);\n- Top:=ScaleY(76);\n- Width:=ScaleX(281);\n- Height:=ScaleY(17);\n- Font.Style:=[fsBold];\n- TabOrder:=1;\n- end;\n- LblGitCmd:=TLabel.Create(EnvPage);\n- with LblGitCmd do begin\n- Parent:=EnvPage.Surface;\n- Caption:=\n- 'This option is considered safe and no conflicts with other tools are known.' + #13 +\n- 'Only Git will be added to your PATH. Use this option if you want to use Git' + #13 +\n- 'from a Cygwin Prompt (make sure to not have Cygwin''s Git installed).';\n- Left:=ScaleX(28);\n- Top:=ScaleY(100);\n- Width:=ScaleX(405);\n- Height:=ScaleY(39);\n- end;\n-\n- // 3rd choice\n- RdbGitCmdTools:=TRadioButton.Create(EnvPage);\n- with RdbGitCmdTools do begin\n- Parent:=EnvPage.Surface;\n- Caption:='Run Git and included Unix tools from the Windows Command Prompt';\n- Left:=ScaleX(4);\n- Top:=ScaleY(152);\n- Width:=ScaleX(405);\n- Height:=ScaleY(17);\n- Font.Style:=[fsBold];\n- TabOrder:=2;\n- end;\n- LblGitCmdTools:=TLabel.Create(EnvPage);\n- with LblGitCmdTools do begin\n- Parent:=EnvPage.Surface;\n- Caption:='Both Git and its accompanying Unix tools will be added to your PATH.';\n- Left:=ScaleX(28);\n- Top:=ScaleY(176);\n- Width:=ScaleX(405);\n- Height:=ScaleY(13);\n- end;\n- LblGitCmdToolsWarn:=TLabel.Create(EnvPage);\n- with LblGitCmdToolsWarn do begin\n- Parent:=EnvPage.Surface;\n- Caption:=\n- 'Warning: This will override Windows tools like find.exe and' + #13 +\n- 'sort.exe. Select this option only if you understand the implications.';\n- Left:=ScaleX(28);\n- Top:=ScaleY(192);\n- Width:=ScaleX(376);\n- Height:=ScaleY(26);\n- Font.Color:=255;\n- Font.Style:=[fsBold];\n- end;\n-\n // Create a custom page for using PuTTY's plink instead of ssh.\n PuTTYPage:=CreateCustomPage(\n- EnvPage.ID,\n+ wpSelectTasks,\n 'Choosing the SSH executable',\n 'Which Secure Shell client program would you like Git to use?'\n );\n@@ -537,45 +443,6 @@ begin\n end;\n end;\n \n- // Modify the PATH variable as requested by the user.\n- if RdbGitCmd.Checked or RdbGitCmdTools.Checked then begin\n- i:=GetArrayLength(EnvPath);\n- SetArrayLength(EnvPath,i+1);\n-\n- // List \\cmd before \\bin so \\cmd has higher priority and programs in\n- // there will be called in favor of those in \\bin.\n- EnvPath[i]:=ExpandConstant('{app}\\cmd');\n-\n- if RdbGitCmdTools.Checked then begin\n- SetArrayLength(EnvPath,i+2);\n- EnvPath[i+1]:=ExpandConstant('{app}\\bin');\n-\n- // Set HOME for the Windows Command Prompt, but only if it has not been set manually before.\n- EnvHome:=GetEnvStrings('HOME',IsAdminLoggedOn);\n- i:=GetArrayLength(EnvHome);\n- if (i=0) or ((i=1) and (Length(EnvHome[0])=0)) then begin\n- SetArrayLength(EnvHome,1);\n- EnvHome[0]:=ExpandConstant('{%USERPROFILE}');\n- if not SetEnvStrings('HOME',IsAdminLoggedOn,True,EnvHome) then begin\n- Msg:='Line {#emit __LINE__}: Unable to set the HOME environment variable.';\n- MsgBox(Msg,mbError,MB_OK);\n- Log(Msg);\n- // This is not a critical error, the user can probably fix it manually,\n- // so we continue.\n- end;\n-\n- // Mark that we have changed HOME.\n- if not SetIniString('Environment','HOME',EnvHome[0],FileName) then begin\n- Msg:='Line {#emit __LINE__}: Unable to write to file \"'+FileName+'\".';\n- MsgBox(Msg,mbError,MB_OK);\n- Log(Msg);\n- // This is not a critical error, though uninstall / reinstall will probably not run cleanly,\n- // so we continue.\n- end;\n- end;\n- end;\n- end;\n-\n // Set the current user's PATH directories.\n if not SetEnvStrings('PATH',IsAdminLoggedOn,True,EnvPath) then begin\n Msg:='Line {#emit __LINE__}: Unable to set the PATH environment variable.';\n"},{"id":"92410","messageId":"20081006141840.GO10360@machine.or.cz","threadId":"15770","inReplyTo":"alpine.DEB.1.00.0810061621110.22125@pacific.mpi-cbg.de.mpi-cbg.de","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Petr Baudis","fromEmail":"pasky@suse.cz","sentAt":"2008-10-06T14:18:40Z","receivedAt":"2008-10-06T14:18:40Z","isPatch":true,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":" Hi,\n\nOn Mon, Oct 06, 2008 at 04:22:08PM +0200, Johannes Schindelin wrote:\n> On Fri, 3 Oct 2008, Petr Baudis wrote:\n> \n> > -InfoBeforeFile=gpl-2.0.rtf\n> \n> I'd rather keep it in, especially in a corporate environment.\n\n why? How is it relevant for the users?\n\n (Corporate environment or not, I actually never understood showing GPL\nin the installers. It always seemed to me that people just feel the need\nto fill the space where the EULA usually appears. ;-)\n\n\t\t\t\tPetr \"Pasky\" Baudis\n"},{"id":"92408","messageId":"alpine.DEB.1.00.0810061621110.22125@pacific.mpi-cbg.de.mpi-cbg.de","threadId":"15770","inReplyTo":"20081003122727.GE10360@machine.or.cz","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2008-10-06T14:22:08Z","receivedAt":"2008-10-06T14:22:08Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Fri, 3 Oct 2008, Petr Baudis wrote:\n\n> -InfoBeforeFile=gpl-2.0.rtf\n\nI'd rather keep it in, especially in a corporate environment.\n\nCiao,\nDscho\n"},{"id":"92417","messageId":"alpine.DEB.1.00.0810061718240.22125@pacific.mpi-cbg.de.mpi-cbg.de","threadId":"15770","inReplyTo":"20081006141840.GO10360@machine.or.cz","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2008-10-06T15:23:19Z","receivedAt":"2008-10-06T15:23:19Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Mon, 6 Oct 2008, Petr Baudis wrote:\n\n> Hi,\n> \n> On Mon, Oct 06, 2008 at 04:22:08PM +0200, Johannes Schindelin wrote:\n> > On Fri, 3 Oct 2008, Petr Baudis wrote:\n> > \n> > > -InfoBeforeFile=gpl-2.0.rtf\n> > \n> > I'd rather keep it in, especially in a corporate environment.\n> \n> why? How is it relevant for the users?\n\nTo give them an idea about their _rights_. Because they are entitled by \nthe GPL to get -- in source -- the changes that were done to that \nparticular distributions.\n\nSpeaking from experience, companies usually do not want to tell you about \nyour rights with regard to free software.\n\nAnd in this case, I have a personal interest. I feel that my efforts to \nthe msysGit project are not really rewarded. People like to use it, but \nthey hardly give anything back (you being a very notable exception).\n\nCiao,\nDscho\n"},{"id":"92422","messageId":"m3zllhpvby.fsf@localhost.localdomain","threadId":"15770","inReplyTo":"alpine.DEB.1.00.0810061718240.22125@pacific.mpi-cbg.de.mpi-cbg.de","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2008-10-06T16:00:37Z","receivedAt":"2008-10-06T16:00:37Z","isPatch":true,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Johannes Schindelin writes:\n> On Mon, 6 Oct 2008, Petr Baudis wrote:\n> > On Mon, Oct 06, 2008 at 04:22:08PM +0200, Johannes Schindelin wrote:\n> > > On Fri, 3 Oct 2008, Petr Baudis wrote:\n> > > \n> > > > -InfoBeforeFile=gpl-2.0.rtf\n> > > \n> > > I'd rather keep it in, especially in a corporate environment.\n> > \n> > why? How is it relevant for the users?\n> \n> To give them an idea about their _rights_. Because they are entitled by \n> the GPL to get -- in source -- the changes that were done to that \n> particular distributions.\n> \n> Speaking from experience, companies usually do not want to tell you about \n> your rights with regard to free software.\n> \n> And in this case, I have a personal interest. I feel that my efforts to \n> the msysGit project are not really rewarded. People like to use it, but \n> they hardly give anything back (you being a very notable exception).\n\nWouldn't it be better to keep it, but move it from EULA position\nto for example 'License' item in 'Help' entry (below 'About') in\ngitk and git-gui? IIRC there was a lot of controversy about Firefox\nusing EULA, although MS Windows users are most probably used to it...\n\n-- \nJakub Narebski\nPoland\nShadeHawk on #git\n"},{"id":"92424","messageId":"alpine.DEB.1.00.0810061810360.22125@pacific.mpi-cbg.de.mpi-cbg.de","threadId":"15770","inReplyTo":"m3zllhpvby.fsf@localhost.localdomain","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2008-10-06T16:12:15Z","receivedAt":"2008-10-06T16:12:15Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Mon, 6 Oct 2008, Jakub Narebski wrote:\n\n> Wouldn't it be better to keep it [showing the GPL], but move it from \n> EULA position to for example 'License' item in 'Help' entry (below \n> 'About') in gitk and git-gui? IIRC there was a lot of controversy about \n> Firefox using EULA, although MS Windows users are most probably used to \n> it...\n\nWell, I consider it a courtesy to the msysGit people to leave it where it \nis. Of course, this is free software and you are free to disregard my \nwish.\n\nCiao,\nDscho\n"},{"id":"92431","messageId":"20081006172348.GQ10360@machine.or.cz","threadId":"15770","inReplyTo":"alpine.DEB.1.00.0810061718240.22125@pacific.mpi-cbg.de.mpi-cbg.de","subject":"Re: [FYI][PATCH] Customizing the WinGit installer","fromName":"Petr Baudis","fromEmail":"pasky@suse.cz","sentAt":"2008-10-06T17:23:48Z","receivedAt":"2008-10-06T17:23:48Z","isPatch":true,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":"\n Hi,\n\nOn Mon, Oct 06, 2008 at 05:23:19PM +0200, Johannes Schindelin wrote:\n> To give them an idea about their _rights_. Because they are entitled by \n> the GPL to get -- in source -- the changes that were done to that \n> particular distributions.\n> \n> Speaking from experience, companies usually do not want to tell you about \n> your rights with regard to free software.\n> \n> And in this case, I have a personal interest. I feel that my efforts to \n> the msysGit project are not really rewarded. People like to use it, but \n> they hardly give anything back (you being a very notable exception).\n\n Maybe this is a little of a misunderstanding. This is not a \"consumer\"\naimed customized Git version. This is for internal Git usage within the\ncorporation, the users have varying interest in Git but it is generally\nvery low, they just regard it as another tool and they will never hack\nor customize it; I just don't see a point in bothering them with another\nextra installation step. Maybe in more general Git distribution I would\nagree this change indeed is not that appropriate (even though in my own\nGPL'd project, I wouldn't add the GPL step to the installer anyway).\n\n-- \n\t\t\t\tPetr \"Pasky\" Baudis\nPeople who take cold baths never have rheumatism, but they have\ncold baths.\n"},{"id":"92434","messageId":"alpine.LFD.2.00.0810061031380.3208@nehalem.linux-foundation.org","threadId":"15770","inReplyTo":"alpine.DEB.1.00.0810061810360.22125@pacific.mpi-cbg.de.mpi-cbg.de","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Linus Torvalds","fromEmail":"torvalds@linux-foundation.org","sentAt":"2008-10-06T17:34:23Z","receivedAt":"2008-10-06T17:34:23Z","isPatch":true,"sender":{"key":"torvalds@linux-foundation.org","avatar":"https://avatars.githubusercontent.com/u/1024025?v=4"},"body":"\n\nOn Mon, 6 Oct 2008, Johannes Schindelin wrote:\n> \n> Well, I consider it a courtesy to the msysGit people to leave it where \n> it is.\n\nUmm. I consider it to be UNACCEPTABLY STUPID to claim \"courtesy\" if it \nactually makes something just more irritating to users.\n\nI absolutely detest clicking through EULA's or other self-serving crap. I \nhate software that bothers me with their license details. Nobody reads \nthose things anyway, and an extra click or an extra window I didn't ask \nfor - _especially_ in open source software - is just a bug.\n\nBugs aren't \"courtesy\". Not to the user, and certainly not to the \ndeveloper.\n\n\t\t\tLinus\n"},{"id":"92437","messageId":"803aj9pq6o.fsf@tiny.isode.net","threadId":"15770","inReplyTo":"alpine.LFD.2.00.0810061031380.3208@nehalem.linux-foundation.org","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Bruce Stephens","fromEmail":"bruce.stephens@isode.com","sentAt":"2008-10-06T17:51:43Z","receivedAt":"2008-10-06T17:51:43Z","isPatch":true,"sender":{"key":"bruce.stephens@isode.com","avatar":null},"body":"Linus Torvalds writes:\n\n[...]\n\n> I absolutely detest clicking through EULA's or other self-serving crap. I \n> hate software that bothers me with their license details. Nobody reads \n> those things anyway, and an extra click or an extra window I didn't ask \n> for - _especially_ in open source software - is just a bug.\n\nBy coincidence, I recently bought an Asus EEE PC which comes with a\nQuick Use Guide whose first step on Launching First Run Wizard reads\n\"Toggle 'I agree with this license' and click 'Next' to continue.\",\nvery sensibly not bothering to suggest that the user read the license.\n\n[...]\n"},{"id":"92438","messageId":"b19eae4e0810061054l737960bes289af8adc9edc561@mail.gmail.com","threadId":"15770","inReplyTo":"alpine.LFD.2.00.0810061031380.3208@nehalem.linux-foundation.org","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Robin Burchell","fromEmail":"w00t@inspircd.org","sentAt":"2008-10-06T17:54:54Z","receivedAt":"2008-10-06T17:54:54Z","isPatch":true,"sender":{"key":"w00t@inspircd.org","avatar":null},"body":"On Mon, Oct 6, 2008 at 6:34 PM, Linus Torvalds\n wrote:\n>\n> I absolutely detest clicking through EULA's or other self-serving crap. I\n> hate software that bothers me with their license details. Nobody reads\n> those things anyway, and an extra click or an extra window I didn't ask\n> for - _especially_ in open source software - is just a bug.\n>\n\nI'll just jump in and add my (admittedly inexperienced) $0.02 -\nnamely, my confusion as to why the GPL would be displayed in place of\na EULA when ...it's not a EULA. It is not about how the software may\nbe used at all:\n\n Activities other than copying, distribution and modification are not\n covered by this License; they are outside its scope. The act of running\n the Program is not restricted, and the output from the Program is\n covered only if its contents constitute a work based on the Program\n (independent of having been made by running the Program). Whether that\n is true depends on what the Program does.\n"},{"id":"92439","messageId":"alpine.DEB.1.00.0810061959280.22125@pacific.mpi-cbg.de.mpi-cbg.de","threadId":"15770","inReplyTo":"alpine.LFD.2.00.0810061031380.3208@nehalem.linux-foundation.org","subject":"Re: [FYI][PATCH] Customizing the WinGit installer","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2008-10-06T18:01:38Z","receivedAt":"2008-10-06T18:01:38Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"\nHi,\n\nOn Mon, 6 Oct 2008, Linus Torvalds wrote:\n\n> On Mon, 6 Oct 2008, Johannes Schindelin wrote:\n> > \n> > Well, I consider it a courtesy to the msysGit people to leave it where \n> > it is.\n> \n> Umm. I consider it to be UNACCEPTABLY STUPID to claim \"courtesy\" if it \n> actually makes something just more irritating to users.\n> \n> I absolutely detest clicking through EULA's or other self-serving crap. \n> I hate software that bothers me with their license details. Nobody reads \n> those things anyway, and an extra click or an extra window I didn't ask \n> for - _especially_ in open source software - is just a bug.\n> \n> Bugs aren't \"courtesy\". Not to the user, and certainly not to the \n> developer.\n\nIt is not a bug. It is a message that tells the Windows user that this is \nGPLed software, and that they are free to copy it. Windows users (and \ndevelopers on Windows are more of users than developers) are often not \naware of that fact.\n\nThat is the reason why many Open Source projects do not bother on Linux, \nbut do bother on Windows.\n\nSo I guess we simply disagree here.\n\nCiao,\nDscho\n"},{"id":"92442","messageId":"alpine.LFD.2.00.0810061246460.3208@nehalem.linux-foundation.org","threadId":"15770","inReplyTo":"alpine.DEB.1.00.0810061959280.22125@pacific.mpi-cbg.de.mpi-cbg.de","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Linus Torvalds","fromEmail":"torvalds@linux-foundation.org","sentAt":"2008-10-06T19:47:16Z","receivedAt":"2008-10-06T19:47:16Z","isPatch":true,"sender":{"key":"torvalds@linux-foundation.org","avatar":"https://avatars.githubusercontent.com/u/1024025?v=4"},"body":"\n\nOn Mon, 6 Oct 2008, Johannes Schindelin wrote:\n>\n> It is not a bug.\n\nIf it's unwanted functionality, that's a bug. That's kind of the whole \n(and only) difference between \"bug\" and \"feature\". Do people want it?\n\nAnd Petr made it clear that he didn't want it. And I can certainly \nunderstand why - so it's not just some confused user. As such, it's \nclearly not a feature, and is clearly a bug.\n\n\t\t\tLinus\n"},{"id":"92446","messageId":"20081006201125.GI16289@zakalwe.fi","threadId":"15770","inReplyTo":"alpine.DEB.1.00.0810061959280.22125@pacific.mpi-cbg.de.mpi-cbg.de","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Heikki Orsila","fromEmail":"shdl@zakalwe.fi","sentAt":"2008-10-06T20:11:25Z","receivedAt":"2008-10-06T20:11:25Z","isPatch":true,"sender":{"key":"shdl@zakalwe.fi","avatar":null},"body":"On Mon, Oct 06, 2008 at 08:01:38PM +0200, Johannes Schindelin wrote:\n> It is not a bug. It is a message that tells the Windows user that this is \n> GPLed software, and that they are free to copy it. Windows users (and \n> developers on Windows are more of users than developers) are often not \n> aware of that fact.\n> \n> That is the reason why many Open Source projects do not bother on Linux, \n> but do bother on Windows.\n\nIt's pointless and harmful to present any non-relevant information for \nusers. I simply hate programs that waste my precious seconds. I want it \nNOW, not soon. Developers can tell the license if they want to \ndistribute it, but users do not need it at all.\n\n> So I guess we simply disagree here.\n\nI disagree VIOLENTLY with you. I've been utterly struck with this \nWindows crap here.. I spent a day installing stupid trivial software \nand answering pointless EULAs. I REALLY REALLY hate extra questions.. \nThis is my 20th reboot..\n\n-- \nHeikki Orsila\nheikki.orsila@iki.fi\nhttp://www.iki.fi/shd\n"},{"id":"92453","messageId":"48EA80D4.5030209@lyx.org","threadId":"15770","inReplyTo":"alpine.DEB.1.00.0810061959280.22125@pacific.mpi-cbg.de.mpi-cbg.de","subject":"Re: [FYI][PATCH] Customizing the WinGit installer","fromName":"Abdelrazak Younes","fromEmail":"younes@lyx.org","sentAt":"2008-10-06T21:19:16Z","receivedAt":"2008-10-06T21:19:16Z","isPatch":true,"sender":{"key":"younes@lyx.org","avatar":null},"body":"On 06/10/2008 20:01, Johannes Schindelin wrote:\n> Hi,\n>\n> On Mon, 6 Oct 2008, Linus Torvalds wrote:\n>\n>> On Mon, 6 Oct 2008, Johannes Schindelin wrote:\n>>> Well, I consider it a courtesy to the msysGit people to leave it where\n>>> it is.\n>> Umm. I consider it to be UNACCEPTABLY STUPID to claim \"courtesy\" if it\n>> actually makes something just more irritating to users.\n>>\n>> I absolutely detest clicking through EULA's or other self-serving crap.\n>> I hate software that bothers me with their license details. Nobody reads\n>> those things anyway, and an extra click or an extra window I didn't ask\n>> for - _especially_ in open source software - is just a bug.\n>>\n>> Bugs aren't \"courtesy\". Not to the user, and certainly not to the\n>> developer.\n>\n> It is not a bug. It is a message that tells the Windows user that this is\n> GPLed software, and that they are free to copy it. Windows users (and\n> developers on Windows are more of users than developers) are often not\n> aware of that fact.\n\nWell, maybe so called \"Windows pseudo-developers\" do not feel welcome to \nparticipate to this particular project? Or maybe this project has not \nreached yet the minimum visibility threshold? git has a lot of \ncontributors on Linux because it is visible there. Do you think all \nusers of git on Linux systematically contributes?\n\nDon't get me wrong, I am truly thankful for msysgit but, IMHO, if you \nproduce and publish a free software you should just accept and be glad \nthat it is used. Otherwise, just don't publish it.\n\n> That is the reason why many Open Source projects do not bother on Linux,\n> but do bother on Windows.\n\nAnd a lot other projects do not bother with this even on Windows and Mac \ntoo.\n\nRegards,\nAbdel.\n"},{"id":"92472","messageId":"20081007015336.GU21650@dpotapov.dyndns.org","threadId":"15770","inReplyTo":"alpine.LFD.2.00.0810061246460.3208@nehalem.linux-foundation.org","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Dmitry Potapov","fromEmail":"dpotapov@gmail.com","sentAt":"2008-10-07T01:53:36Z","receivedAt":"2008-10-07T01:53:36Z","isPatch":true,"sender":{"key":"dpotapov@gmail.com","avatar":"https://avatars.githubusercontent.com/u/6568595?v=4"},"body":"On Mon, Oct 06, 2008 at 12:47:16PM -0700, Linus Torvalds wrote:\n> \n> On Mon, 6 Oct 2008, Johannes Schindelin wrote:\n> >\n> > It is not a bug.\n> \n> If it's unwanted functionality, that's a bug. That's kind of the whole \n> (and only) difference between \"bug\" and \"feature\". Do people want it?\n\nIt looks like you have changed your opinion since what you wrote half\na year ago.\n\nBack then you said:\n> Yeah. Why not just rather than the whole ok/cancel discussion, go with a \n> single button saying \"good for me!\" and be done with it.\n> \n> IOW, the license thing should be considered *informational* rather than a \n> choice. Because to a user, that's exactly what the GPL is.\n\nhttp://article.gmane.org/gmane.comp.version-control.msysgit/1677\n\nAlso, please, re-read what Junio said in this thread:\n\n\"I think removing the license dialog is a bad idea. You need to\ntell the end-user about his rights, and one of the things is\nthat he can get source to git under the terms of GPLv2. The bug\nis not about showing the license, but is about refusing to\ninstall unless the end user agrees with it.\"\n\nI think such a step as removing the license should be approved by\nJunio and not be taken behind his back as it's happening now.\n\n\nDmitry\n"},{"id":"92473","messageId":"20081007015942.GV21650@dpotapov.dyndns.org","threadId":"15770","inReplyTo":"20081006201125.GI16289@zakalwe.fi","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Dmitry Potapov","fromEmail":"dpotapov@gmail.com","sentAt":"2008-10-07T01:59:42Z","receivedAt":"2008-10-07T01:59:42Z","isPatch":true,"sender":{"key":"dpotapov@gmail.com","avatar":"https://avatars.githubusercontent.com/u/6568595?v=4"},"body":"On Mon, Oct 06, 2008 at 11:11:25PM +0300, Heikki Orsila wrote:\n> \n> I disagree VIOLENTLY with you. I've been utterly struck with this \n> Windows crap here.. I spent a day installing stupid trivial software \n> and answering pointless EULAs. I REALLY REALLY hate extra questions.. \n\nGit installer should NOT ask you whether you agree with the license,\nbut it merely shows you the license. (At least, it is what was decided\nhalf a year ago).\n\n> This is my 20th reboot..\n\nI am pretty sure that 20th reboot has nothing to do with how many times\nthe license has been shown...\n\nDmitry\n"},{"id":"92476","messageId":"alpine.LFD.2.00.0810061909260.3208@nehalem.linux-foundation.org","threadId":"15770","inReplyTo":"20081007015336.GU21650@dpotapov.dyndns.org","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Linus Torvalds","fromEmail":"torvalds@linux-foundation.org","sentAt":"2008-10-07T02:10:37Z","receivedAt":"2008-10-07T02:10:37Z","isPatch":true,"sender":{"key":"torvalds@linux-foundation.org","avatar":"https://avatars.githubusercontent.com/u/1024025?v=4"},"body":"\n\nOn Tue, 7 Oct 2008, Dmitry Potapov wrote:\n> \n> It looks like you have changed your opinion since what you wrote half\n> a year ago.\n\nBack then, nobody had really complained and sent in a patch to make it \noptional.\n\nThat changes things. Once some user actually complains, and sends in a \nfix to make the whole dialog optional, I don't see why anybody would ever \nargue against such a patch being accepted.\n\n\t\tLinus\n"},{"id":"92479","messageId":"32541b130810061940y4864c420h1a6504b843d4e746@mail.gmail.com","threadId":"15770","inReplyTo":"20081007015942.GV21650@dpotapov.dyndns.org","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Avery Pennarun","fromEmail":"apenwarr@gmail.com","sentAt":"2008-10-07T02:40:53Z","receivedAt":"2008-10-07T02:40:53Z","isPatch":true,"sender":{"key":"apenwarr@gmail.com","avatar":"https://avatars.githubusercontent.com/u/20592?v=4"},"body":"On Mon, Oct 6, 2008 at 9:59 PM, Dmitry Potapov wrote:\n> On Mon, Oct 06, 2008 at 11:11:25PM +0300, Heikki Orsila wrote:\n>>\n>> I disagree VIOLENTLY with you. I've been utterly struck with this\n>> Windows crap here.. I spent a day installing stupid trivial software\n>> and answering pointless EULAs. I REALLY REALLY hate extra questions..\n>\n> Git installer should NOT ask you whether you agree with the license,\n> but it merely shows you the license. (At least, it is what was decided\n> half a year ago).\n\nFWIW, the only reason *installers* ever show licenses is because they\nwant to give you *additional* restrictions on top of normal copyright,\nand making you \"click through\" the license before using the\napplication might (or might not) make those restrictions more legally\nenforceable than simply including that license along with the program.\n\nInstallers don't show you the license to increase visibility; they do\nit for legal reasons.\n\nThe most visible place to put a license is in either a splash screen\nor a Help|About dialog. If I'm using a program and I want to know the\nlicense, that's where I look.\n\nThis whole discussion is silly because end users, who are the ones we\nclaim to be worried about, will not be the ones running the installer\nanyway at a large company. It'll be the IT department guy, who\nalready knows the license and will have to run the installer 500\ntimes. And if it's not a large company, individuals will see the\nlicense when they go to download the software.\n\nHave fun,\n\nAvery\n"},{"id":"92480","messageId":"20081007030824.GW21650@dpotapov.dyndns.org","threadId":"15770","inReplyTo":"alpine.LFD.2.00.0810061909260.3208@nehalem.linux-foundation.org","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Dmitry Potapov","fromEmail":"dpotapov@gmail.com","sentAt":"2008-10-07T03:08:24Z","receivedAt":"2008-10-07T03:08:24Z","isPatch":true,"sender":{"key":"dpotapov@gmail.com","avatar":"https://avatars.githubusercontent.com/u/6568595?v=4"},"body":"On Mon, Oct 06, 2008 at 07:10:37PM -0700, Linus Torvalds wrote:\n> \n> That changes things. Once some user actually complains, and sends in a \n> fix to make the whole dialog optional, I don't see why anybody would ever \n> argue against such a patch being accepted.\n\nFirst, he did not complain. He did not even mention that in the commit\nmessage. He mentioning some other things like removing release notes,\nbut not the license. Second, I would expect that any change that goes\nagainst the previous achieved agreement may deserve some discussion,\nand not blindly accepted just because the user sent a patch. Okay, I\ndon't really care whether the installer shows the license or not...\nPerhaps, something like Help|License would be a better place for it.\n\nDmitry\n"},{"id":"92503","messageId":"20081007074228.GJ16289@zakalwe.fi","threadId":"15770","inReplyTo":"20081007015942.GV21650@dpotapov.dyndns.org","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Heikki Orsila","fromEmail":"shdl@zakalwe.fi","sentAt":"2008-10-07T07:42:28Z","receivedAt":"2008-10-07T07:42:28Z","isPatch":true,"sender":{"key":"shdl@zakalwe.fi","avatar":null},"body":"On Tue, Oct 07, 2008 at 05:59:42AM +0400, Dmitry Potapov wrote:\n> On Mon, Oct 06, 2008 at 11:11:25PM +0300, Heikki Orsila wrote:\n> > \n> > I disagree VIOLENTLY with you. I've been utterly struck with this \n> > Windows crap here.. I spent a day installing stupid trivial software \n> > and answering pointless EULAs. I REALLY REALLY hate extra questions.. \n> \n> Git installer should NOT ask you whether you agree with the license,\n> but it merely shows you the license. (At least, it is what was decided\n> half a year ago).\n\nIt shouldn't even show that. Also, decisions can be changed..\n\n> > This is my 20th reboot..\n> \n> I am pretty sure that 20th reboot has nothing to do with how many times\n> the license has been shown...\n\nWell, I've clicked through annoying \"I accept\" EULAs at least that many \ntimes already..\n\n-- \nHeikki Orsila\nheikki.orsila@iki.fi\nhttp://www.iki.fi/shd\n"},{"id":"92504","messageId":"20081007074642.GK16289@zakalwe.fi","threadId":"15770","inReplyTo":"20081007030824.GW21650@dpotapov.dyndns.org","subject":"Re: [msysGit] [FYI][PATCH] Customizing the WinGit installer","fromName":"Heikki Orsila","fromEmail":"shdl@zakalwe.fi","sentAt":"2008-10-07T07:46:42Z","receivedAt":"2008-10-07T07:46:42Z","isPatch":true,"sender":{"key":"shdl@zakalwe.fi","avatar":null},"body":"On Tue, Oct 07, 2008 at 07:08:24AM +0400, Dmitry Potapov wrote:\n> On Mon, Oct 06, 2008 at 07:10:37PM -0700, Linus Torvalds wrote:\n> > \n> > That changes things. Once some user actually complains, and sends in a \n> > fix to make the whole dialog optional, I don't see why anybody would ever \n> > argue against such a patch being accepted.\n> \n> First, he did not complain. He did not even mention that in the commit\n> message.\n\nWhat does that matter? People are complaining now.\n\n> He mentioning some other things like removing release notes,\n> but not the license. Second, I would expect that any change that goes\n> against the previous achieved agreement may deserve some discussion,\n> and not blindly accepted just because the user sent a patch.\n\nAre you thinking this is some kind of decision by committee \norganization? NO. Patches are picked up by maintainers. Anyone can \ncomment on the patches as they are published.\n\n> Okay, I\n> don't really care whether the installer shows the license or not...\n\nThen why argue about it? I care very much.\n\n-- \nHeikki Orsila\nheikki.orsila@iki.fi\nhttp://www.iki.fi/shd\n"},{"id":"92505","messageId":"20081007092819.GW10360@machine.or.cz","threadId":"15770","inReplyTo":"alpine.LFD.2.00.0810061909260.3208@nehalem.linux-foundation.org","subject":"Re: [msysGit] Re: [FYI][PATCH] Customizing the WinGit installer","fromName":"Petr Baudis","fromEmail":"pasky@suse.cz","sentAt":"2008-10-07T09:28:19Z","receivedAt":"2008-10-07T09:28:19Z","isPatch":true,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":"On Mon, Oct 06, 2008 at 07:10:37PM -0700, Linus Torvalds wrote:\n> On Tue, 7 Oct 2008, Dmitry Potapov wrote:\n> > \n> > It looks like you have changed your opinion since what you wrote half\n> > a year ago.\n> \n> Back then, nobody had really complained and sent in a patch to make it \n> optional.\n> \n> That changes things. Once some user actually complains, and sends in a \n> fix to make the whole dialog optional, I don't see why anybody would ever \n> argue against such a patch being accepted.\n\nNote that as mentioned in the original mail, my patch was not meant\nfor application upstream, just as an example for others who would like\nto customize the Git installer for a particular environment. We wanted\nto make Git installation/usage as simple as possible, reducing any\nunnecessary steps we could - and this was an easy one.\n\nI don't think the idea of showing GPL during installation makes\na lot of sense on its own, but I don't care much either and clicking\nthrough licences and EULAs in wizards is a fact of life on Windows,\nsadly. At least it doesn't require you to scroll down through the\nlicence as IIRC OpenOffice or GIMP or a similar beast did.\n\n-- \n\t\t\t\tPetr \"Pasky\" Baudis\nPeople who take cold baths never have rheumatism, but they have\ncold baths.\n"},{"id":"92603","messageId":"7vd4ibr0xp.fsf@gitster.siamese.dyndns.org","threadId":"15770","inReplyTo":"20081007092819.GW10360@machine.or.cz","subject":"Re: [msysGit] Re: [FYI][PATCH] Customizing the WinGit installer","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-10-08T07:38:42Z","receivedAt":"2008-10-08T07:38:42Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Petr Baudis writes:\n\n> Note that as mentioned in the original mail, my patch was not meant\n> for application upstream, just as an example for others who would like\n> to customize the Git installer for a particular environment. We wanted\n> to make Git installation/usage as simple as possible, reducing any\n> unnecessary steps we could - and this was an easy one.\n\nI do not personally think showing the license is necessary in the\ninstaller, but unlike the earlier one that refused to proceed until the\nuser says \"I Accept\" (which was just plain silly), I think the current one\nis Ok.\n\nThe details are entirely up to people who have actually been involved in\nmsysgit packaging work, but it may be possible to package an installer\nthat can be used for installing multiple machines with minimum\ninteraction, and make the process of extracting that installer show\nnotices on licensing and legalese.\n"}]}