Remove LibGit2 dep

Project.toml

@@ -2,16 +2,14 @@ name = "DocStringExtensions"
 uuid = "ffbed154-4ef7-542d-bbb7-c09d3a79fcae"
 version = "0.9.3"
 
-[deps]
-LibGit2 = "76f85450-5226-5b5a-8eaa-529ad045b433"
-
 [compat]
 julia = "1"
 
 [extras]
+LibGit2 = "76f85450-5226-5b5a-8eaa-529ad045b433"
 Markdown = "d6f4376e-aef5-505a-96c1-9c027394607a"
 Pkg = "44cfe95a-1eb2-52ea-b672-e2afdf69b78f"
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"
 
 [targets]
-test = ["Markdown", "Pkg", "Test"]
+test = ["LibGit2", "Markdown", "Pkg", "Test"]

src/DocStringExtensions.jl

@@ -75,7 +75,6 @@ module DocStringExtensions
 
 # Imports.
 
-import LibGit2
 
 # Exports.
 

src/utilities.jl

@@ -517,51 +517,16 @@ end
 """
 $(:SIGNATURES)
 
-Get the URL (file and line number) where a method `m` is defined.
+Get the URL (file and line number) where a method `m` is defined. May return an empty string
+if the URL can not be determined.
 
-Note that this is based on the implementation of `Base.url`, but handles URLs correctly
-on TravisCI as well.
+Requires the `LibGit2` to be present in the environment.
 """
-url(m::Method) = url(m.module, string(m.file), m.line)
-
-function url(mod::Module, file::AbstractString, line::Integer)
-    file = Sys.iswindows() ? replace(file, '\\' => '/') : file
-    if Base.inbase(mod) && !isabspath(file)
-        local base = "https://github.com/JuliaLang/julia/tree"
-        if isempty(Base.GIT_VERSION_INFO.commit)
-            return "$base/v$VERSION/base/$file#L$line"
-        else
-            local commit = Base.GIT_VERSION_INFO.commit
-            return "$base/$commit/base/$file#L$line"
-        end
-    else
-        if isfile(file)
-            local d = dirname(file)
-            try # might not be in a git repo
-                LibGit2.with(LibGit2.GitRepoExt(d)) do repo
-                    LibGit2.with(LibGit2.GitConfig(repo)) do cfg
-                        local u = LibGit2.get(cfg, "remote.origin.url", "")
-                        local m = match(LibGit2.GITHUB_REGEX, u)
-                        u = m === nothing ? get(ENV, "TRAVIS_REPO_SLUG", "") : m.captures[1]
-                        local commit = string(LibGit2.head_oid(repo))
-                        local root = LibGit2.path(repo)
-                        if startswith(file, root) || startswith(realpath(file), root)
-                            local base = "https://github.com/$u/tree"
-                            local filename = file[(length(root) + 1):end]
-                            return "$base/$commit/$filename#L$line"
-                        else
-                            return ""
-                        end
-                    end
-                end
-            catch err
-                isa(err, LibGit2.GitError) || rethrow()
-                return ""
-            end
-        else
-            return ""
-        end
-    end
+function url(m::Method)
+    # Base.url() something produces file:// URLs, but we want these to be sharable web URLs.
+    # So we just say we couldn't determine the URL if we get one of those.
+    url = Base.url(m)
+    return startswith(url, "file://") ? "" : url
 end
 
 # This is compat to make sure that we have ismutabletype available pre-1.7.