BLFS Trac wrote: > #1855: Error in PDL-2.4.2 installation > ------------------------------------------+--------------------------------- > Reporter: [EMAIL PROTECTED] | Owner: [EMAIL PROTECTED] > Type: defect | Status: closed > > Priority: high | Milestone: 6.2 > > Component: BOOK | Version: SVN > > Severity: major | Resolution: fixed > > Keywords: Perl PDL ExtUtils::MakeMaker | > ------------------------------------------+--------------------------------- > Changes (by [EMAIL PROTECTED]): > > * resolution: => fixed > * status: assigned => closed > > Comment: > > Fixed in r5931 by adding this sed to the PDL instructions: > > sed -i 's/\(dirname);\)/\1\nuse blib;/' Demos/BAD*demo.pm.PL
Interesting construct. I was confused at first by the \1 in the replacement until I realized the parens were unbalanced in the pattern. Since the text we are looking for is at the end of the line, I wonder why you just didn't use: sed -i 's/dirname);/&\nuse blib;/' Demos/BAD*demo.pm.PL or sed -i '/dirname);/a\nuse blib;' Demos/BAD*demo.pm.PL which do the same thing. I've never liked escaping parens or braces if I can avoid them. BTW, this is just a discussion. No changes are needed. -- Bruce -- http://linuxfromscratch.org/mailman/listinfo/blfs-book FAQ: http://www.linuxfromscratch.org/blfs/faq.html Unsubscribe: See the above information page
