Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] ripgrep: new package
@ 2019-10-01  1:37 sam.voss at gmail.com
  2019-10-02 19:39 ` Thomas Petazzoni
  0 siblings, 1 reply; 2+ messages in thread
From: sam.voss at gmail.com @ 2019-10-01  1:37 UTC (permalink / raw)
  To: buildroot

From: Sam Voss <sam.voss@gmail.com>

Signed-off-by: Sam Voss <sam.voss@gmail.com>
---
 DEVELOPERS                   |  3 +++
 package/Config.in            |  1 +
 package/ripgrep/Config.in    | 14 ++++++++++++++
 package/ripgrep/ripgrep.hash |  2 ++
 package/ripgrep/ripgrep.mk   | 33 +++++++++++++++++++++++++++++++++
 5 files changed, 53 insertions(+)
 create mode 100644 package/ripgrep/Config.in
 create mode 100644 package/ripgrep/ripgrep.hash
 create mode 100644 package/ripgrep/ripgrep.mk

diff --git a/DEVELOPERS b/DEVELOPERS
index ac73c7daa5..8aac60f93b 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -2077,6 +2077,9 @@ F:	package/urg/
 F:	package/ussp-push/
 F:	support/misc/toolchainfile.cmake.in
 
+N:	Sam Voss <sam.voss@gmail.com>
+F:	package/ripgrep/
+
 N:	Santosh Multhalli <santosh.multhalli@rockwellcollins.com>
 F:	package/valijson/
 
diff --git a/package/Config.in b/package/Config.in
index 4d8fce21c0..b17e04e86e 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -172,6 +172,7 @@ menu "Development tools"
 	source "package/make/Config.in"
 	source "package/patch/Config.in"
 	source "package/pkgconf/Config.in"
+	source "package/ripgrep/Config.in"
 	source "package/sed/Config.in"
 	source "package/subversion/Config.in"
 	source "package/tree/Config.in"
diff --git a/package/ripgrep/Config.in b/package/ripgrep/Config.in
new file mode 100644
index 0000000000..b7eaf115ae
--- /dev/null
+++ b/package/ripgrep/Config.in
@@ -0,0 +1,14 @@
+config BR2_PACKAGE_RIPGREP
+	bool "ripgrep"
+	depends on BR2_PACKAGE_HOST_RUSTC_ARCH_SUPPORTS
+	select BR2_PACKAGE_HOST_CARGO
+	help
+	  ripgrep is a line-oriented search tool that
+	  recursively searches your current directory
+	  for a regex pattern while respecting your gitignore
+	  rules.
+
+	  ripgrep is similar to other popular search tools like
+	  The Silver Searcher, ack and grep.
+
+	  https://github.com/BurntSushi/ripgrep
diff --git a/package/ripgrep/ripgrep.hash b/package/ripgrep/ripgrep.hash
new file mode 100644
index 0000000000..d590bf66fa
--- /dev/null
+++ b/package/ripgrep/ripgrep.hash
@@ -0,0 +1,2 @@
+# Locally calculated
+sha256 7035379fce0c1e32552e8ee528b92c3d01b8d3935ea31d26c51a73287be74bb3 ripgrep-0.8.1.tar.gz
diff --git a/package/ripgrep/ripgrep.mk b/package/ripgrep/ripgrep.mk
new file mode 100644
index 0000000000..3bb4ca0604
--- /dev/null
+++ b/package/ripgrep/ripgrep.mk
@@ -0,0 +1,33 @@
+################################################################################
+#
+# ripgrep
+#
+################################################################################
+
+RIPGREP_VERSION = 0.8.1
+RIPGREP_SITE = $(call github,burntsushi,ripgrep,$(RIPGREP_VERSION))
+RIPGREP_LICENSE = MIT
+RIPGREP_LICENSE_FILES = LICENSE-MIT
+
+RIPGREP_DEPENDENCIES = host-cargo
+RIPGREP_CARGO_ENV = CARGO_HOME=$(HOST_DIR)/share/cargo
+RIPGREP_CARGO_MODE = $(if $(BR2_ENABLE_DEBUG),debug,release)
+
+RIPGREP_BIN_DIR = target/$(RUSTC_TARGET_NAME)/$(RIPGREP_CARGO_MODE)
+
+RIPGREP_CARGO_OPTS = \
+	--$(RIPGREP_CARGO_MODE) \
+		--target=$(RUSTC_TARGET_NAME) \
+		--manifest-path=$(@D)/Cargo.toml
+
+define RIPGREP_BUILD_CMDS
+	$(TARGET_MAKE_ENV) $(RIPGREP_CARGO_ENV) \
+		cargo build $(RIPGREP_CARGO_OPTS)
+endef
+
+define RIPGREP_INSTALL_TARGET_CMDS
+	$(INSTALL) -D -m 0755 $(@D)/$(RIPGREP_BIN_DIR)/rg \
+		$(TARGET_DIR)/usr/bin/rg
+endef
+
+$(eval $(generic-package))
-- 
2.23.0

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH] ripgrep: new package
  2019-10-01  1:37 [Buildroot] [PATCH] ripgrep: new package sam.voss at gmail.com
@ 2019-10-02 19:39 ` Thomas Petazzoni
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2019-10-02 19:39 UTC (permalink / raw)
  To: buildroot

Hello Sam,

+Eric Le Bihan on Cc to get some review on this cargo/rust related
package.

On Mon, 30 Sep 2019 20:37:03 -0500
sam.voss at gmail.com wrote:

