Skip to content

Commit

Permalink
Tweak language again
Browse files Browse the repository at this point in the history
  • Loading branch information
nealrichardson committed Aug 7, 2019
1 parent 36cfe28 commit 44c9659
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 5 deletions.
5 changes: 3 additions & 2 deletions r/R/install-arrow.R
Original file line number Diff line number Diff line change
Expand Up @@ -99,8 +99,9 @@ SEE_ARROW_INSTALL <- paste(
"See the Apache Arrow project installation page",
"<https://arrow.apache.org/install/>",
"to find pre-compiled binary packages for some common Linux distributions,",
"including Debian, Ubuntu, and CentOS. You'll need to install 'libarrow-dev'",
"and 'libparquet-dev'."
"including Debian, Ubuntu, and CentOS. You'll need to install",
"'libparquet-dev' on Debian and Ubuntu, or 'parquet-devel' on CentOS. This",
"will also automatically install the arrow C++ library as a dependency."
)

THEN_REINSTALL <- paste(
Expand Down
2 changes: 1 addition & 1 deletion r/README.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ Install the latest release of `arrow` from CRAN with
install.packages("arrow")
```

On macOS and Windows, installing a binary package from CRAN will handle Arrow's C++ dependencies for you. On Linux, you'll need to first install the C++ library. See the [Arrow project installation page](https://arrow.apache.org/install/) to find pre-compiled binary packages for some common Linux distributions, including Debian, Ubuntu, and CentOS. You'll need to install `libarrow-dev` and `libparquet-dev`.
On macOS and Windows, installing a binary package from CRAN will handle Arrow's C++ dependencies for you. On Linux, you'll need to first install the C++ library. See the [Arrow project installation page](https://arrow.apache.org/install/) to find pre-compiled binary packages for some common Linux distributions, including Debian, Ubuntu, and CentOS. You'll need to install `libparquet-dev` on Debian and Ubuntu, or `parquet-devel` on CentOS. This will also automatically install the arrow C++ library as a dependency.

If you install the `arrow` package from source and the C++ library is not found, the R package functions will notify you that Arrow is not available. Call

Expand Down
4 changes: 3 additions & 1 deletion r/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,9 @@ Arrow’s C++ dependencies for you. On Linux, you’ll need to first install
the C++ library. See the [Arrow project installation
page](https://arrow.apache.org/install/) to find pre-compiled binary
packages for some common Linux distributions, including Debian, Ubuntu,
and CentOS. You’ll need to install `libarrow-dev` and `libparquet-dev`.
and CentOS. You’ll need to install `libparquet-dev` on Debian and
Ubuntu, or `parquet-devel` on CentOS. This will also automatically
install the arrow C++ library as a dependency.

If you install the `arrow` package from source and the C++ library is
not found, the R package functions will notify you that Arrow is not
Expand Down
2 changes: 1 addition & 1 deletion r/tests/testthat/test-install-arrow.R
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ r_only({
test_that("Linux on release version gets pointed to PPA first, then C++", {
expect_match(
install_arrow_msg(FALSE, "0.13.0", os="linux"),
"'libparquet-dev'. Or, see the Arrow C++ developer guide",
"dependency. Or, see the Arrow C++ developer guide",
fixed = TRUE
)
})
Expand Down

0 comments on commit 44c9659

Please sign in to comment.