Commit 24711376 authored by Fabio Pelosin's avatar Fabio Pelosin

Merge pull request #436 from cameronboehmer/master

Better spec source validation
parents 799e0afd c717eae0
...@@ -296,7 +296,7 @@ module Pod ...@@ -296,7 +296,7 @@ module Pod
messages << "Missing summary" unless spec.summary messages << "Missing summary" unless spec.summary
messages << "Missing homepage" unless spec.homepage messages << "Missing homepage" unless spec.homepage
messages << "Missing author(s)" unless spec.authors messages << "Missing author(s)" unless spec.authors
messages << "Missing source" unless spec.source messages << "Missing or invalid source: #{spec.source}" unless source_valid?
# attributes with multiplatform values # attributes with multiplatform values
return messages unless platform_valid? return messages unless platform_valid?
...@@ -315,6 +315,10 @@ module Pod ...@@ -315,6 +315,10 @@ module Pod
!spec.platform || [:ios, :osx].include?(spec.platform.name) !spec.platform || [:ios, :osx].include?(spec.platform.name)
end end
def source_valid?
spec.source && !(spec.source =~ /http:\/\/EXAMPLE/)
end
def paths_starting_with_a_slash_errors def paths_starting_with_a_slash_errors
messages = [] messages = []
%w[source_files resources clean_paths].each do |accessor| %w[source_files resources clean_paths].each do |accessor|
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment