nailing-cargo: Break out get_dependency_tables

NFC.  This function needs enhancing.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Work around cargo annoyance with workspaces

When the invocation $subdir is a workspace, with -u, cargo complains
that the packages in the linkfarm are different to the ones in the
source area.

Fix this by, in this case, adjusting those path references to point
into the linkfarm.

cargo seems to use realpath on one of these paths, so we must do so
too, or cargo complains that two things are different which are
literally two different paths to the very same symlink.

With this change I am able to build lexpr-rs, both in its own right,
and as a dependency.  Closes #1.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Make a shallow symlink farm

Fighting Cargo to avoid this has become too much hard work.

In particular, otherwise with workspaces we would have to edit all of
the member paths.  And having done that the sources themselves claim
to be part of a different workspace, so it's all awful.

I have also seen some weird behaviours where cargo didn't work quite
right because the package wasn't convincing enough.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: oot_massage_cmdline: Initialise $pre and $post

Otherwise if we don't write anything to them, we produce an undef
warning.

No functional change with current code.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Make oot_absdir available earlier

No functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Chase down workspace members

When workspaces are in use, we need to look at their members.
This is quite a palaver.

We want to process them after all the stuff in the config, so we must
queue them up when we see them.  And we want to chase recursively,
perhaps.  (We certainly will do when we extend this to path
dependencies.)  So rather than an actual recursive function, we have a
queue.

We use realpath(1) to check that the paths we find don't
escape.  (Strictly, this code might [l]stat or readdir various
pathnames controlled by hostile cargo metadata, but that's all.
Basically, I'm assuming it's OK to pass a hostile path to realpath(1)
and then check what realpath(1) said.)

Now we record a manifest as relevant if it has a [workspace], even if
it didn't have a package.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Change read_manifest return convention

Now it returns $ws too, which is going to be workspace information.

Right now it's just undef, so no functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Pass $org_subdir to read_manifest

Currently this is just $subdir.  But we are going to recursively chase
down workspaces and path dependencies, and we want to know where they
came from so we can check the paths for sanity.

No functional change yet

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
.

nailing-cargo: Print $why in error messages

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Pass $why to read_manifest

This will help with error messages.

Not used yet.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Make ignored-subdir message more accurate

Maybe there was a Cargo.toml but we didn't like it.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Print a better error about missing update mode

If the user forgot generate-lockfile, we would fail with a confusing
ENOENT error from the command script.  Instead, spot this situation.

This also gives us a place to do decide to mkdir -p or something.

Related to #5, but does not fix it completely.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Try to fix salsa formatting

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix link (actually)

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix link

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix link

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Improve grammar

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix grammar

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix typo

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix busted link

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Discussion of the scope of the local crate feature

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

misc.online: Fix

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

misc.online: Support for `auto`

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Update for actual -u functionality

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Allow --cargo-lock-update in in-tree builds

We pass --locked otherwise!

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Restore -u and -U options

People will need these occasionally - even in the basic runes.

They were still in README.md.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Mention generate-lockfile

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Clarify `oot.command'

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Clarify `oot.user'

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Delete reference to `fetch' in `online' discusson

This is now mentioned elsewhere.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Minor pronoun clarification

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix leftover ref to -ARPI, should be -T

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Add explicit invitation to submit oot sudo support

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Add explicit invitation to submit bugs/fixes for props

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Change an `out of tree' to `out-of-tree'

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Fix handling of missing oot.use

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Fix handling of unrecogised (ie cargo) short options

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Trigger out of tree with oot.user too

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Punctation fix

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Minor fixes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Reorder options to put most usual first

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Collapse function list a bit

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Cross-references, add missing </a>

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Cross-references

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Paragraph indentation, etc.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Pass options right after the cargo subommand

This means that the user can pass non-option arguments to cargo (eg
for cargo run).

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

New scheme for subcommand handling - seems to work, ish

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

New scheme for subcommand handling - wip

Does not work yet.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

New scheme for subcommand handling - wip

Does not work yet.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: wip new argument parsing control spec

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Implement new arg parsing rules

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Move arg parsing into a subroutine

No functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

oot: provide `disable' mode

This allows more-locally overriding a more-global oot setting.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: update-readme-toc: add autogeneration warning

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Provide and run update-readme-toc

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: wip TOC, seems to generate nice output

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: wip TOC

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: wip TOC

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: wip TOC

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Minor fiddling

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Proposed invocation rework

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Config reference tidying

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Config reference: misc.online

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Config reference: arch

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: oot config: discuss build dir contents

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Config reference: oot

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Config reference: packages and subdirs

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Config, start having section names in headings

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Clarify the precise `unified filesystem' view needs

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix missing bullet point re alt lock leafname

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: With oot.use=ssh, user may be just username

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix discussion of Cargo.lock leafname filesystem

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Improve Make Cargo.lock leafnamme options

Make the grammar and punctuation more consistent.  Reorganise the list
to make it easier to read.

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Make -O | --offline ordering consistent

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix bungled markup for `fetch`

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Add missing comma to -m description

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Reorganise -A description to clarify it

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Add missing word in -A description

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Clarify wording re `packages`

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Remove a confusing comma

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Write ` ` around Cargo.foo filenames

This was previously not always done consistently.

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Make 4th function grammar consistent

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Remove ` ` around make

I think now ` ` are not used around program names.

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Remove ` ` around nailing-cargo and cargo

Reported-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Clarify dirtying bug

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: More minor docs updates

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Discourage `[packages]' slightly

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Reword Limitations paragraph slightly

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: More fixes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Spelling fixes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix a formatting error

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Try LICENSE as a SPDX file

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Add LICENSE which will help placate Gitlab

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Minor improvements

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Fix mangled -A<arch> description

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: Various updates

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Change target-dir option

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Add `others` to copyright statement

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

README.md: documentation

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

nailing-cargo: Rename -T argument to -A

This avoids a clash!

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>