-
-
Notifications
You must be signed in to change notification settings - Fork 12.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
newnode 2.1.5 (new formula) #162683
newnode 2.1.5 (new formula) #162683
Changes from all commits
cc01a26
9d53b49
1003532
4984c90
669f2db
3d132b8
3fdbb12
5777bd3
1ece97d
a726984
032c4d8
47a4614
92b2426
6fd2073
bef4833
480e96e
83028ed
69533d0
73f531a
f7160d8
b57c191
d282a5e
42a477f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,65 @@ | ||||||||||||||||||||||||||
class Newnode < Formula | ||||||||||||||||||||||||||
desc "Web proxy that uses a distributed p2p network to circumvent censorship" | ||||||||||||||||||||||||||
homepage "https://www.newnode.com/newnode-vpn" | ||||||||||||||||||||||||||
url "https://github.com/clostra/newnode.git", | ||||||||||||||||||||||||||
tag: "2.1.5", | ||||||||||||||||||||||||||
revision: "18a3f713267b2a08e34cbe04253df891889997a2" | ||||||||||||||||||||||||||
license "GPL-2.0-only" | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
depends_on "autoconf" => :build | ||||||||||||||||||||||||||
depends_on "automake" => :build | ||||||||||||||||||||||||||
depends_on "coreutils" => :build | ||||||||||||||||||||||||||
depends_on "libtool" => :build | ||||||||||||||||||||||||||
depends_on "pkg-config" => :build | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
on_macos do | ||||||||||||||||||||||||||
depends_on xcode: ["9.3", :build] | ||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||
Comment on lines
+15
to
+17
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You don't need the |
||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
on_linux do | ||||||||||||||||||||||||||
depends_on "gnu-sed" => :build | ||||||||||||||||||||||||||
depends_on "grep" => :build | ||||||||||||||||||||||||||
Comment on lines
+20
to
+21
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
It's unlikely that these are needed, since |
||||||||||||||||||||||||||
depends_on "llvm" => :build | ||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Makes this uses_from_macos "llvm" => :build (outside of the |
||||||||||||||||||||||||||
depends_on "wget" | ||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
fails_with :gcc do | ||||||||||||||||||||||||||
cause "Requires -fblocks support" | ||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
def install | ||||||||||||||||||||||||||
if OS.linux? | ||||||||||||||||||||||||||
inreplace "https_wget.c", "/usr/bin/wget", "#{Formula["wget"].opt_bin}/wget" | ||||||||||||||||||||||||||
ENV.prepend_path "PATH", Formula["gnu-sed"].opt_libexec/"gnubin" | ||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||
Comment on lines
+31
to
+34
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
unless
Suggested change
|
||||||||||||||||||||||||||
system "./build.sh" | ||||||||||||||||||||||||||
bin.install "client" => "newnode" | ||||||||||||||||||||||||||
path = (var/"cache/newnode") | ||||||||||||||||||||||||||
path.mkpath | ||||||||||||||||||||||||||
path.chmod 0775 | ||||||||||||||||||||||||||
SMillerDev marked this conversation as resolved.
Show resolved
Hide resolved
Comment on lines
+37
to
+39
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should probably be in a |
||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
service do | ||||||||||||||||||||||||||
run [opt_bin/"newnode", "-p", "8006"] | ||||||||||||||||||||||||||
keep_alive true | ||||||||||||||||||||||||||
working_dir var/"cache/newnode" | ||||||||||||||||||||||||||
log_path var/"log/newnode.log" | ||||||||||||||||||||||||||
error_log_path var/"log/newnode-error.log" | ||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
test do | ||||||||||||||||||||||||||
port = free_port | ||||||||||||||||||||||||||
pid = fork do | ||||||||||||||||||||||||||
exec bin/"newnode", "-p", port.to_s | ||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||
sleep 5 | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
ENV["https_proxy"] = "http://localhost:#{port}" | ||||||||||||||||||||||||||
begin | ||||||||||||||||||||||||||
system "curl", "https://brew.sh" | ||||||||||||||||||||||||||
ensure | ||||||||||||||||||||||||||
sleep 5 | ||||||||||||||||||||||||||
Process.kill("TERM", pid) | ||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||
Comment on lines
+58
to
+63
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
I don't think you need the |
||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Align the arguments of
url
. Though, we really should use a tarball instead of a Git checkout.