Age | Commit message (Collapse) | Author | Files | Lines |
|
Summain default has changed, so to be compatible with old
summain output files, we have to force the checksum algorithm to what
the old default was.
|
|
Root directory is now excluded from kdirstat output done by obnam.
That solution was probably a workaround to a k4dirstat bug, however,
this broke displaying any --root="/" backups and since we now have
qdirstat (and k4dirstat became deprecated) which fixed the bug, it is
time to reintroduce the root dir into the kdirstat output. Discussion
with Ian about this can be read in the thread "Kdirstat merged to
Qdirstat".
|
|
There needs to be some whitspace after the FIFO tag, otherwise k4dirstat
(silently!) stops parsing at this entry.
This was wrong in the initial commit to add FIFO support to kdirstat
4ce858d7c593 ("kdirstat: Handled named pipes (fifos)"), manual testing at the
time apparently only verified that the file was generated, not that it was
valid, sorry.
I updated the test yarns to also check for FIFO, however this is only run
"ASSUMING extended attributes are allowed for users", which does not appear to
be the case on my system. Also the testshould probably test for all sorts of
files, not just regular and FIFO.
Signed-off-by: Ian Campbell <ijc@hellion.org.uk>
|
|
|
|
|
|
|
|
|
|
For some reason this was missing from the mode_str lookup, it is the only type
listed on
https://github.com/thomas-joiner/k4dirstat/blob/master/kdirstat/cache-file-format.txt
which wasn't handled.
For unknown file types emit a commented "#UNHANDLED ..." line rather than
crashing with a backtrace.
Add a yarn for this scenario, although I've not been able to run it due to
"ASSUMING extended attributes are allowed for users".
Real world use has been tested by Florian Geyer,
http://listmaster.pepperfish.net/pipermail/obnam-support-obnam.org/2015-September/003836.html
Cc: Florian Geyer <blueice@fobos.de>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Some stuff found by pycharm code checker.
|
|
On Sat, 2014-12-06 at 19:15 +0200, Lars Wirzenius wrote:
> On Sun, Nov 16, 2014 at 11:24:11AM +0000, Ian Campbell wrote:
> > This implements an ls variant whose output format is compatible with the
> > kdirstat cache format. The best reference I can find for this is from an old fork:
> > https://github.com/thomas-joiner/k4dirstat/blob/master/kdirstat/cache-file-format.txt
> >
> > Recent versions of k4dirstat appear to be able to read the files produced just
> > fine, although this document is not present in their source AFAICT.
> >
> > I find k4dirstat to be an excellent way to visualise what is included in a
> > backup, e.g. for the purposes of excluding file types or marking directories
> > with a CACHEDIR.TAG. My previous solution involved post-processing the output
> > of ls which require a ssh to the machine with the files (for stat() purposes).
> > This approach pulls all of the data out of the obnam repo, so is much faster,
> > less hacky and gets the status of the actual generation and not the current
> > files etc.
>
> Thank you, Ian. I've applied the patch and pushed the changes to
> master on git.liw.fi.
Thanks.
> It would be nice if someone wrote a test case for this (in yarn form),
> so that I'll notice when I break it.
Your wish etc etc... (it even found a bug!)
8<----------------
>From 8f1aecfee8ec34e5d060f81ee47f9fd8aacdc201 Mon Sep 17 00:00:00 2001
From: Ian Campbell <ijc@hellion.org.uk>
Date: Sun, 7 Dec 2014 15:19:57 +0000
Subject: [PATCH] Add a yarn for testing kdirstat cachefile generation.
Rather simplistic, but caught one bug (unwanted newline at start of file) which
is also fixed here.
|
|
|
|
This implements an ls variant whose output format is compatible with the
kdirstat cache format. The best reference I can find for this is from an old fork:
https://github.com/thomas-joiner/k4dirstat/blob/master/kdirstat/cache-file-format.txt
Recent versions of k4dirstat appear to be able to read the files produced just
fine, although this document is not present in their source AFAICT.
I find k4dirstat to be an excellent way to visualise what is included in a
backup, e.g. for the purposes of excluding file types or marking directories
with a CACHEDIR.TAG. My previous solution involved post-processing the output
of ls which require a ssh to the machine with the files (for stat() purposes).
This approach pulls all of the data out of the obnam repo, so is much faster,
less hacky and gets the status of the actual generation and not the current
files etc.
|
|
This fills in the displayed file mode permission mode bits such as
character, block, setuid, sticky, etc.
|
|
|
|
|
|
|
|
Add test case. Fix code.
Reported-by: Junyx
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Reported-by: Biltong
|
|
Patch by Lars Kruse.
|
|
Patch by Itamar Turner-Trauring.
|
|
Patch by Martijn Grendelman.
|
|
|
|
Baseo on patch by Damien Couroussé, but with changes
to make it apply to the current version of Obnam, and
to make the test suite pass.
|
|
Patch from Peter Valdemar Mørch
|
|
This way, user can use --output=foo.
|
|
|
|
|
|
|
|
Patch by Peter Valdemar Mørch.
|
|
Thanks, Edward Allcutt, for the suggestion.
|
|
|