public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-commits] proj/devmanual:master commit in: tools-reference/sed/
@ 2021-03-29 20:44 Ulrich Müller
  0 siblings, 0 replies; 3+ messages in thread
From: Ulrich Müller @ 2021-03-29 20:44 UTC (permalink / raw
  To: gentoo-commits

commit:     0b00ff1312e734ae453a9f61f504bdc400b2b2df
Author:     Sam James <sam <AT> gentoo <DOT> org>
AuthorDate: Sat Mar 20 14:48:24 2021 +0000
Commit:     Ulrich Müller <ulm <AT> gentoo <DOT> org>
CommitDate: Mon Mar 29 20:41:37 2021 +0000
URL:        https://gitweb.gentoo.org/proj/devmanual.git/commit/?id=0b00ff13

tools-reference/sed: emphasise need for delimiter care

Bug: https://bugs.gentoo.org/685160
Signed-off-by: Sam James <sam <AT> gentoo.org>
Signed-off-by: Ulrich Müller <ulm <AT> gentoo.org>

 tools-reference/sed/text.xml | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/tools-reference/sed/text.xml b/tools-reference/sed/text.xml
index b971946..f116eac 100644
--- a/tools-reference/sed/text.xml
+++ b/tools-reference/sed/text.xml
@@ -132,7 +132,12 @@ The above will replace <c>irksome texting</c> with
 <p>
 If the pattern or the replacement string contains the forward slash character,
 it is usually easiest to use a different delimiter. Most punctuation characters
-are allowed, although backslash and any form of brackets should be avoided.
+are allowed, although backslash and any form of brackets should be avoided. You
+should choose your delimiter <b>with care</b> to ensure it cannot appear in any
+strings involved in the subject/replacement. For example, using <c>sed</c> with
+CFLAGS is hazardous because it is user-supplied data (so may contain any
+character), but one should in particular avoid e.g.
+<uri link="https://bugs.gentoo.org/685160">the colon</uri> here.
 </p>
 
 <codesample lang="ebuild">


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

* [gentoo-commits] proj/devmanual:master commit in: tools-reference/sed/
@ 2021-03-29 20:44 Ulrich Müller
  0 siblings, 0 replies; 3+ messages in thread
From: Ulrich Müller @ 2021-03-29 20:44 UTC (permalink / raw
  To: gentoo-commits

commit:     86795e1cd244f0c51efcc7484617e5533799737b
Author:     Sam James <sam <AT> gentoo <DOT> org>
AuthorDate: Sun Mar 21 01:39:00 2021 +0000
Commit:     Ulrich Müller <ulm <AT> gentoo <DOT> org>
CommitDate: Mon Mar 29 20:41:36 2021 +0000
URL:        https://gitweb.gentoo.org/proj/devmanual.git/commit/?id=86795e1c

tools-reference/sed: drop stray character

Signed-off-by: Sam James <sam <AT> gentoo.org>
Signed-off-by: Ulrich Müller <ulm <AT> gentoo.org>

 tools-reference/sed/text.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools-reference/sed/text.xml b/tools-reference/sed/text.xml
index aac3524..b971946 100644
--- a/tools-reference/sed/text.xml
+++ b/tools-reference/sed/text.xml
@@ -65,7 +65,7 @@ constructs are specific to <c>GNU sed 4</c> <d/> on non-GNU userland archs, the
 guaranteed to be installed as part of <c>@system</c>. This was not
 always the case,
 which is why some packages, particularly those which use <c>sed -i</c>, have
-<c>DEPEND</c> s upon <c>&gt;=sys-apps/sed-4</c>.
+a <c>DEPEND</c> upon <c>&gt;=sys-apps/sed-4</c>.
 </p>
 </body>
 


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

* [gentoo-commits] proj/devmanual:master commit in: tools-reference/sed/
@ 2022-06-24  9:47 Ulrich Müller
  0 siblings, 0 replies; 3+ messages in thread
From: Ulrich Müller @ 2022-06-24  9:47 UTC (permalink / raw
  To: gentoo-commits

commit:     2f5c310d924b12bac53e2d75319f9f15db5fabb6
Author:     Ulrich Müller <ulm <AT> gentoo <DOT> org>
AuthorDate: Wed Jun 22 13:39:48 2022 +0000
Commit:     Ulrich Müller <ulm <AT> gentoo <DOT> org>
CommitDate: Fri Jun 24 09:46:46 2022 +0000
URL:        https://gitweb.gentoo.org/proj/devmanual.git/commit/?id=2f5c310d

tools-reference/sed: Add cross reference to "Further reading"

Signed-off-by: Ulrich Müller <ulm <AT> gentoo.org>

 tools-reference/sed/text.xml | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/tools-reference/sed/text.xml b/tools-reference/sed/text.xml
index f116eac..cbca421 100644
--- a/tools-reference/sed/text.xml
+++ b/tools-reference/sed/text.xml
@@ -893,10 +893,11 @@ write this
 <body>
 
 <p>
-The author recommends <e>Mastering Regular Expressions</e> by Jeffrey E. F. Friedl
-for those who wish to learn more about regexes. This text is remarkably devoid
-of phrases like "let <c>t</c> be a finite contiguous sequence such that <c>t[n] ∈ ∑
-∀ n</c>", and was <e>not</e> written by someone whose pay cheque depended upon them being
+The author recommends <uri link="::appendices/further-reading/#Books">
+<e>Mastering Regular Expressions</e> by Jeffrey E. F. Friedl</uri> for those who
+wish to learn more about regexes. This text is remarkably devoid of phrases like
+"let <c>t</c> be a finite contiguous sequence such that <c>t[n] ∈ ∑ ∀ n</c>",
+and was <e>not</e> written by someone whose pay cheque depended upon them being
 able to express simple concepts with pages upon pages of mathematical and Greek
 symbols.
 </p>


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

end of thread, other threads:[~2022-06-24  9:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-03-29 20:44 [gentoo-commits] proj/devmanual:master commit in: tools-reference/sed/ Ulrich Müller
  -- strict thread matches above, loose matches on Subject: below --
2021-03-29 20:44 Ulrich Müller
2022-06-24  9:47 Ulrich Müller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox