Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Added Juju 1.12.0 stable #21858

Closed
wants to merge 3 commits into from
Closed

Added Juju 1.12.0 stable #21858

wants to merge 3 commits into from

Conversation

rochacon
Copy link
Contributor

Juju reached the 1.12.0 stable [1] version. As discussed on #21185 this Formula will compile Juju from source.

[1] https://launchpad.net/~juju/+archive/stable

@@ -0,0 +1,31 @@
require 'formula'

BUILD="1"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

inline this stuff

@rochacon
Copy link
Contributor Author

@adamv done. 😄

@adamv
Copy link
Contributor

adamv commented Aug 13, 2013

Does the go build pull down any dependencies?

@rochacon
Copy link
Contributor Author

No, all sources needed are included in the tarball.

@adamv
Copy link
Contributor

adamv commented Aug 15, 2013

@BrewTestBot test this please

@rochacon
Copy link
Contributor Author

@adamv I've added a devel block to use the latest development version, is it ok to push to this pull request? Here is the diff:

diff --git a/Library/Formula/juju.rb b/Library/Formula/juju.rb
index 6fd8c7b..5584933 100644
--- a/Library/Formula/juju.rb
+++ b/Library/Formula/juju.rb
@@ -9,6 +9,11 @@ class Juju < Formula

   depends_on 'go' => :build

+  devel do
+    url "https://launchpad.net/juju-core/trunk/1.13.1/+download/juju-core_1.13.1.tar.gz"
+    sha1 "1c1346378763d6aa469c434c1467b8ea4f8530be"
+  end
+
   def install
     ENV['GOPATH'] = buildpath

@adamv
Copy link
Contributor

adamv commented Aug 15, 2013

I've asked the build bot to run a test on this.

require 'formula'

class Juju < Formula
homepage 'https://launchpad.net/juju-core'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Homepage should be more useful: https://juju.ubuntu.com/

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed.

@MikeMcQuaid
Copy link
Member

@BrewTestBot test this please

@adamv adamv closed this in 4d8c818 Aug 17, 2013
lkollar pushed a commit to lkollar/homebrew that referenced this pull request Aug 18, 2013
Closes Homebrew#21858.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
@jrwren
Copy link

jrwren commented Aug 19, 2013

Huge thanks! Well done!

handyman5 pushed a commit to handyman5/homebrew that referenced this pull request Oct 7, 2013
Closes Homebrew#21858.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
@Homebrew Homebrew locked and limited conversation to collaborators Feb 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants