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] Rosetta detector has output when running on x86 #38049

Closed
jonkeane opened this issue Oct 5, 2023 · 0 comments · Fixed by #38052
Closed

[R] Rosetta detector has output when running on x86 #38049

jonkeane opened this issue Oct 5, 2023 · 0 comments · Fixed by #38052
Assignees
@jonkeane
Labels
Component: R Type: bug
Milestone
14.0.0

Comments

@jonkeane
Copy link
Member

jonkeane commented Oct 5, 2023 •
edited

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

In #37777 we added a detector that checks if Rosetta translation is being used. It turns out this will produce warnings on x86 systems. We should fix that.

sysctl: unknown oid 'sysctl.proc_translated'
...
Warning message:
In system("sysctl -n sysctl.proc_translated", intern = TRUE) :
  running command 'sysctl -n sysctl.proc_translated' had status 1

Component(s)

R
@jonkeane jonkeane self-assigned this Oct 5, 2023
@github-actions github-actions bot mentioned this issue Oct 5, 2023
Merged
paleolimbot pushed a commit that referenced this issue Oct 6, 2023
@jonkeane
GH-38049: [R] Prevent on_rosetta() from warning (#38052)
fdecb6a 
### Rationale for this change

Stop extraneous warnings.

### What changes are included in this PR?

Prevent warning when detecting translation, but on x86

### Are these changes tested?

Yes

### Are there any user-facing changes?

Yes, there will be fewer extraneous warnings.
* Closes: #38049

Authored-by: Jonathan Keane <jkeane@gmail.com>
Signed-off-by: Dewey Dunnington <dewey@voltrondata.com>
@paleolimbot paleolimbot added this to the 14.0.0 milestone Oct 6, 2023
JerAguilon pushed a commit to JerAguilon/arrow that referenced this issue Oct 23, 2023
@jonkeane
apacheGH-38049: [R] Prevent on_rosetta() from warning (apache#38052)
5786e9b 
### Rationale for this change

Stop extraneous warnings.

### What changes are included in this PR?

Prevent warning when detecting translation, but on x86

### Are these changes tested?

Yes

### Are there any user-facing changes?

Yes, there will be fewer extraneous warnings.
* Closes: apache#38049

Authored-by: Jonathan Keane <jkeane@gmail.com>
Signed-off-by: Dewey Dunnington <dewey@voltrondata.com>
loicalleyne pushed a commit to loicalleyne/arrow that referenced this issue Nov 13, 2023
@jonkeane @loicalleyne
apacheGH-38049: [R] Prevent on_rosetta() from warning (apache#38052)
c561163 
### Rationale for this change

Stop extraneous warnings.

### What changes are included in this PR?

Prevent warning when detecting translation, but on x86

### Are these changes tested?

Yes

### Are there any user-facing changes?

Yes, there will be fewer extraneous warnings.
* Closes: apache#38049

Authored-by: Jonathan Keane <jkeane@gmail.com>
Signed-off-by: Dewey Dunnington <dewey@voltrondata.com>
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
@jonkeane @dgreiss
apacheGH-38049: [R] Prevent on_rosetta() from warning (apache#38052)
afe7377 
### Rationale for this change

Stop extraneous warnings.

### What changes are included in this PR?

Prevent warning when detecting translation, but on x86

### Are these changes tested?

Yes

### Are there any user-facing changes?

Yes, there will be fewer extraneous warnings.
* Closes: apache#38049

Authored-by: Jonathan Keane <jkeane@gmail.com>
Signed-off-by: Dewey Dunnington <dewey@voltrondata.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jonkeane jonkeane

Labels
Component: R Type: bug
Projects
None yet
Milestone
14.0.0
Development

Successfully merging a pull request may close this issue.

GH-38049: [R] Prevent on_rosetta() from warning jonkeane/arrow
2 participants
@jonkeane @paleolimbot