search

dbuenzli / topkg

The transitory OCaml software packager
http://erratique.ch/software/topkg
ISC License
68 stars 25 forks source link

Try to use a better remote path for uploading documentation #46

Closed samoht closed 8 years ago

samoht commented 8 years ago

This is a quick patch to make tokpg publish doc work a bit better. This doesn't work yet but feedback is welcome :-)

topkg: [INFO] Parsing OPAM file opam
topkg: [EXEC:17049] ["which"; "github-topkg-delegate"]
topkg: [EXEC:17050] ["toy-github-topkg-delegate"; "ipc"; "debug"; "publish";
                     "doc"; "https://mirage.github.io/mirage-http/";
                     "mirage-http"; "v2.5.1-18-ga311db0";
                     "### 2.5.3 (13-06-2016)\n\n* Switch to topkg (#25, @samoht)\n* Fix memory leak in the callback when an exception is raised (#24, @hannesm)";
                     "/Users/thomas/git/mirage-http/_build/mirage-http-2.5.1-18-ga311db0/_build/doc/api.docdir"]
toy-github-topkg-delegate: [ERROR] "": invalid path
topkg: [ERROR] Delegate toy-github-topkg-delegate errored with 2
dbuenzli commented 8 years ago

This is a quick patch to make tokpg publish doc work a bit better.

To be precise this is to make the toy-github-topkg-delegate work better, could you please make that precise in the commit msg.

dbuenzli commented 8 years ago

I pushed https://github.com/dbuenzli/topkg/commit/93de7229c089e46ecce68024699adee7211e5d41 to get better error messages.

samoht commented 8 years ago

I've added more commits. The logic for loading the branch for a remote repo changed a bit, not sure if that impact anything else but everything seems to still compile fine...

samoht commented 8 years ago

Hum, this is still broken.

dbuenzli commented 8 years ago

Thanks Thomas for improving these tedious tidbits. Your patches are in and massaged by 1b512f06fdcc902faa2e743facafe5