getopt: remove skip logic

The 'skip' variable was set to indicate that the "rest of
the args" is to be copied into the 'rest' vec. We can do
this directly and avoid the 'if' case in the loop
altogether.

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

getopt: indentation reduction

See the `git diff -w` output:
|diff --git a/src/getopts.rs b/src/getopts.rs
|index 6548a01..517cc37 100644
|--- a/src/getopts.rs
|+++ b/src/getopts.rs
|@@ -72,7 +72,10 @@ pub fn parse_arguments(
|     while pos < args.len() {
|         if skip {
|             rest.push(args[pos].clone());
|-        } else {
|+            pos += 1;
|+            continue;
|+        }
|+
|         match parse_argument(&args[pos]) {
|             RawArgument::Separator => {
|                 skip = true;
|@@ -135,7 +138,6 @@ pub fn parse_arguments(
|                 rest.push(value);
|             }
|         }
|-        }
|
|         pos += 1;
|     }

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

parse_arguments: work with utf8 bytes and reduce indentation

We are only caring about '=' and '-' which are single-byte
codepoints, so there's no reason not to work on a byte
slice.

Also, some indentation reduction:

Transform
    if (a) {
        ...
        return A;
    }
    return B;
into
    if (!a)
        return B;
    return A;

and
    if (a)
        if (b)
            foo()
into
    if (a && b)
        return;

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

src/catar/encoder.rs: write filenames

src/catar/encoder.rs: write file data

src/catar/encoder.rs: write entry and symlink data

make map_struct functions public

catar/encoder.rs: improve error handling

cleanups, avoid compiler warnings

catar/encoder: detect symlink and regular files

catar/encoder.rs: first try

timer: setup_timeout_handler cannot fail

and if it does it panics anyway

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

backup-client: allow to specify directories (prepare for catar backup)

start implementing catar

new helper map_struct and map_struct_mut

cleanup: reduce compiler warnings

remove outdated comment

touch_chunk: use libc::lutimensat

move lookup_datastore() to backup/datastore.rs

gather usage statistics during garbage collection

open_file_locked: improve error message

sweep_used_chunks: print percentage

use openssl for faster hashing

chunk_store: reduce number of directories

Else, scans everything takes too long ...

src/api3/datastore.rs: new file

use return value (avoid compiler warning)

fix mutability for chunk store

backup-client: add optional chunk-size parameter

backup-client: do not start garbage collection after backup

backup-client: use 1M chunks, make chunk_size configurable

backup-clinet: specify target file name

add test code to access static global state

use proxmox-backup as package name

fix tests

try to improve error messages

cleanups: avoid compiler warnings

cleanup - avoid too much indentation

cleanup

ChunkStore::sweep_used_chunks - use openat to speedup directory walks

use timers with a signal for file locking

* rename lock_file -> open_file_locked,
* add lock_file as a function working on already-opened
  files
* change timeout types to std::time::Duration

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

add tools::timer module

This wraps POSIX timers via timer_create(2).

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

add rust standard .gitignore file

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

move lock_file to tools.rs

impl sweep_used_chunks, first try

move sweep_used_chunks to chunk_store

start impl garbage collection

start implementing DataStore

start implementing ImageIndexReader

avoid compiler warnings

fix chunk store file paths

improve error messages

image_index.rs: implement atomic write/rename

rebane ImageIndex to ImageIndexWriter

use u64 to store time

image_index.rs: write header

image_index.rs:write idx of chunks (use mmap)

backup-client: add bash completion for datastore names

install backup-client binary

add backup-client bash completion

remove debug println

move file_chunker to tools

file_chunker: use Read trait

backup-client: pass chunk_size as parameter to file_chunker

backup-client: read file by chunks

bin/backup-client.rs: nbew tool to play with chunk store

api3/config/datastore.rs: create datastore files

improve completion

complete_datastore_name: write more compact code

implement completion functions

section_config.rs: remove debug output

add comment

add comment to make intensions more clear

cleanup parameter types

implement completion for enums

improve completion

do not use default completion

improve bash completion

impl bash completion: first try

install bash completion scripts

Completion itself is not implemented.

cli/command.rs: add new type CliCommandMap

implement builder pattern for CliCommand

cli/command.rs: implement prefix match

impl From for CommandLineInterface

rename CmdDef to CommandLineInterface

src/cli/command.rs: move code into separate file

try to support nested cli command definitions

api3/config/datastore.rs: implement delete

file_set_contents: correctly use perm parameter

implement tools::file_set_contents

api3/config/datastore.rs: impl create

bin/pbs-datastore.rs: start implementing cli commands

rename entry to item

api3/config/datastore.rs: return real data

config/datastore.rs - open file for reading

create config dir

simplify code

remove some compiler warnings

rename data_store to datastore

cleanup api3 structure

add data_store configuration