Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

go 1.15.6 #66331

Closed
wants to merge 35 commits into from
Closed

go 1.15.6 #66331

wants to merge 35 commits into from

Conversation

qinxin0720
Copy link
Contributor

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

qinxin0720 and others added 30 commits December 6, 2020 18:03
Closes #66300.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
Closes #66301.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
Closes #66291.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
Closes #66060.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
Closes #66264.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
Closes #66272.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
Closes #66307.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
Closes #66246.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
- bumped emscripten version
- changed llvm revision according to (see formula for details):
  - https://github.com/emscripten-core/emsdk/blob/master/emscripten-releases-tags.txt
  - https://chromium.googlesource.com/emscripten-releases/+/37fc7647c754ac9a28ad588c143b82286de0ef71/DEPS
- use recommended binaryen revision
    - The same has been done with llvm. See: #63183
- updated license to reflect bundled llvm and binaryen

Closes #66150.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
This was failing to build from source soley because the sha256 was
incorrect for the tarball.

I don't have any direct knowledge of thie history of this package,
but I suspect what happened is that the git history got rewritten
per the comment here: vapier/ncompress#26 (comment)
and that meant that github's generated tarball got changed slightly.

Closes #66305.

Signed-off-by: FX Coudert <[email protected]>
* gptfdisk: fix compile on Big Sur
  * Starting on 11.0, OS/X doesn't have physical files appearing
  as /usr/lib/*.dylib for the standard libraries, but the toolchain
  can still link against them (using the .tbd files inside the SDK
  to guide it)  Just use "-lncurses" which seems to work.

  * "brew test" wouldn't work for me.  The first problem was that
  hdiutil doesn't seem capable of creating a 128k disk image any more,
  but the command works when I run it manually with a 512k size.
  However, it still doesn't work due to a permission issue when
  running inside "brew test" -- I think something changed in how
  the test sandboxing works so that hdiutil can no longer run.
  (hdiutil can need expanded permissions since it temporarily
  mounts the new image I think)  I couldn't find a clean way
  to work around it so I just changed the test to use this package
  itself to make the test partition.  A better test in some ways,
  a worse test in others.

  * Tweak the license info to make "brew audit" happy
* gptfdisk: tweak test code style per review

Closes #66304.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
* The included configure script is too old to work now, so force
  an autoreconf

* There are still some issues with Xcode 12's treatment of
  -Wimplicit-function-declaration as an error, so tweak CFLAGS too

* It also was failing to download correctly because archive.org
  auto-gunzip'ed the response, breaking the sha256 check.  The
  other provided mirror URL was even more broken, failing to
  negotiate an https connection.  I found this voidlinux.org
  site that has the tarball with the expected sha though.

  None of this would be needed except that the original source of
  this package at cs.columbia.edu returns 404 if you try to
  access the URL over https instead of http.

Closes #66310.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
Closes #66318.

Signed-off-by: FX Coudert <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
@chenrui333
Copy link
Member

Thanks @qinxin0720. let's use the other clean PR, #66355

@chenrui333 chenrui333 closed this Dec 6, 2020
@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Jan 6, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Jan 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants