qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] Fix install(1) usage to be compatible with OpenBSD's install(1).
@ 2011-08-28  8:01 Brad
  2011-09-02 15:38 ` Anthony Liguori
  0 siblings, 1 reply; 2+ messages in thread
From: Brad @ 2011-08-28  8:01 UTC (permalink / raw)
  To: qemu-devel

Fix install(1) usage to be compatible with OpenBSD's install(1).

When creating a directory via the -d flag the -p flag cannot be
used at the same time. Also in the context of installing QEMU it
doesn't make sense to use the -p flag anyway so use the [default]
-c flag instead.

Signed-off-by: Brad Smith <brad@comstyle.com>

---
 configure |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/configure b/configure
index 1340c33..ad60ea0 100755
--- a/configure
+++ b/configure
@@ -3081,9 +3081,9 @@ echo "TOOLS=$tools" >> $config_host_mak
 echo "ROMS=$roms" >> $config_host_mak
 echo "MAKE=$make" >> $config_host_mak
 echo "INSTALL=$install" >> $config_host_mak
-echo "INSTALL_DIR=$install -d -m0755 -p" >> $config_host_mak
-echo "INSTALL_DATA=$install -m0644 -p" >> $config_host_mak
-echo "INSTALL_PROG=$install -m0755 -p" >> $config_host_mak
+echo "INSTALL_DIR=$install -d -m 0755" >> $config_host_mak
+echo "INSTALL_DATA=$install -c -m 0644" >> $config_host_mak
+echo "INSTALL_PROG=$install -c -m 0755" >> $config_host_mak
 echo "PYTHON=$python" >> $config_host_mak
 echo "CC=$cc" >> $config_host_mak
 echo "CC_I386=$cc_i386" >> $config_host_mak
-- 
1.7.6


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

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

* Re: [Qemu-devel] [PATCH] Fix install(1) usage to be compatible with OpenBSD's install(1).
  2011-08-28  8:01 [Qemu-devel] [PATCH] Fix install(1) usage to be compatible with OpenBSD's install(1) Brad
@ 2011-09-02 15:38 ` Anthony Liguori
  0 siblings, 0 replies; 2+ messages in thread
From: Anthony Liguori @ 2011-09-02 15:38 UTC (permalink / raw)
  To: Brad; +Cc: qemu-devel

On 08/28/2011 03:01 AM, Brad wrote:
> Fix install(1) usage to be compatible with OpenBSD's install(1).
>
> When creating a directory via the -d flag the -p flag cannot be
> used at the same time. Also in the context of installing QEMU it
> doesn't make sense to use the -p flag anyway so use the [default]
> -c flag instead.
>
> Signed-off-by: Brad Smith<brad@comstyle.com>

Applied.  Thanks.

Regards,

Anthony Liguori

>
> ---
>   configure |    6 +++---
>   1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/configure b/configure
> index 1340c33..ad60ea0 100755
> --- a/configure
> +++ b/configure
> @@ -3081,9 +3081,9 @@ echo "TOOLS=$tools">>  $config_host_mak
>   echo "ROMS=$roms">>  $config_host_mak
>   echo "MAKE=$make">>  $config_host_mak
>   echo "INSTALL=$install">>  $config_host_mak
> -echo "INSTALL_DIR=$install -d -m0755 -p">>  $config_host_mak
> -echo "INSTALL_DATA=$install -m0644 -p">>  $config_host_mak
> -echo "INSTALL_PROG=$install -m0755 -p">>  $config_host_mak
> +echo "INSTALL_DIR=$install -d -m 0755">>  $config_host_mak
> +echo "INSTALL_DATA=$install -c -m 0644">>  $config_host_mak
> +echo "INSTALL_PROG=$install -c -m 0755">>  $config_host_mak
>   echo "PYTHON=$python">>  $config_host_mak
>   echo "CC=$cc">>  $config_host_mak
>   echo "CC_I386=$cc_i386">>  $config_host_mak

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

end of thread, other threads:[~2011-09-02 15:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-28  8:01 [Qemu-devel] [PATCH] Fix install(1) usage to be compatible with OpenBSD's install(1) Brad
2011-09-02 15:38 ` Anthony Liguori

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).