> From: Sam Voss <sam.voss@gmail.com>
> 
> Signed-off-by: Sam Voss <sam.voss@gmail.com>

The commit title should be "package/ripgrep: new package".


> diff --git a/package/ripgrep/Config.in b/package/ripgrep/Config.in
> new file mode 100644
> index 0000000000..b7eaf115ae
> --- /dev/null
> +++ b/package/ripgrep/Config.in
> @@ -0,0 +1,14 @@
> +config BR2_PACKAGE_RIPGREP
> +	bool "ripgrep"
> +	depends on BR2_PACKAGE_HOST_RUSTC_ARCH_SUPPORTS

I believe you need:

	depends on BR2_PACKAGE_HOST_RUSTC_TARGET_ARCH_SUPPORTS

Indeed, I built on ARMv5, and it failed with:

PATH="/home/thomas/projets/buildroot/output/host/bin:/home/thomas/projets/buildroot/output/host/sbin:/home/thomas/sys/bin/:/usr/libexec/python3-sphinx:/usr/share/Modules/bin:/usr/lib64/ccache:/usr/local/bin:/usr/local/sbin:/usr/bin:/usr/sbin:/home/thomas/.rvm/bin:/home/thomas/sys/bin:/home/thomas/.gem/ruby/2.1.0/bin:/home/thomas/.rvm/bin:/home/thomas/.local/bin" CARGO_HOME=/home/thomas/projets/buildroot/output/host/share/cargo cargo build --release --target= --manifest-path=/home/thomas/projets/buildroot/output/build/ripgrep-0.8.1/Cargo.toml
error: Couldn't load Cargo configuration

Caused by:
  could not parse TOML configuration in `/home/thomas/projets/buildroot/output/host/share/cargo/config`

Caused by:
  could not parse input as TOML

Caused by:
  expected a table key, found a right bracket at line 1
make[1]: *** [package/pkg-generic.mk:241: /home/thomas/projets/buildroot/output/build/ripgrep-0.8.1/.stamp_built] Error 101
make: *** [Makefile:84: _all] Error 2

This is due to RUSTC_TARGET_NAME being empty when
BR2_PACKAGE_HOST_RUSTC_TARGET_ARCH_SUPPORTS is false.

Then you don't need the dependency on
BR2_PACKAGE_HOST_RUSTC_ARCH_SUPPORTS, because
BR2_PACKAGE_HOST_RUSTC_TARGET_ARCH_SUPPORTS depends on it.

> +	select BR2_PACKAGE_HOST_CARGO
> +	help
> +	  ripgrep is a line-oriented search tool that
> +	  recursively searches your current directory
> +	  for a regex pattern while respecting your gitignore
> +	  rules.
> +
> +	  ripgrep is similar to other popular search tools like
> +	  The Silver Searcher, ack and grep.
> +
> +	  https://github.com/BurntSushi/ripgrep
> diff --git a/package/ripgrep/ripgrep.hash b/package/ripgrep/ripgrep.hash
> new file mode 100644
> index 0000000000..d590bf66fa
> --- /dev/null
> +++ b/package/ripgrep/ripgrep.hash
> @@ -0,0 +1,2 @@
> +# Locally calculated
> +sha256 7035379fce0c1e32552e8ee528b92c3d01b8d3935ea31d26c51a73287be74bb3 ripgrep-0.8.1.tar.gz

Please include a hash for the license file.

> diff --git a/package/ripgrep/ripgrep.mk b/package/ripgrep/ripgrep.mk
> new file mode 100644
> index 0000000000..3bb4ca0604
> --- /dev/null
> +++ b/package/ripgrep/ripgrep.mk
> @@ -0,0 +1,33 @@
> +################################################################################
> +#
> +# ripgrep
> +#
> +################################################################################
> +
> +RIPGREP_VERSION = 0.8.1
> +RIPGREP_SITE = $(call github,burntsushi,ripgrep,$(RIPGREP_VERSION))
> +RIPGREP_LICENSE = MIT
> +RIPGREP_LICENSE_FILES = LICENSE-MIT
> +
> +RIPGREP_DEPENDENCIES = host-cargo
> +RIPGREP_CARGO_ENV = CARGO_HOME=$(HOST_DIR)/share/cargo
> +RIPGREP_CARGO_MODE = $(if $(BR2_ENABLE_DEBUG),debug,release)
> +
> +RIPGREP_BIN_DIR = target/$(RUSTC_TARGET_NAME)/$(RIPGREP_CARGO_MODE)
> +
> +RIPGREP_CARGO_OPTS = \
> +	--$(RIPGREP_CARGO_MODE) \
> +		--target=$(RUSTC_TARGET_NAME) \
> +		--manifest-path=$(@D)/Cargo.toml

Keep all lines aligned:

FOO = \
	baz \
	bar \
	foobaz

> +
> +define RIPGREP_BUILD_CMDS
> +	$(TARGET_MAKE_ENV) $(RIPGREP_CARGO_ENV) \
> +		cargo build $(RIPGREP_CARGO_OPTS)
> +endef
> +
> +define RIPGREP_INSTALL_TARGET_CMDS
> +	$(INSTALL) -D -m 0755 $(@D)/$(RIPGREP_BIN_DIR)/rg \
> +		$(TARGET_DIR)/usr/bin/rg
> +endef
> +
> +$(eval $(generic-package))

Could you fix those issues, and send an updated version ?

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-10-02 19:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-10-01  1:37 [Buildroot] [PATCH] ripgrep: new package sam.voss at gmail.com
2019-10-02 19:39 ` Thomas Petazzoni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox