Opened 19 years ago

Closed 19 years ago

#1551 closed defect (fixed)

Section 6.54.2 nitpick

Reported by: randy@… Owned by: manuel@…
Priority: lowest Milestone:
Component: Book Version: TESTING
Severity: trivial Keywords:
Cc:

Description

Noted in the first paragraph of configuring shadow:

"This package contains utilities to add, modify, and delete users and groups; set and change their passwords; and perform other administrative tasks."

It is my belief that this is improper use of semicolons. It is my understanding that semicolons can be used where a period could also be used. The clauses separated by semicolons should be complete sentences.

I suggest changing the semicolons to commas in the listed sentence.

Change History (6)

comment:1 by randy@…, 19 years ago

Also noted in the same paragraph:

"must be shadow-compliant"

Because "shadow" is being used as a Package name here, perhaps it should be capitalized?

comment:2 by Matthew Burgess, 19 years ago

I don't know how authoratative it is, but http://owl.english.purdue.edu/handouts/grammar/g_overvw.html says

"In addition to using a semicolon to join related independent clauses in compound sentences, you can use a semicolon to separate items in a series if the elements of the series already include commas."

In this context, I believe the semicolon usage is correct, as we're listing functionality of the programs, and the first item in the list contains commas.

As for comment 1, I agree of course: s/shadow-compliant/Shadow-compliant/

comment:3 by manuel@…, 19 years ago

Owner: changed from lfs-book@… to manuel@…

comment:4 by manuel@…, 19 years ago

Status: newassigned

Fixed issue 2 in trunk r6065. Waiting to merge it in testing. Is issue 1 a WONTFIX?

comment:5 by Matthew Burgess, 19 years ago

Yes, I think so, as Randy hasn't hit me with a grammarian-cluebat yet :) Randy, feel free to re-open if you think it still needs addressing. Manuel, please merge r6065 to testing, then close this one off as 'FIXED', thanks.

comment:6 by manuel@…, 19 years ago

Resolution: fixed
Status: assignedclosed

Fixed issue 2 in testing r6067.

Note: See TracTickets for help on using tickets.