OpenGL 4.6 support. about glow HOT 11 CLOSED

I believe the right answer is to change the default source where we fetch the XML files. It appears that Khronos maintains its GitHub repositories but not its SVN ones. Unfortunately it means pulling the files from more locations. The docs appear to be located in https://github.com/KhronosGroup/OpenGL-Refpages, and egl.xml appears to be located in https://github.com/KhronosGroup/EGL-Registry. Perhaps the easiest answer is to remove the download command and instead add documentation from where to fetch the files?

from glow.

I made support for GitHub basic auth.

from glow.

I rewrote main.go and download.go (link, branch OpenGL4.6) to fetch the XML files from GitHub but there is a problem with hourly rate limit. For unauthenticated requests, the rate limit is 60 requests per hour which is not enough to download all the files.

from glow.

We can make it a requirement to supply a github token to get the authenticated rate limit of 5000 per hour.

from glow.

I'm attempting to resolve this in #92.

from glow.

@errcw Did you see #91 before making #92? If so, can you explain in what ways do they differ?

from glow.

Alas, no. I commented on that review. I'll drop #92 once #91 goes in.

from glow.

When I try to download gl.xml, the empty file is returned. The cause of the error is the file size:

{
  "message": "This API returns blobs up to 1 MB in size. The requested blob is too large to fetch via the API, but you can use the Git Data API to request blobs up to 100 MB in size.",
  "errors": [
    {
      "resource": "Blob",
      "field": "data",
      "code": "too_large"
    }
  ],
  "documentation_url": "https://developer.github.com/v3/repos/contents/#get-contents"
}

Please check the pull request "Add support for download of files larger than 1 MB.".

from glow.

Is this issue resolved by #91 and #94 (which have been merged by now)?

from glow.

Yes. You can close the issue.

from glow.

We should download and commit the latest XML from the new source to fully resolve this issue. Reopening for that.

from glow.