Skip to content
This repository has been archived by the owner on Feb 24, 2020. It is now read-only.

Documentation: fix formatting, typos, outdated info #1824

Merged
merged 6 commits into from Dec 7, 2015

Conversation

krnowak
Copy link
Collaborator

@krnowak krnowak commented Dec 2, 2015

First commit is quite large, because of the one-sentence-per-line formatting, but it mostly is not changing any text.

The rest of the commits are fixing various mistakes.

The generated archive `stage1.aci` will not contain bash, systemd that comes from external sources.
The only binaries in the archive are built from the sources in the rkt git repository.
Since stage1.aci needs external binaries, they will be taken from the host at run-time.
Packages using the `--with-stage1-flavor=host` option must therefore add a run-time dependency on systemd and bash.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

flavorS?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bah, right you are.

@jonboulle
Copy link
Contributor

While I appreciate the one-sentence-per-line standardisation (and I think it's the right thing for now), I would be remiss if I didn't draw attention to the fact that it might imminently change, per coreos/docs#633

@jonboulle
Copy link
Contributor

While we're here we should probably also see if @joshix has some time to review.


## Downloading Images (ACIs)

[aci-images]: https://github.com/appc/spec/blob/master/spec/aci.md#app-container-image
[appc-discovery]: https://github.com/appc/spec/blob/master/spec/discovery.md#app-container-image-discovery

rkt runs applications packaged as [Application Container Images (ACI)][aci-images] an open-source specification. ACIs consist of the root filesystem of the application container, a manifest and an optional signature.
rkt runs applications packaged as [Application Container Images (ACI)][aci-images] an open-source specification.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rkt runs applications packaged according to the open-source [App Container Image][aci-images] specification.

@@ -4,45 +4,35 @@ User namespaces
Background
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Headings in this file need to be ATX: # Heading.

s/Background/## Background/

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fixed, but somehow github insists on the diff not being outdated.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have been having that problem with several changesets the last week or so.

@joshix
Copy link
Contributor

joshix commented Dec 4, 2015

Needs:

  • address final review comments
  • branch conflict resolve
  • rebase to squash commits

then LGTM to land as an improvement iteration.

We can worry about unbreaking the lines later as we formalize coreos/docs#633 and continue to iterate on all of rkt/Documentation/.

One sentence per line. Add some whitespace between caption and the
text or between text and bullet listings. Remove some whitespace
errors. Fix language tag in some code snippets. Maybe fix some minor
typos.
rkt usually will not be a static binary. We tried once and the result
has problems with networking. It is probably like this because
statically linking glibc into a binary makes the glibc not to use its
plugin based name service switch fully, which is responsible, among
others, for resolving domain names.
Let's default to Carly Container with carly@example.com email. Taken
from appc/spec docs.
Most of them spotted by Josh Wood
@iaguis
Copy link
Member

iaguis commented Dec 7, 2015

LGTM

krnowak added a commit that referenced this pull request Dec 7, 2015
Documentation: fix formatting, typos, outdated info
@krnowak krnowak merged commit 4a67d68 into rkt:master Dec 7, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants