[release-0.4] Pkg.publish() failing

[sbromberger]

julia> Pkg.tag("LightGraphs")
INFO: Tagging LightGraphs v0.5.1
INFO: Committing METADATA for LightGraphs

julia> Pkg.publish()
INFO: Validating METADATA
INFO: Pushing LightGraphs permanent tags: v0.5.1
Warning: Permanently added the RSA host key for IP address '192.30.252.129' to the list of known hosts.
INFO: Submitting METADATA changes
INFO: Forking JuliaLang/METADATA.jl to sbromberger
ERROR: strangely formatted HTTP response

[toivoh]

I got the same thing a week ago.

[sbromberger]

@toivoh did you resolve it? (How?)

[yuyichao]

Given the error message is not Pkg.publish() has been moved to the package PkgDev.jl. I assume this is release-0.4?

[sbromberger]

Sorry - yes - this is 0.4:

Julia Version 0.4.4-pre+35
Commit 1a9429e* (2016-02-12 13:21 UTC)
Platform Info:
  System: Darwin (x86_64-apple-darwin15.3.0)
  CPU: Intel(R) Core(TM) i7-5557U CPU @ 3.10GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Haswell)
  LAPACK: libopenblas64_
  LIBM: libopenlibm
  LLVM: libLLVM-svn

[tkelman]

https or git protocol?

[sbromberger]

@tkelman - how do I tell? ~/.julia/v0.4/METADATA/.git/config shows url = https://github.com/JuliaLang/METADATA.jl.git.

[toivoh]

I resolved it by manually creating a pull request based on the changes in the METADATA repo. So, not the ideal solution.

[tkelman]

Since the error was "strangely formatted HTTP response," this might be due to something changing on github's API. Worth trying to reconstruct the same curl command that Pkg.publish was attempting to do and see what the full response looks like.

[sbromberger]

@tkelman can you post the command? I'm happy to try it.

[tkelman]

Probably something like curl -i -u $(Pkg.GitHub.token()):x-oauth-basic https://api.github.com/repos/JuliaLang/METADATA.jl/forks

[sbromberger]

Output: https://gist.github.com/sbromberger/6c2c83f185e972f9c48d

Looks like JSON.

[tkelman]

It is JSON - it then runs through

julia/base/pkg/github.jl

Lines 40 to 53 in 147d6d0

	out, proc = open(`curl -i -s -S $opts $url`,"r")
	head = readline(out)
	status = parse(Int,split(head,r"\s+";limit=3)[2])
	header = Dict{AbstractString,AbstractString}()
	for line in eachline(out)
	if !ismatch(r"^\s*$",line)
	(k,v) = split(line, r":\s*"; limit=2)
	header[k] = v
	continue
	end
	wait(proc); return status, header, readall(out)
	end
	end
	error("strangely formatted HTTP response")

[wildart]

I think it was some kind of problem with the response. Most likely it was empty.

@sbromberger Do you still have this problem?

So far following GitHub repo fork code works without problem, and creates fork for me.

PkgDev.GitHub.POST("repos/JuliaLang/Example.jl/forks")

[slundberg]

@tkelman I think this may be caused by c0d41c9 since after moving the return statement inside a do-block there is now no way for the function to not hit the error at the end. This is likely only effecting the current 0.4 release, not the dev 0.5 version.

[tkelman]

huh? return from inside a do block should work fine? if not that seems like a bigger bug

[tkelman]

Oh I think I see what you mean, the return is from the anonymous function inside the do block. Wish you had figured that out before I tagged 0.4.4. And that we had some automated tests of publish, register, etc.

[slundberg]

Sorry my first comment could have been clearer. But yes, that's what I was getting at. I agree, hopefully the new PkgDev will end up with more unit tests etc.