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

git-lfs 2.9.2 #45467

Closed
wants to merge 2 commits into from
Closed

git-lfs 2.9.2 #45467

wants to merge 2 commits into from

Conversation

chenrui333
Copy link
Member

Created with brew bump-formula-pr.

@alebcay
Copy link
Member

alebcay commented Oct 18, 2019

On Catalina (test failure):

==> git init
Initialized empty Git repository in /private/tmp/git-lfs-test-20191018-42001-13u090c/.git/
==> git lfs track test
/usr/local/bin/git-lfs: /usr/local/bin/git-lfs: cannot execute binary file

On Mojave (build failure):

==> gem install ronn
Successfully installed mustache-1.1.0
Building native extensions.  This could take a while...
ERROR:  Error installing ronn:
	ERROR: Failed to build gem native extension.

    current directory: /private/tmp/git-lfs-20191018-68359-kiz07/git-lfs-2.9.0/src/github.com/git-lfs/git-lfs/.gem_home/gems/rdiscount-2.2.0.1/ext
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/bin/ruby -r ./siteconf20191018-68371-6y65rn.rb extconf.rb
mkmf.rb can't find header files for ruby at /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/include/ruby.h

extconf failed, exit code 1

Gem files will remain installed in /private/tmp/git-lfs-20191018-68359-kiz07/git-lfs-2.9.0/src/github.com/git-lfs/git-lfs/.gem_home/gems/rdiscount-2.2.0.1 for inspection.
Results logged to /private/tmp/git-lfs-20191018-68359-kiz07/git-lfs-2.9.0/src/github.com/git-lfs/git-lfs/.gem_home/extensions/universal-darwin-18/2.3.0/rdiscount-2.2.0.1/gem_make.out

@alebcay alebcay added build failure CI fails while building the software test failure CI fails while running the test-do block labels Oct 18, 2019
@chenrui333 chenrui333 changed the title git-lfs 2.9.0 git-lfs 2.9.1 Nov 26, 2019
@chenrui333
Copy link
Member Author

bump the version to v2.9.1

@Bo98
Copy link
Member

Bo98 commented Nov 26, 2019

==> git lfs track test
/usr/local/bin/git-lfs: /usr/local/bin/git-lfs: cannot execute binary file
Error: git-lfs: failed

This might be Homebrew/brew#6539

@chenrui333
Copy link
Member Author

Circle back on this.

@chenrui333
Copy link
Member Author

Tested on my catalina machine, seems working fine

$ brew test --verbose git-lfs
Testing git-lfs
/usr/bin/sandbox-exec -f /private/tmp/homebrew20191128-30869-1b6u9gm.sb ruby -W0 -I $LOAD_PATH -- /usr/local/Homebrew/Library/Homebrew/test.rb /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/git-lfs.rb --verbose
==> git init
Initialized empty Git repository in /private/tmp/git-lfs-test-20191128-30870-e6ocdz/.git/
==> git lfs track test
Tracking "test"

@Bo98
Copy link
Member

Bo98 commented Nov 29, 2019

The Catalina failure is the tar bug. Mojave failure I think can be fixed by adjusting the SDK path (I'll check later).

@chenrui333 chenrui333 changed the title git-lfs 2.9.1 git-lfs 2.9.2 Dec 28, 2019
@chenrui333
Copy link
Member Author

Pushed a new release and rebased to master (give another try on this PR)

@fxcoudert
Copy link
Member

On Mojave:

==> gem install ronn
Successfully installed mustache-1.1.1
Building native extensions.  This could take a while...
ERROR:  Error installing ronn:
	ERROR: Failed to build gem native extension.

    current directory: /private/tmp/git-lfs-20191228-50033-1a6g4ls/git-lfs-2.9.2/src/github.com/git-lfs/git-lfs/.gem_home/gems/rdiscount-2.2.0.1/ext
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/bin/ruby -r ./siteconf20191228-50047-x6c9kn.rb extconf.rb
mkmf.rb can't find header files for ruby at /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/include/ruby.h

extconf failed, exit code 1

Gem files will remain installed in /private/tmp/git-lfs-20191228-50033-1a6g4ls/git-lfs-2.9.2/src/github.com/git-lfs/git-lfs/.gem_home/gems/rdiscount-2.2.0.1 for inspection.
Results logged to /private/tmp/git-lfs-20191228-50033-1a6g4ls/git-lfs-2.9.2/src/github.com/git-lfs/git-lfs/.gem_home/extensions/universal-darwin-18/2.3.0/rdiscount-2.2.0.1/gem_make.out

@bk2204
Copy link
Contributor

bk2204 commented Jan 3, 2020

From Googling, it looks like the problem is that we're using XCode 11 on Mojave, which installs the 10.15 SDK with Ruby 2.6 (Catalina's) development headers, not Ruby 2.3 (Mojave's). I think the easiest way to solve this would be something like the following:

diff --git a/Formula/git-lfs.rb b/Formula/git-lfs.rb
index b88cc89fa7..c2d9bd5979 100644
--- a/Formula/git-lfs.rb
+++ b/Formula/git-lfs.rb
@@ -12,10 +12,7 @@ class GitLfs < Formula
   end
 
   depends_on "go" => :build
-  uses_from_macos "ruby"
-
-  # System Ruby uses old TLS versions no longer supported by RubyGems.
-  depends_on "ruby" => :build if MacOS.version <= :sierra
+  depends_on "ruby" => :build
 
   def install
     ENV["GIT_LFS_SHA"] = ""

This is completely untested, since I'm running Catalina, not Mojave, but presumably it should work.

@chenrui333
Copy link
Member Author

Let me give a try.

@chenrui333
Copy link
Member Author

Now it is the tar issue with the Catalina:

14:17:06 ==> git lfs track test
14:17:06 /usr/local/bin/git-lfs: /usr/local/bin/git-lfs: cannot execute binary file
14:17:06 Error: git-lfs: failed
14:17:06 Failed executing:

@chenrui333 chenrui333 closed this in bb3aeec Jan 3, 2020
@chenrui333
Copy link
Member Author

chenrui333 commented Jan 4, 2020

Relates to Homebrew/brew#6539

@chenrui333 chenrui333 mentioned this pull request Jan 5, 2020
5 tasks
@lock lock bot added the outdated PR was locked due to age label Feb 3, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Feb 3, 2020
@chenrui333 chenrui333 deleted the git-lfs-2.9.0 branch December 18, 2022 04:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
build failure CI fails while building the software outdated PR was locked due to age test failure CI fails while running the test-do block
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants