Juju Website

Merge lp:~elmo/juju-website/juju-website into lp:juju-website

  1. juju-website
  2. Merge into juju-website
Proposed by James Troup
Status: Merged
Merge reported by: Marco Ceppi
Merged at revision: not available
Proposed branch: lp:~elmo/juju-website/juju-website
Merge into: lp:juju-website
Diff against target: 18 lines (+2/-2)
1 file modified
template-get-started.php (+2/-2)
To merge this branch: bzr merge lp:~elmo/juju-website/juju-website
Reviewer Review Type Date Requested Status
Marco Ceppi (community) Approve
Review via email: mp+196457@code.launchpad.net

Description of the change

https://juju.ubuntu.com/install/ currently says:

  sudo apt-get update & sudo apt-get install juju-core

which ends up running the two commands at once which isn't what was
intended or expected. This patch quotes the '&' characters to try and
fix this.

To post a comment you must log in.
Revision history for this message
Marco Ceppi (marcoceppi) wrote :

Hi Elmo, I've forwarded this on to the design team to update. The website should reflect this shortly.

review: Approve
Revision history for this message
Anthony Dillon (ya-bo-ng) wrote :

The mark up for this page is in the wordpress CMS. I have updated the single & to a double &&.

Thank you,
Ant.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'template-get-started.php'
2--- template-get-started.php 2013-07-09 15:38:27 +0000
3+++ template-get-started.php 2013-11-25 02:08:42 +0000
4@@ -20,12 +20,12 @@
5 <p><code>sudo apt-get install juju-core</code></p>
6 <p>If you are using 12.04 you need to use the PPA:</p>
7
8- <p><code>sudo add-apt-repository ppa:juju/devel<br/>sudo apt-get update && sudo apt-get install juju-core</code></p>
9+ <p><code>sudo add-apt-repository ppa:juju/devel<br/>sudo apt-get update &amp;&amp; sudo apt-get install juju-core</code></p>
10 </a>
11
12 <p>sudo apt-get install juju-core</code></p>
13 <p>Juju is provided automatically as a backport in Ubuntu Server. Backports are usually enabled by default, but if you've turned them off here's how you can <a href="http://askubuntu.com/a/285474/235">enable them</a>. If apt complains that it cannot find the packages you might need to use the PPA:</p>
14- <p><code>sudo add-apt-repository ppa:juju/devel<br/>sudo apt-get update && sudo apt-get install juju-core</code></p>
15+ <p><code>sudo add-apt-repository ppa:juju/devel<br/>sudo apt-get update &amp;&amp; sudo apt-get install juju-core</code></p>
16 <p>Regular updates will be made to the backported packages, unless you want to follow along with Juju development we recommend users to not use the PPA.</p>
17 <p>You can switch between the default (0.7) version and 1.10 versions by using the following:</p>
18 <p><code>sudo update-alternatives --set juju /usr/lib/juju-0.7/bin/juju</code></p>

Subscribers

People subscribed via source and target branches

to all changes: