Merge lp:~dobey/ubuntuone-windows-installer/update-from-trunk into lp:ubuntuone-windows-installer/stable-3-0

Proposed by dobey
Status: Merged
Approved by: dobey
Approved revision: 106
Merged at revision: 106
Proposed branch: lp:~dobey/ubuntuone-windows-installer/update-from-trunk
Merge into: lp:ubuntuone-windows-installer/stable-3-0
Diff against target: 48 lines (+38/-0)
1 file modified
scripts/ubuntuone.xml (+38/-0)
To merge this branch: bzr merge lp:~dobey/ubuntuone-windows-installer/update-from-trunk
Reviewer Review Type Date Requested Status
Roberto Alsina (community) Approve
Review via email: mp+101466@code.launchpad.net

Commit message

[Brian Curtin]

    - Add Windows firewall exceptions to the installer.

To post a comment you must log in.
Revision history for this message
Roberto Alsina (ralsina) wrote :

We need to raise the version to 3.0.0 in ubuntuone.xml and probably other places, I will do a patch tomorrow morning.

Revision history for this message
Roberto Alsina (ralsina) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'scripts/ubuntuone.xml'
--- scripts/ubuntuone.xml 2012-03-23 21:02:45 +0000
+++ scripts/ubuntuone.xml 2012-04-11 00:47:18 +0000
@@ -226,6 +226,44 @@
226 <type>REG_SZ</type>226 <type>REG_SZ</type>
227 <value>${installdir}\dist\ubuntu-sso-login.exe</value>227 <value>${installdir}\dist\ubuntu-sso-login.exe</value>
228 </registrySet>228 </registrySet>
229 <setInstallerVariableFromRegEx>
230 <!-- Help get the paths correct on Windows 7
231 netsh complains about path separators from installdir -->
232 <name>forwardslash_installdir</name>
233 <pattern>/</pattern>
234 <substitution>\</substitution>
235 <text>${installdir}</text>
236 </setInstallerVariableFromRegEx>
237 <runProgram>
238 <!-- Inbound Firewall allowance -->
239 <program>netsh.exe</program>
240 <programArguments>advfirewall firewall add rule name="Ubuntu One" dir=in action=allow program="${forwardslash_installdir}\dist\ubuntuone-syncdaemon.exe" enable=yes</programArguments>
241 <ruleEvaluationLogic>or</ruleEvaluationLogic>
242 <ruleList>
243 <!-- This usage of netsh is for Vista and 7. -->
244 <platformTest type="windows-vista" />
245 <platformTest type="windows-7" />
246 </ruleList>
247 </runProgram>
248 <runProgram>
249 <!-- Outbound Firewall allowance -->
250 <program>netsh.exe</program>
251 <programArguments>advfirewall firewall add rule name="Ubuntu One" dir=out action=allow program="${forwardslash_installdir}\dist\ubuntuone-syncdaemon.exe" enable=yes</programArguments>
252 <ruleEvaluationLogic>or</ruleEvaluationLogic>
253 <ruleList>
254 <!-- This usage of netsh is for Vista and 7. -->
255 <platformTest type="windows-vista" />
256 <platformTest type="windows-7" />
257 </ruleList>
258 </runProgram>
259 <runProgram>
260 <program>netsh.exe</program>
261 <programArguments>firewall add allowedprogram "${installdir}\dist\ubuntuone-syncdaemon.exe" "Ubuntu One" ENABLE</programArguments>
262 <ruleList>
263 <!-- This usage of netsh is for XP. -->
264 <platformTest type="windows-xp" />
265 </ruleList>
266 </runProgram>
229 <runProgram>267 <runProgram>
230 <program>${windows_folder_system}\RunDll32.exe</program>268 <program>${windows_folder_system}\RunDll32.exe</program>
231 <programArguments>"${installdir}\dist\VistaLib32.dll" RunNonElevated "${installdir}\dist\ubuntuone-control-panel-qt.exe" --installer --with-icon &amp;</programArguments>269 <programArguments>"${installdir}\dist\VistaLib32.dll" RunNonElevated "${installdir}\dist\ubuntuone-control-panel-qt.exe" --installer --with-icon &amp;</programArguments>

Subscribers

People subscribed via source and target branches

to all changes: