kou commented on code in PR #45854:
URL: https://github.com/apache/arrow/pull/45854#discussion_r2015255789


##########
dev/release/02-source-test.rb:
##########
@@ -84,7 +84,12 @@ def test_csharp_git_commit_information
   def test_python_version
     source
     Dir.chdir("#{@tag_name_no_rc}/python") do
-      sh("python3", "setup.py", "sdist")
+      FileUtils.mv("dummy.git", "../.git")
+      sh("python3", "-m", "pip", "install", "build")

Review Comment:
   `dummy.git` is for C#: https://github.com/apache/arrow/issues/21580 / 
https://github.com/apache/arrow/pull/4488
   
   It exists in our source archive for providing repository information to C# 
packages.
   
   This test checks whether we can create an sdist from the source archive not 
Git (cloned) repository. Does Meson require Git repository to create an sdist? 
We want to create release artifacts including sdist from a source archive that 
is signed and voted.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to