some clippy fixes

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.10.1-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rustfmt

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

decoder/aio: add contents() and content_size() calls

Returns a decoder::Contents without a wrapper type, since in this case
we don't want to hide the SeqRead implementation (as done in
decoder::sync). For conviencience also implement AsyncRead if "tokio-io"
is enabled.

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>

rustfmt

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

encoder: only flush owned outputs in 'finish'

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

encoder: flush after writing last entry

some writers may need to be flushed to write out all data

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

rustfmt

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.10.0-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

explicitly pad StatxTimestamp with zeros

otherwise we can have random data in there

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

update d/control

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.9.0-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

add EncoderOutput::to_borrowed()

may more "explicit" than using `.as_mut().into()`, but
otherwise just seems like a useful addition to show the
lifetime constraints on this

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

revert Send requirement from previous patch

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

make aio::Encoder actually behave with async

To really use the encoder with async/await, it needs to support
SeqWrite implementations that are Send. This requires changing a whole
bunch of '&mut dyn SeqWrite' trait objects to instead take a 'T:
SeqWrite' generic parameter directly instead. Most of this is quite
straightforward, though incurs a lot of churn (FileImpl needs a generic
parameter now for example).

The trickiest part is returning a new Encoder instance in
create_directory, as the trait object trick with
SeqWrite::as_trait_object doesn't work if SeqWrite is implemented for
generic '&mut S'.

Instead, work with the generic object directly, and express the
owned/borrowed state in the Encoder (to avoid nested borrowing) as an
enum EncoderOutput.

Add to the aio test to ensure the Encoder is now actually useable.

Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

cargo fmt

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

make Send test not depend on tokio

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.8.0-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

fix Encoder to be Send + Sync

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

debcargo: fix maintainer directive

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

drop futures dependency

we're not actually using it anymore now

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

typo fix: this was supposed to be from_tokio

it's on `tokio-io` after all and there previously used to be
a `form_futures` for the `Async*` traits from `futures`

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

clippy fixups

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

move extra build-dependencies from .builtbot.yml to d/control

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add futures to builbot.yml

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

update builbot.yml for tokio 1.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 0.7.0-1

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

build: fix --no-default-features

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

fix example

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

remove futures-io feature

we don't use it, and it adds unnecessary duplication/complexity.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

clippy: use matches! instead of match

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

update to tokio 1.0

unfortunately, futures::io::AsyncRead and tokio::io::AsyncRead no longer
share a do_poll_read signature, so we need to adapt one to the other
(and also no longer generate some wrapper implementations via macro).

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

bump version to 0.6.2-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

fix `decode_entry` on special files

When using the random accessor to access FIFOs or sockets,
the ranged reader limits the data to only that entry, and
the `decode_entry` will never see a `PAYLOAD` or
`GOODBYE_TABLE` item to finish the entry.
Instead, it'll reach EOF and we need to handle this.
The accessor now tells the decoder to expect EOF as a valid
condition for ending the entry.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

clippy

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.6.1-1

Decoder<StandardReader<T>>: allow access to input

We need this to query the status of the input when decode fails.

more build dep fixups for buildbot

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

have the buildbot install anyhow for now

there are some build/test/dev dependencies issues with
debcargo still open, once those are resolved we can maybe
revert this

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

formatting fixup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

d/control bump

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.6.0-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

encoder: let finish return writer, add into_writer

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.5.0-2

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

fixup d/control

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.5.0-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rename Entry to Stat, add Metadata::builder_from_stat

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

fixup build and test dependencies

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

generate and check-in d/control

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.4.0-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

drop poll_close and poll_position from SeqWrite

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

doc fixup

This is a SignedDuration now instead of a Result

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

introduce StatxTimestamp helper type

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

Makefile: fix dinstzall target

Makefile: build packages inside build/ directory

Makefile: add upload target

bump version to 0.3.0-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

add entry v1 compatiblity test

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

implement Entry v2

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

update mk-format-hashes for a new ENTRY

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

introduce StatxTimestamp helper type

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

add format description to format module

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.2.1-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

more clippy lints

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

clippy: fix bad cast in max allowed content size

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

add some random access tests with hardlinks

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

accessor: impl ReadAt for byte slices

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

tests: add some ACLs

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

acl: User/Group::new method

with some convenience to not require the Permissions type
wraparound.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

buildsys: pass --all-features to cargo test

Otherwise tests which depend on optional dependencies won't
be run. Particularly the one in the `tests/` subdir.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rustc compat fixup

lowers crate's rustc requirement to << 1.43

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

tests: cleanup warnings

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

tests: add fifos and sockets

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

document test helpers a bit

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

remove custom PartialEq impl

Hardlink targets are now compared, but only the parts we
also actually decode, not the chosen names in
`Entry.link_key`.

This way we can just `==` compare entries as long as the
files we use for hardlinking use their canonical path as
key.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

some initial sequential encoder/decoder testing

Tests basic files, hardlinks, symlinks. No special metadata
yet.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

Add TODO file

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

formatting fixups

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

start tests directory

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

add test-harness feature for separate unit tests

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

sync encoder: fix metadat lifetime leaking into encoder

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

Add a metadata builder

for now primarily to get started with tests

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

formatting fixup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.2.0-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

new hashes for the entire archive foramt

This breaks all previous pxar files and represents a clear
cut off of the catar format.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.1.9-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

sort some constants

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

formatting fixups

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

make ReadAt trait more correct

A simple `poll_read_at` with an immutable self reference
lacks the information which poll *operation* is being
polled. An associated type won't work well enough
(particularly with trait objects and lifetimes), and GATs
are unstable (and not advanced enough yet), so we need to
improvise.

To be more async-friendly, the `start_read_at()` method's
Pending now includes a reference to the operation which also
grabs the buffer lifetime, and has to be completed with
`poll_complete()`.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

doc update

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.1.8-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

fix device encoding

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.1.7-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

decoder: remove pub(crate) from some internal items

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

accessor: don't mess with internal decoder state

skipping a filename is trivial enough to not abuse internal
decoder methods

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

decoder: fix typos

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

header: implement Display

for nicer/readable error messages.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

header: add size checks

to catch
- headers with full_size < size_of::<Header>
- headers with content_size > header-specific limit

both should in practice only occur for corrupted streams/archives, but
panic-ing/attempting to allocate huge amounts of memory should be
avoided even for those.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 0.1.6-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

encoder: implement a larger shared file copy buffer

In the future we can give it a size.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>