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.