Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] proxychains-ng: copy proxychains.conf to target dir
@ 2013-01-16  4:10 Stefan Fröberg
  2013-01-16  6:29 ` Peter Korsgaard
  0 siblings, 1 reply; 3+ messages in thread
From: Stefan Fröberg @ 2013-01-16  4:10 UTC (permalink / raw)
  To: buildroot

proxychains installation does not copy proxychains.conf to 
$(TARGET_DIR)/etc anymore so we have to do it manually now.

Signed-off-by: Stefan Fr?berg <stefan.froberg@petroprogram.com>
---
 package/proxychains-ng/proxychains-ng.mk |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/package/proxychains-ng/proxychains-ng.mk b/package/proxychains-ng/proxychains-ng.mk
index 9cf24bf..70d35b6 100644
--- a/package/proxychains-ng/proxychains-ng.mk
+++ b/package/proxychains-ng/proxychains-ng.mk
@@ -7,4 +7,10 @@ PROXYCHAINS_NG_VERSION = 4.4
 PROXYCHAINS_NG_SOURCE = proxychains-$(PROXYCHAINS_NG_VERSION).tar.bz2
 PROXYCHAINS_NG_SITE = http://downloads.sourceforge.net/project/proxychains-ng
 
+define PROXYCHAINS_NG_POST_INSTALL_TARGET
+	$(INSTALL) -m 755 $(@D)/src/proxychains.conf $(TARGET_DIR)/etc
+endef
+
+PROXYCHAINS_NG_POST_INSTALL_TARGET_HOOKS += PROXYCHAINS_NG_POST_INSTALL_TARGET
+
 $(eval $(autotools-package))
-- 
1.7.7.6

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

* [Buildroot] [PATCH] proxychains-ng: copy proxychains.conf to target dir
  2013-01-16  4:10 [Buildroot] [PATCH] proxychains-ng: copy proxychains.conf to target dir Stefan Fröberg
@ 2013-01-16  6:29 ` Peter Korsgaard
  2013-01-16 11:45   ` Stefan Fröberg
  0 siblings, 1 reply; 3+ messages in thread
From: Peter Korsgaard @ 2013-01-16  6:29 UTC (permalink / raw)
  To: buildroot

>>>>> "Stefan" == Stefan Fr?berg <stefan.froberg@petroprogram.com> writes:

 Stefan> proxychains installation does not copy proxychains.conf to 
 Stefan> $(TARGET_DIR)/etc anymore so we have to do it manually now.

 Stefan> Signed-off-by: Stefan Fr?berg <stefan.froberg@petroprogram.com>
 Stefan> ---
 Stefan>  package/proxychains-ng/proxychains-ng.mk |    6 ++++++
 Stefan>  1 files changed, 6 insertions(+), 0 deletions(-)

 Stefan> diff --git a/package/proxychains-ng/proxychains-ng.mk b/package/proxychains-ng/proxychains-ng.mk
 Stefan> index 9cf24bf..70d35b6 100644
 Stefan> --- a/package/proxychains-ng/proxychains-ng.mk
 Stefan> +++ b/package/proxychains-ng/proxychains-ng.mk
 Stefan> @@ -7,4 +7,10 @@ PROXYCHAINS_NG_VERSION = 4.4
 Stefan>  PROXYCHAINS_NG_SOURCE = proxychains-$(PROXYCHAINS_NG_VERSION).tar.bz2
 Stefan>  PROXYCHAINS_NG_SITE = http://downloads.sourceforge.net/project/proxychains-ng
 
 Stefan> +define PROXYCHAINS_NG_POST_INSTALL_TARGET
 Stefan> +	$(INSTALL) -m 755 $(@D)/src/proxychains.conf $(TARGET_DIR)/etc

Does the configuration file really need to be executable?

/etc is almost guaranteed to exist, but for consistency please use
install -D . $(TARGET_DIR)/etc/proxychains.conf

-- 
Bye, Peter Korsgaard

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

* [Buildroot] [PATCH] proxychains-ng: copy proxychains.conf to target dir
  2013-01-16  6:29 ` Peter Korsgaard
@ 2013-01-16 11:45   ` Stefan Fröberg
  0 siblings, 0 replies; 3+ messages in thread
From: Stefan Fröberg @ 2013-01-16 11:45 UTC (permalink / raw)
  To: buildroot

16.1.2013 8:29, Peter Korsgaard kirjoitti:
>>>>>> "Stefan" == Stefan Fr?berg <stefan.froberg@petroprogram.com> writes:
>  Stefan> proxychains installation does not copy proxychains.conf to 
>  Stefan> $(TARGET_DIR)/etc anymore so we have to do it manually now.
>
>  Stefan> Signed-off-by: Stefan Fr?berg <stefan.froberg@petroprogram.com>
>  Stefan> ---
>  Stefan>  package/proxychains-ng/proxychains-ng.mk |    6 ++++++
>  Stefan>  1 files changed, 6 insertions(+), 0 deletions(-)
>
>  Stefan> diff --git a/package/proxychains-ng/proxychains-ng.mk b/package/proxychains-ng/proxychains-ng.mk
>  Stefan> index 9cf24bf..70d35b6 100644
>  Stefan> --- a/package/proxychains-ng/proxychains-ng.mk
>  Stefan> +++ b/package/proxychains-ng/proxychains-ng.mk
>  Stefan> @@ -7,4 +7,10 @@ PROXYCHAINS_NG_VERSION = 4.4
>  Stefan>  PROXYCHAINS_NG_SOURCE = proxychains-$(PROXYCHAINS_NG_VERSION).tar.bz2
>  Stefan>  PROXYCHAINS_NG_SITE = http://downloads.sourceforge.net/project/proxychains-ng
>  
>  Stefan> +define PROXYCHAINS_NG_POST_INSTALL_TARGET
>  Stefan> +	$(INSTALL) -m 755 $(@D)/src/proxychains.conf $(TARGET_DIR)/etc
>
> Does the configuration file really need to be executable?

Uuups!  sorry

> /etc is almost guaranteed to exist, but for consistency please use
> install -D . $(TARGET_DIR)/etc/proxychains.conf
>
Okay

Fixing and resubmitting

Stefan

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

end of thread, other threads:[~2013-01-16 11:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-16  4:10 [Buildroot] [PATCH] proxychains-ng: copy proxychains.conf to target dir Stefan Fröberg
2013-01-16  6:29 ` Peter Korsgaard
2013-01-16 11:45   ` Stefan Fröberg

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