From: Frank Terbeck <ft@bewatermyfriend.org>
To: zsh-workers@zsh.org
Subject: Re: PATCH: Add `--enable-additional-fpath' configure option
Date: Mon, 18 Apr 2011 13:45:37 +0200 [thread overview]
Message-ID: <8762qb6avy.fsf@ft.bewatermyfriend.org> (raw)
In-Reply-To: <110417132318.ZM11940@torch.brasslantern.com> (Bart Schaefer's message of "Sun, 17 Apr 2011 13:23:18 -0700")
Bart Schaefer wrote:
> On Apr 17, 3:49pm, Frank Terbeck wrote:
> } Comments? I won't be committing this before I get ACKs. :)
>
> This looks OK #ifdef-wise; if it compiles both ways you probaby didn't
> break anything. :-) I have a feeling that there's a more compact way to
> do it (#if in fewer places), but maybe that would be less clear to a
> reader of the code.
Heh. I did test it both ways. But I just gave it a spin with more than 2
additional directories and it broke (a little), because s/// != s///g
with sed. The following patch is needed on top of the old one. I'll be
committing this in the evening.
diff --git a/Src/zsh.mdd b/Src/zsh.mdd
index c420bcc..9f19b45 100644
--- a/Src/zsh.mdd
+++ b/Src/zsh.mdd
@@ -87,7 +87,7 @@ zshpaths.h: Makemod $(CONFIG_INCS)
fi; \
fi
@if test x$(additionalfpath) != x; then \
- fpath_tmp="`echo $(additionalfpath) | sed -e 's:,:\", \":'`"; \
+ fpath_tmp="`echo $(additionalfpath) | sed -e 's:,:\", \":g'`"; \
echo "#define ADDITIONAL_FPATH { \"$$fpath_tmp\" }" >> zshpaths.h.tmp; \
fi
@if cmp -s zshpaths.h zshpaths.h.tmp; then \
prev parent reply other threads:[~2011-04-18 11:48 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <ft@bewatermyfriend.org>
2011-04-17 13:49 ` Frank Terbeck
2011-04-17 18:11 ` Peter Stephenson
2011-04-17 20:23 ` Bart Schaefer
2011-04-18 11:45 ` Frank Terbeck [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8762qb6avy.fsf@ft.bewatermyfriend.org \
--to=ft@bewatermyfriend.org \
--cc=zsh-workers@zsh.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
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).