[PATCH] configure: replace $(realpath emacs) with $(cd emacs && pwd -P)

[PATCH] configure: replace $(realpath emacs) with $(cd emacs && pwd -P)

[Tomi Ollila]

For portability; the realpath command (e.g. from GNU coreutils)
is not so common outside Linux systems.

The "$(cd emacs && pwd -P)" replaces that realpath(1) execution
suitably in this context (using just bash(1) builtins).
---

Quotes ("") added for it to be better example for someone(tm) who
may look it while writing some other shell script elsewhere;
here, as $IFS characters are not (currently!) allowed in $PWD
those makes no difference...

 configure | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure b/configure
index 40e8b255..cfa9c09b 100755
--- a/configure
+++ b/configure
@@ -1536,7 +1536,7 @@ EOF
     if [ $WITH_PYTHON_DOCS = "1" ]; then
         echo "tags.add('WITH_PYTHON')"
     fi
-    printf "rsti_dir = '%s'\n" $(realpath emacs)
+    printf "rsti_dir = '%s'\n" "$(cd emacs && pwd -P)"
 } > sphinx.config
 
 # Finally, after everything configured, inform the user how to continue.
-- 
2.25.1

Re: [PATCH] configure: replace $(realpath emacs) with $(cd emacs && pwd -P)

[David Bremner]

Tomi Ollila <tomi.ollila@iki.fi> writes:

> For portability; the realpath command (e.g. from GNU coreutils)
> is not so common outside Linux systems.
>
> The "$(cd emacs && pwd -P)" replaces that realpath(1) execution
> suitably in this context (using just bash(1) builtins).

Applied to master. It seems not to make things worse, although my
testing of the OOT build was hampered by the failure of the new python
bindings to build OOT [1].


[1]: id:87h7qnubsw.fsf@tethera.net