pijul_org / pijul

fix: Directories were not correctly deleted when recording

Prior to this patch, the following test case:

mkdir test
cd test
pijul init
mkdir test
pijul record -n -a -m"First patch"
pijul remove test
pijul record -a -m"Second patch"
pijul status

would give the following output:

Changes not yet recorded:
  (use "pijul record ..." to record a new patch)

        deleted:   test

Untracked files:
  (use "pijul add <file>..." to track them)

        test

This is not right, as the second patch should have removing the directory test/ from the tree. This is a regression compared to pijul-0.9, and a carreful comparison of log output highlights that the syncs variable in record command was empty.

Once again, comparing libpijul source as in crates.io and current version showed there were a missing InodeUpdate::Deleted insertion, for no easy to explain version. Adding this missing insertion fixes the issue.

By lthms on March 21, 2018
This patch is not signed.
A7zrwxdv5utq8AomwXxqAN5TFCWNjsmg7ZGHZ8nUVrinrf57DuMSV8VZyaoV7oduvia7aV1PToXxLdi6oKvugpxy
This patch is in the following branches:
latest
master
testing
In file libpijul/src