Skip to content

Conversation

pthariensflame
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>)?

See also #66778, which this obviates.

@BrewTestBot BrewTestBot added lua Lua use is a significant feature of the PR or issue python Python use is a significant feature of the PR or issue ruby Ruby use is a significant feature of the PR or issue labels Dec 12, 2020
@chenrui333 chenrui333 changed the title Update macvim from 8.2-166 to 8.2-168 macvim 8.2-168 Dec 13, 2020
chenrui333
chenrui333 previously approved these changes Dec 13, 2020
@pthariensflame
Copy link
Contributor Author

I'm not sure what the failure on Catalina is about.

@gromgit
Copy link
Contributor

gromgit commented Dec 13, 2020

https://github.com/Homebrew/homebrew-core/pull/66828/checks?check_run_id=1543711707#step:6:998

    clang -x objective-c-header -target arm64-apple-macos10.15 -fmessage-length\=0 -fdiagnostics-show-note-include-stack -fmacro-backtrace-limit\=0 -Wno-trigraphs -fpascal-strings -Os -Wno-missing-field-initializers -Wno-missing-prototypes -Wno-implicit-atomic-properties -Wno-objc-interface-ivars -Wno-arc-repeated-use-of-weak -Wno-missing-braces -Wparentheses -Wswitch -Wno-unused-function -Wno-unused-label -Wno-unused-parameter -Wunused-variable -Wunused-value -Wno-empty-body -Wno-uninitialized -Wno-unknown-pragmas -Wno-shadow -Wno-four-char-constants -Wno-conversion -Wno-constant-conversion -Wno-int-conversion -Wno-bool-conversion -Wno-enum-conversion -Wno-float-conversion -Wno-non-literal-null-conversion -Wno-objc-literal-conversion -Wno-shorten-64-to-32 -Wpointer-sign -Wno-newline-eof -Wno-selector -Wno-strict-selector-match -Wno-undeclared-selector -Wno-deprecated-implementations -DDISABLE_SPARKLE\=1 -DOBJC_OLD_DISPATCH_PROTOTYPES\=1 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk -fstrict-aliasing -Wprotocol -Wdeprecated-declarations -Wno-sign-conversion -Wno-infinite-recursion -Wno-comma -Wno-block-capture-autoreleasing -Wno-strict-prototypes -Wno-semicolon-before-method-body -iquote /tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-generated-files.hmap -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-own-target-headers.hmap -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-all-target-headers.hmap -iquote /tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-project-headers.hmap -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/Release/include -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources-normal/arm64 -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources/arm64 -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources -F/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/Release -c /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk/System/Library/Frameworks/AppKit.framework/Headers/AppKit.h -MD -MT dependencies -MF /tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/SharedPrecompiledHeaders/SharedPrecompiledHeaders/9532461868649930696/AppKit.h.d -iquote /tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-generated-files.hmap -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-own-target-headers.hmap -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-all-target-headers.hmap -iquote /tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-project-headers.hmap -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/Release/include -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources-normal/arm64 -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources/arm64 -I/tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources -o /tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/SharedPrecompiledHeaders/SharedPrecompiledHeaders/9532461868649930696/AppKit.h.gch --serialize-diagnostics /tmp/macvim-20201213-38936-13mrnl1/macvim-snapshot-168/src/MacVim/build/SharedPrecompiledHeaders/SharedPrecompiledHeaders/9532461868649930696/AppKit.h.dia

CI building arm64 targets on Catalina?

@fxcoudert
Copy link
Member

Macvim's Xcode project sometimes wants to build universal binaries, and we should avoid it. That's what the removed patch was going, and this should be filed as an issue with upstream

@ychin
Copy link
Contributor

ychin commented Dec 20, 2020

Hi, I just made a new release (r169) for MacVim so we should make a new one for that instead.

The way MacVim currently prevents building universal binaries is via the --with-macarchs=x86_64 flag (link). You could add that to the configure flags to make it build a thin binary. See macvim-dev/macvim#1137 where I added that.

This won't allow for non-Rosetta builds on Apple Silicon though. But maybe that's a different fix later.

Edit: To be clear, that means the patch isn't necessary with this flag. You don't need to add it back.

@chenrui333 chenrui333 added the build failure CI fails while building the software label Dec 20, 2020
@pthariensflame
Copy link
Contributor Author

@ychin Yay! I'll handle that tonight then.

@pthariensflame pthariensflame changed the title macvim 8.2-168 macvim 8.2-169 Dec 21, 2020
@pthariensflame
Copy link
Contributor Author

@chenrui333 How does this look now?

@ychin
Copy link
Contributor

ychin commented Dec 21, 2020

I'm pretty sure you want arm64 for arch, not aarch64.

@pthariensflame
Copy link
Contributor Author

@ychin All should be resolved now.

@BrewTestBot
Copy link
Member

🤖 A scheduled task has triggered a merge.

@pthariensflame pthariensflame deleted the patch-1 branch December 25, 2020 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build failure CI fails while building the software lua Lua use is a significant feature of the PR or issue python Python use is a significant feature of the PR or issue ruby Ruby use is a significant feature of the PR or issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants