-
-
Notifications
You must be signed in to change notification settings - Fork 69
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
cleanup_before: clean /Library/Frameworks
directory
#916
Conversation
These directories contain headers and libraries that we don't want to use, so let's get rid of them when building on a GitHub macOS runner. The Mono framework in particular contains Gettext headers that I've seen mess up CMake builds in the past.
lib/tests/cleanup_before.rb
Outdated
delete_or_move Pathname.glob(HOMEBREW_CELLAR/"*") | ||
|
||
frameworks_dir = Pathname("/Library/Frameworks") | ||
%w[ | ||
Mono.framework | ||
PluginManager.framework | ||
Python.framework | ||
R.framework | ||
Xamarin.Android.framework | ||
Xamarin.Mac.framework | ||
Xamarin.iOS.framework | ||
].each do |framework| | ||
path = frameworks_dir/framework | ||
test "sudo", "mv", path, HOMEBREW_TEMP if path.exist? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this use delete_or_move
instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs sudo
, unfortunately. Would you rather we refactor delete_or_move
to use sudo
when requested?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@carlocab Might make sense but: up to you, no strong feelings other way!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
cfe64a6
to
3570759
Compare
5410898
to
3bb4434
Compare
Thanks @carlocab! |
These directories contain headers and libraries that we don't want to
use, so let's get rid of them when building on a GitHub macOS runner.
The Mono framework in particular contains Gettext headers that I've seen
mess up CMake builds in the past.