Ticket #29 (closed task: fixed)

Opened 7 years ago

Last modified 7 years ago

Replace backsticks (`) with $().

Reported by: William Owned by: chris@…
Priority: minor Milestone: CLFS Standard 1.0.0
Component: BOOK Version: CLFS Standard 1.0.0
Keywords: Cc:

Description

I would like to rise this issue again since some users tends to read it as single quote (') instead. To make the book clearer, I suggest replace those backsticks with $().

This issue exists in LFS/BLFS too.

William

Change History

comment:1 Changed 7 years ago by jim

Could you provide some examples of the necessary changes for the team to evaluate.

comment:2 Changed 7 years ago by William

Two examples:

  1. LFS toolchain adjusting.

SPECFILE=dirname $(gcc -print-libgcc-file-name)/specs && gcc -dumpspecs > $SPECFILE

can be written as

SPECFILE=$(dirname $(gcc -print-libgcc-file-name))/specs && gcc -dumpspecs > $SPECFILE

Why do we use both syntax at the same time?

  1. BLFS X.Org.

sed -i -e "s@#include <linux/config.h>@/* & */@" \

grep -lr linux/config.h *

can be changed to sed -i -e "s@#include <linux/config.h>@/* & */@" \

$(grep -lr linux/config.h .)

For the sake of clarity.

comment:3 Changed 7 years ago by jim

  • Owner changed from clfs-commits@… to mattd

Will assign a couple of people to look at this to see what it would affect. Matt, can you see what changes would be needed to support this.

comment:4 Changed 7 years ago by jim

  • Milestone set to CLFS 1.0

comment:5 Changed 7 years ago by chris@…

The only places in the book where backticks are used are the "variables" section (for setting the LFS_HOST, as well as LFS_TARGET for mips64) and the toolchain adjustment.

comment:6 Changed 7 years ago by jim

  • Version changed from unstable to 1.0

comment:7 Changed 7 years ago by jim

  • Version changed from 1.0 to 1.0.0

comment:8 Changed 7 years ago by chris@…

  • Owner changed from mattd to chris@…
  • Status changed from new to assigned

comment:9 Changed 7 years ago by chris@…

  • Status changed from assigned to closed
  • Resolution set to fixed

Fixed in r2015.

Note: See TracTickets for help on using tickets.