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

Homebrew on Apple Silicon / XCode 13.2 support #1457

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jgavris
Copy link
Contributor

@jgavris jgavris commented Dec 26, 2021

  • Supports detecting brew --prefix for the new /opt/homebrew home on Apple Silicon.
  • Fixes shared_ptr cast warning in RMateServer.
  • vfork is now deprecated (has same behavior as fork on macOS 12 and above, and will be removed eventually).

@@ -334,16 +334,17 @@ bool write_data () const
{
reactivate_callback_t () : _shared_count(std::make_shared<size_t>(0))
{
_terminal = std::make_shared<NSRunningApplication*>([NSWorkspace.sharedWorkspace frontmostApplication]);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In file included from Applications/TextMate/src/RMateServer.mm:1:
In file included from /Users/jgavris/code/textmate/Shared/PCH/prelude.mm:6:
In file included from /Users/jgavris/code/textmate/Shared/PCH/prelude.cc:9:
In file included from /Applications/Xcode-v13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.1.sdk/usr/include/c++/v1/algorithm:653:
/Applications/Xcode-v13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.1.sdk/usr/include/c++/v1/memory:2684:27: error: cast of a non-Objective-C pointer type 'typename _CompressedPair::_Base2 *' (aka '__compressed_pair_elem<NSRunningApplication *__strong, 1> *') to 'NSRunningApplication *__strong *' is disallowed with ARC
            _Tp *__elem = reinterpret_cast<_Tp*>(__second);
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Applications/Xcode-v13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.1.sdk/usr/include/c++/v1/memory:2635:53: note: in instantiation of member function 'std::__shared_ptr_emplace<NSRunningApplication *, std::allocator<NSRunningApplication *>>::_Storage::__get_elem' requested here
    _Tp* __get_elem() _NOEXCEPT { return __storage_.__get_elem(); }
                                                    ^
/Applications/Xcode-v13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.1.sdk/usr/include/c++/v1/memory:3387:76: note: in instantiation of member function 'std::__shared_ptr_emplace<NSRunningApplication *, std::allocator<NSRunningApplication *>>::__get_elem' requested here
    return shared_ptr<_Tp>::__create_with_control_block((*__control_block).__get_elem(), _VSTD::addressof(*__control_block));
                                                                           ^
/Applications/Xcode-v13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.1.sdk/usr/include/c++/v1/memory:3394:19: note: in instantiation of function template specialization 'std::allocate_shared<NSRunningApplication *, std::allocator<NSRunningApplication *>, NSRunningApplication *, void>' requested here
    return _VSTD::allocate_shared<_Tp>(allocator<_Tp>(), _VSTD::forward<_Args>(__args)...);
                  ^
Applications/TextMate/src/RMateServer.mm:337:21: note: in instantiation of function template specialization 'std::make_shared<NSRunningApplication *, NSRunningApplication *, void>' requested here
                        _terminal = std::make_shared<NSRunningApplication*>([NSWorkspace.sharedWorkspace frontmostApplication]);
                                         ^
1 error generated.

KittyMac added a commit to KittyMac/textmate that referenced this pull request Jul 7, 2022
on M1 and XCode 13.2
@hovsater
Copy link

@sorbits would it be possible to give this a look? Would really appreciate being able to build TextMate on my Mac again. 🙂

@faisal
Copy link

faisal commented Jul 14, 2023

@sorbits do you expect to do something with this?

@garsaud
Copy link

garsaud commented Oct 2, 2023

It seems that this repo is dead. Last commit is from 2021-10-12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants