Date: prev next · Thread: first prev next last
2010 Archives by date, by thread · List index


Hi Sebastian,

Sebastian Spaeth wrote:
On Sat, 13 Nov 2010 16:09:26 +0100, Giuseppe Castagno wrote:
--- a/bin/install-artwork

+. bin/setup

While that patch might fix things, I believe a script in ./bin should
not assume anything about what the current working directory is. We
should either use a relative directory, relative to the script
location. Or we should source some configuration file created by
./configure to read in the relevant directories.

I think you are right, I attached a revised version of the patch.

I believe this one is a little better, unless I'm completely wrong and the entire ef57e0668c2171036759f244a8bb6b6cc753f594 commit should be reverted for some other reason...

beppec56.

--
Kind Regards,
Giuseppe Castagno
Acca Esse http://www.acca-esse.eu
giuseppe.castagno at acca-esse.eu
beppec56 at openoffice.org
From 932e360a71a2175a2dccdf0fde1e7a2c169b9e54 Mon Sep 17 00:00:00 2001
From: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
Date: Sat, 13 Nov 2010 16:02:56 +0100
Subject: [PATCH] Fix for commit ef57e0668c2171036759f244a8bb6b6cc753f594.

File sourced outside install-artwork.
---
 Makefile.am         |    2 +-
 bin/install-artwork |    2 --
 2 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/Makefile.am b/Makefile.am
index b9da4ee..63a92cf 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -53,7 +53,7 @@ $(STAMP_DIR)/artwork.install : $(OOBUILDDIR)/unpack \
                              $(STAMP_DIR)/prebuild \
                              $(wildcard $(top_srcdir)/src/*.png) \
                              $(wildcard $(top_srcdir)/src/*.bmp)
-       $(TOOLSDIR)/bin/install-artwork $(top_srcdir)/src $(OOBUILDDIR)
+       . $(TOOLSDIR)/bin/setup $(TOOLSDIR)/bin/install-artwork $(top_srcdir)/src $(OOBUILDDIR)
        touch $@
 
 patch.apply: $(OOBUILDDIR)/unpack $(STAMP_DIR)/refresh $(STAMP_DIR)/patch.apply
diff --git a/bin/install-artwork b/bin/install-artwork
index cbb314d..be73b4d 100755
--- a/bin/install-artwork
+++ b/bin/install-artwork
@@ -1,7 +1,5 @@
 #!/bin/sh
 
-. ./setup
-
 # installs artwork, opportunistically from src to dest
 
 src=$1
-- 
1.6.3.3


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.