[lfs-dev] LFS-8.1-systemd comments.

Bruce Dubbs bruce.dubbs at gmail.com
Wed Dec 27 19:12:47 PST 2017


Continuing:

"6.48.1

--disable-ldconfig
This switch prevents installation of a systemd unit that runs ldconfig at 
boot, making the boot time longer. Remove it if the described feature is 
desired, even though it's not useful for source distributions such as LFS.

This switch prevents installation of a systemd unit that runs ldconfig at 
boot, which is not useful for source distributions such as LFS and makes 
the boot time longer. Remove it if the described feature is desired."

I like this.

---

"--disable-sysusers: ... Both files were created early in this chapter.
                                                     ^
--disable-sysusers: ... Both files were created earlier in this chapter."

Will be fixed.

---
"6.48.2

systemd-mount: is a tool to temporarily mount or auto-mount a drive.

Why not

systemd-mount: A tool to temporarily mount or auto-mount a drive."

OK

"Elsewhere descriptions like "foo: is a tool" are written "foo: Is a 
tool". The short descriptions are in tabular form, it is acceptable to 
start sentences with a capital and end them without a full stop. It should 
be consistent though.  My preference would be to avoid making the command 
to be described part of the sentence, but it occurs quite a lot so I 
haven't flagged them all.

ditto systemd-socket-activate

libsystemd: systemd utility library
             ^
libsystemd: Systemd utility library"

I reviewed all the entries and changed them as needed.
---

"6.49.1

Now prepare procps-ng for compilation:
^^^ ^       ^
Prepare Procps-ng for compilation:"


I removeed the 'Now', but prefer to use package names in the case
that matches the package.

---
"6.49.2

slabtop: Displays detailed kernel slap cache information in real time
                                      ^
slabtop: Displays detailed kernel slab cache information in real time"

Fixed.

---
"6.64.2

All descriptions not capitalized.

dbus-test-tool: is a tool to help packages test D-Bus.
Inconsistent full stop                               ^

ditto for dbus-update-activation-environment"

Fixed.

---
"6.65.3

uuidd: A daemon ... guranteed-unique fashion
                      ^
uuidd: A daemon ... guaranteed-unique fashion"

Fixed.

---
"6.63.3  <!-- s/63/66/ -->

apropos: Searches the whatis database ...

apropos is a link to wahtis"

This was fixed some time ago.
---
"6.69.1

However, this test suite outputs a lot of binary data to the screen, which 
can cause issues with the settings of the current terminal. This can be 
resolved by redirecting the output to a log file.

Maybe something like ...

Since this test suite outputs a lot of binary data to the screen, which 
can cause issues with the settings of the current terminal, the command 
above redirects the output to a log file."

Reworded to:

The test suite outputs a lot of binary data to the screen.  This can
cause issues with the settings of the current terminal.  The problem can 
be avoided by redirecting the output to a log file as shown above.  A
successful test will result in the words "ALL DONE" in the log file
at completion.

---

"A successful test will result in the words "ALL DONE" at completion.

This is not output with &>, nor is it the last line of output. I did

wc -l vim-8.0.586-test.log
37879 vim-8.0.586-test.log
grep -n "ALL DONE" vim-8.0.586-test.log
37797:ALL DONE"

By now, the user should be able to use vim itself to examine the log file 
or use grep to search it.

---
"6.69.3

vimdiff: Edits two or three versions of a file with vim and show differences
                                                                 ^
vimdiff: Edits two or three versions of a file with vim and shows differences"

Fixed.
---
"7.2.1.1

Do not include DNS or Domains entries if you intend to use a static 
/etc/resolv.conf file.

Do not include DNS or Domain entries if you intend to use a static 
/etc/resolv.conf file."

It sounds weird, but the 'Domains' refers to an explicit entry immediately
above the line.

---
"7.3.3.2

To do this, add a "softdep" line in any /etc/modprobe.d/<filename>.conf 
file. For example:
                                     ^^^
To do this, add a "softdep" line in the corresponding ... ?"

Changed to "... to the corresponding ..."

---
"7.5

You can get list of available time zones by running:
            ^
You can get a list of available time zones by running:"

Fixed.

---

"Please note that timedatectl command can be used only on a system booted 
with systemd
                 ^
Please note that the timedatectl command can be used only on a system 
booted with systemd"

Fixed.

---
"7.6

You can also use localectl utility with the corresponding ...
                 ^
You can also use the localectl utility with the corresponding ..."

Fixed.

---

"Using any of the parameters listed above requires XKeyboard Config 
package from BLFS.
                                                   ^
Using any of the parameters listed above requires the XKeyboard Config 
package from BLFS."

Fixed.

---
"7.7

Note that you can modify /etc/locale.conf with systemd localectl utility.
                                               ^
Note that you can modify /etc/locale.conf with the systemd localectl utility."

Fixed.

---

"Please note that localectl command can be used only on a system booted 
with systemd.
                 ^
Please note that the localectl command can be used only on a system booted 
with systemd."

Fixed.

---
"7.8

The inputrc file is the configuration file for Readline library, ... . It 
works by tranlating
                                               ^ 
               ^
The inputrc file is the configuration file for the Readline library, ... . 
It works by translating"

Fixed.

---
"7.9

See systemd-system.conf(5) manual page for details on each configuration 
option.
    ^
See the systemd-system.conf(5) manual page for details on each 
configuration option.

or

See \command{man 5 systemd-system.conf} for details on each configuration 
option.

as is done elsewhere in the text."

I'm going to leave this as is.  The syntax above is used oftern in man 
pages and the user should not be confused.
---
"7.10.4

See tmpfiles.d(5) manual page for file format details.
    ^
See the tmpfiles.d(5) manual page for file format details.

or

See \command{man 5 tmpfiles.d} for file format details."

Same response as above.

---
"7.10.5

See systemd.unit(5) manual page for more information.
    ^
See the systemd.unit(5) manual page for more information.

or

See \command{man 5 systemd.unit} for more information."

Same response as above.

---
"7.10.8

Long Running Processes or Long-Running Processes? Is the process long or 
is it running long?"

The text makes it clear.

---

"Enable process lingering for only needed users:
Enable process lingering only for users who need it:"

Changed to "selected users"

---
"8.3

Be sure to enable or disable following features ...
                             ^
Be sure to enable or disable the following features ..."

Fixed.

---

"Caution: If the host system has a separate /boot partition, the files 
copied below should go there. The easiest way to do that is to bind /boot 
on the host to /mnt/lfs/boot before proceeding. As the root user in the 
host system

mount --bind /boot /mnt/lfs/boot

Even though it's pretty obvious, maybe mention this should be done from 
outside chroot."

It does say 'As the root user in the <emphasis>host system</emphasis>', 
but it still can use a bit more emphasis.  Fixed.

---

"Is it not maybe safer to use cp -iv to copy to /boot (in case of separate 
mount)?"

Good idea.  I have not seen this as a problem before, but it has good 
educational value.

---

to be continued....

   -- Bruce








More information about the lfs-dev mailing list