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.

Sign up for GitHub

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

Jump to bottom

[R] autobrew formulae missing branch: "main" preventing local autobrew testing #36687

Closed
paleolimbot opened this issue Jul 14, 2023 · 0 comments · Fixed by #36689
Closed

[R] autobrew formulae missing branch: "main" preventing local autobrew testing #36687

paleolimbot opened this issue Jul 14, 2023 · 0 comments · Fixed by #36689
Assignees
@paleolimbot
Labels
Component: R Type: bug
Milestone
13.0.0

Comments

@paleolimbot
Copy link
Member

Describe the bug, including details regarding any error messages, version, and platform.

Discovered in #36456 (comment) . This doesn't show up in CI because we modify the formula there to point to a specific commit.

==> Fetching apache-arrow-static
==> Cloning https://github.com/apache/arrow.git
Cloning into '/var/folders/4v/hvphchw50373drst9g5smz1w0000gn/T/apache-arrow-static--git'...
fatal: Remote branch master not found in upstream origin
Error: apache-arrow-static: Failed to download resource "apache-arrow-static"
Failure while executing; `/usr/bin/env git clone --branch master --config advice.detachedHead=false --config core.fsmonitor=false https://github.com/apache/arrow.git /var/folders/4v/hvphchw50373drst9g5smz1w0000gn/T/apache-arrow-static--git` exited with 128. Here's the output:
Cloning into '/var/folders/4v/hvphchw50373drst9g5smz1w0000gn/T/apache-arrow-static--git'...
fatal: Remote branch master not found in upstream origin

Component(s)

R
@github-actions github-actions bot mentioned this issue Jul 14, 2023
Merged
kou pushed a commit that referenced this issue Jul 14, 2023
@paleolimbot
GH-36687: [R] Add correct branch name to autobrew formulae to facilit…
1bee643 
…ate local testing (#36689)

### Rationale for this change

It is currently not possible to recreate an autobrew build locally by following the instructions in the comments. This fixes the local copies of the upstream formulas and the instructions so that future debuggers can recreate an autobrew build.

### What changes are included in this PR?

The branch `master` no longer exists and is the default value. This PR adds the revised default branch name ("main").

### Are these changes tested?

No nightly test covers this because this value would be overwritten to test specific commits anyway.

### Are there any user-facing changes?

No.
* Closes: #36687

Authored-by: Dewey Dunnington <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
@kou kou added this to the 14.0.0 milestone Jul 14, 2023
@raulcd raulcd modified the milestones: 14.0.0, 13.0.0 Jul 18, 2023
raulcd pushed a commit that referenced this issue Jul 18, 2023
@paleolimbot @raulcd
GH-36687: [R] Add correct branch name to autobrew formulae to facilit…
3cded6e 
…ate local testing (#36689)

### Rationale for this change

It is currently not possible to recreate an autobrew build locally by following the instructions in the comments. This fixes the local copies of the upstream formulas and the instructions so that future debuggers can recreate an autobrew build.

### What changes are included in this PR?

The branch `master` no longer exists and is the default value. This PR adds the revised default branch name ("main").

### Are these changes tested?

No nightly test covers this because this value would be overwritten to test specific commits anyway.

### Are there any user-facing changes?

No.
* Closes: #36687

Authored-by: Dewey Dunnington <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
chelseajonesr pushed a commit to chelseajonesr/arrow that referenced this issue Jul 20, 2023
@paleolimbot @chelseajonesr
apacheGH-36687: [R] Add correct branch name to autobrew formulae to f…
08b4736 
…acilitate local testing (apache#36689)

### Rationale for this change

It is currently not possible to recreate an autobrew build locally by following the instructions in the comments. This fixes the local copies of the upstream formulas and the instructions so that future debuggers can recreate an autobrew build.

### What changes are included in this PR?

The branch `master` no longer exists and is the default value. This PR adds the revised default branch name ("main").

### Are these changes tested?

No nightly test covers this because this value would be overwritten to test specific commits anyway.

### Are there any user-facing changes?

No.
* Closes: apache#36687

Authored-by: Dewey Dunnington <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
R-JunmingChen pushed a commit to R-JunmingChen/arrow that referenced this issue Aug 20, 2023
@paleolimbot @R-JunmingChen
apacheGH-36687: [R] Add correct branch name to autobrew formulae to f…
ecc92a6 
…acilitate local testing (apache#36689)

### Rationale for this change

It is currently not possible to recreate an autobrew build locally by following the instructions in the comments. This fixes the local copies of the upstream formulas and the instructions so that future debuggers can recreate an autobrew build.

### What changes are included in this PR?

The branch `master` no longer exists and is the default value. This PR adds the revised default branch name ("main").

### Are these changes tested?

No nightly test covers this because this value would be overwritten to test specific commits anyway.

### Are there any user-facing changes?

No.
* Closes: apache#36687

Authored-by: Dewey Dunnington <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@paleolimbot paleolimbot

Labels
Component: R Type: bug
Projects
None yet
Milestone
13.0.0
3 participants
@kou @raulcd @paleolimbot