Page MenuHomeFreeBSD

add instructions for SMTP access setup

Authored by danger on Jan 1 2019, 11:44 PM.


Diff Detail

rD FreeBSD doc repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

danger created this revision.Jan 1 2019, 11:44 PM
danger edited the summary of this revision. (Show Details)Jan 1 2019, 11:49 PM

Noticed only one thing, otherwise it looks good to me.

2330 ↗(On Diff #52470)

There's an errant double-space here.

Thank you for working on this

2300 ↗(On Diff #52470)

Should explain more about the /mail principal, and how to set it

danger added inline comments.Jan 2 2019, 8:26 AM
2300 ↗(On Diff #52470)

From my understanding, it should be created automatically during the referenced process at xref=kerberos-ldap

rpokala added a subscriber: rpokala.Jan 3 2019, 7:00 AM


ygy edited reviewers, added: docs; removed: Doc Committers.Jan 3 2019, 2:01 PM
ygy added a subscriber: ygy.
danger updated this revision to Diff 52585.Jan 5 2019, 2:51 PM

resolved items pointed out in the reviews

danger updated this revision to Diff 52586.Jan 5 2019, 2:55 PM

fix typo

danger added inline comments.Jan 5 2019, 3:02 PM
2277 ↗(On Diff #52586)

should there be the infrastructure?

bcr accepted this revision.Jan 8 2019, 9:04 AM
bcr added a subscriber: bcr.

OK to be added to the committers guide. Thanks!

This revision is now accepted and ready to land.Jan 8 2019, 9:04 AM
mat added a subscriber: mat.Jan 8 2019, 11:23 AM
mat added inline comments.
96–97 ↗(On Diff #52586)
<literal><systemitem class="fqdomainname"></systemitem>:587</literal>
2283–2284 ↗(On Diff #52586)
<literal><systemitem class="fqdomainname"></systemitem>:587</literal>
2292 ↗(On Diff #52586)
2293 ↗(On Diff #52586)
2299–2300 ↗(On Diff #52586)
2305 ↗(On Diff #52586)
2317 ↗(On Diff #52586)
2324–2325 ↗(On Diff #52586)
(<literal>Authenticated sender: <replaceable>username</replaceable></literal>)
bcr requested changes to this revision.Jan 8 2019, 1:35 PM

@mat: Ah, those are good additions to have. I should have noticed. I'm un-approving... ;-)

This revision now requires changes to proceed.Jan 8 2019, 1:35 PM

I added those <literal> tags, but am getting the following errors, any ideas?

doc/en_US.ISO8859-1/articles/committers-guide/article.xml:97: element literal: validity error : Element systemitem is not declared in literal list of possible children

danger updated this revision to Diff 52768.Jan 11 2019, 2:36 PM
  • add the literal tags around the FQDNs
bcr accepted this revision.Jan 11 2019, 8:10 PM

I think it fulfills the criteria for integration into the document. Approved!

This revision is now accepted and ready to land.Jan 11 2019, 8:10 PM
This revision was automatically updated to reflect the committed changes.