Added support for fifos and sockets (#42)

.github/workflows/test.yaml

@@ -69,17 +69,25 @@ jobs:
         run: |
           tar xzf try-dist.tgz/try-*.tgz --strip-components=1
           rm -r try-dist.tgz
-          ./configure
+          ./configure --disable-utils
           make all
+          sudo make install
 
       - name: Run tests with shell fallbacks
         run: |
           ! which try-summary
           ! which try-commit
           scripts/run_tests.sh
 
+      - name: Clean up build
+        run: |
+         sudo make clean
+
       - name: Install utilities
         run: |
+          autoconf
+          ./configure
+          make all
           sudo make install
 
       - name: Run tests with utilities
@@ -116,17 +124,25 @@ jobs:
       - name: Configure and build utilities
         run: |
           autoconf
-          ./configure
+          ./configure --disable-utils
           make all
+          sudo make install
 
       - name: Run tests with shell fallbacks
         run: |
           ! which try-summary
           ! which try-commit
           scripts/run_tests.sh
 
+      - name: Clean up build
+        run: |
+          sudo make clean
+
       - name: Install utilities
         run: |
+          autoconf
+          ./configure
+          make all
           sudo make install
 
       - name: Run tests with utilities

Makefile.in

@@ -16,13 +16,14 @@ INSTALL=@INSTALL@
 DISTDIR=@PACKAGE_TARNAME@-@PACKAGE_VERSION@
 DISTTGZ=$(DISTDIR).tgz
 
-TARGETS=$(if $(findstring yes,@enable_utils@),utils/try-summary utils/try-commit) man/try.1.gz
+TARGETS=utils/make-socket $(if $(findstring yes,@enable_utils@),utils/try-summary utils/try-commit) man/try.1.gz
 
 all: $(TARGETS)
 
 install: $(TARGETS)
 	$(INSTALL) -d $(bindir)
 	$(INSTALL) -m 755 try $(bindir)
+	$(INSTALL) -m 755 utils/make-socket $(bindir)
 ifeq (@enable_utils@, yes)
 	$(INSTALL) -m 755 utils/try-summary $(bindir)
 	$(INSTALL) -m 755 utils/try-commit $(bindir)
@@ -58,7 +59,10 @@ utils/try-summary: utils/ignores.o utils/try-summary.o
 utils/try-commit: utils/ignores.o utils/try-commit.o
 	$(CC) $(CPPFLAGS) $(CFLAGS) -o $@ -g $^
 
-test: try $(if $(findstring yes,@enable_utils@),utils/try-summary utils/try-commit)
+utils/make-socket: utils/make-socket.o
+	$(CC) $(CPPFLAGS) $(CFLAGS) -o $@ -g $^
+
+test: try $(if $(findstring yes,@enable_utils@),utils/try-summary utils/try-commit) utils/make-socket
 	scripts/run_tests.sh
 
 lint:

Vagrantfile

@@ -13,9 +13,16 @@ Vagrant.configure("2") do |config|
       sudo apt-get update
       sudo apt-get install -y git expect curl attr pandoc gcc make autoconf mergerfs
       sudo chown -R vagrant:vagrant try
+      sudo modprobe overlay
       cd try
+
+      autoconf && ./configure --disable-utils
+      make
+      sudo make install
       scripts/run_tests.sh
 
+      sudo make clean
+
       autoconf && ./configure && make
       sudo make install
       which try-commit || exit 2
@@ -78,10 +85,15 @@ Vagrant.configure("2") do |config|
       # This is intentional, if we moved try to lv1 it'd work since itself does not contain a nested mount
       sudo mv /home/vagrant/try /mnt/lv0
       sudo chown -R vagrant:vagrant /mnt/lv0/try
+      sudo modprobe overlay
 
       cd /mnt/lv0/try
+      autoconf && ./configure --disable-utils && make
+      sudo make install
       scripts/run_tests.sh
 
+      sudo make clean
+
       autoconf && ./configure && make
       sudo make install
       which try-commit || exit 2
@@ -97,6 +109,7 @@ Vagrant.configure("2") do |config|
       sudo apt-get update
       sudo apt-get install -y curl attr pandoc gcc make autoconf mergerfs zsh
       sudo chown -R vagrant:vagrant try
+      sudo modprobe overlay
       cd try
 
 
@@ -149,8 +162,18 @@ Vagrant.configure("2") do |config|
       wget https://github.com/trapexit/mergerfs/releases/download/2.40.2/mergerfs-2.40.2-1.el9.x86_64.rpm
       sudo rpm -i mergerfs-2.40.2-1.el9.x86_64.rpm
       sudo chown -R vagrant:vagrant try
+      sudo modprobe overlay
       cd try
+
+      autoconf && ./configure --disable-utils && make
+      sudo make install
+
+      autoconf && ./configure --disable-utils && make
+      sudo make install
       TRY_TOP=$(pwd) scripts/run_tests.sh
+
+      sudo make clean
+
       autoconf && ./configure && make
       sudo make install
       which try-commit || exit 2
@@ -167,8 +190,15 @@ Vagrant.configure("2") do |config|
       wget https://github.com/trapexit/mergerfs/releases/download/2.40.2/mergerfs-2.40.2-1.fc39.x86_64.rpm
       sudo rpm -i mergerfs-2.40.2-1.fc39.x86_64.rpm
       sudo chown -R vagrant:vagrant try
+      sudo modprobe overlay
       cd try
+
+      autoconf && ./configure --disable-utils && make
+      sudo make install
       TRY_TOP=$(pwd) scripts/run_tests.sh
+
+      sudo make clean
+
       autoconf && ./configure && make
       sudo make install
       which try-commit || exit 2

configure.ac

@@ -10,21 +10,27 @@ AC_ARG_ENABLE([utils],
                               [don't compile C utilities for summarizing and committing changes (default is yes)])],
               [enable_utils=${enableval}], [enable_utils=yes])
 
+
+AC_REQUIRE_AUX_FILE([utils/make-socket.c])
+
+# build tools
+AC_PROG_CC
+
+# CFLAGS AND CPPFLAGGS
+if test -z "$CFLAGS"
+then
+        AUTO_CFLAGS="-g -Wall -O2"
+        AUTO_CPPFLAGS="-g -Wall -O2"
+else
+        AUTO_CFLAGS=""
+        AUTO_CPPFLAGS=""
+fi
+
 if test "$enable_utils" = "yes"
 then
         AC_REQUIRE_AUX_FILE([utils/try-commit.c])
 
-        # build tools
-        AC_PROG_CC
-
-        # CFLAGS and CPPFLAGGS
-	if test -z "$CFLAGS"
-	then
-	        AUTO_CFLAGS="-g -Wall -O2"
-	else
-	        AUTO_CFLAGS=""
-	fi
-	AUTO_CPPFLAGS=""
+ 	AUTO_CPPFLAGS=""
 
 	CFLAGS=${CFLAGS-"$AUTO_CFLAGS"}
 	CPPFLAGS=${CPPFLAGS-"$AUTO_CPPFLAGS"}

package.nix

@@ -16,12 +16,7 @@ stdenv.mkDerivation {
   pname = "try";
   version = "latest";
 
-  src = fetchFromGitHub {
-    owner = "binpash";
-    repo = "try";
-    rev = "67052d8f20725f3cdc22ffaec33f7b7c14f1eb6b";
-    hash = "sha256-8mfCmqN50pRAeNTJUlRVrRQulWon4b2OL4Ug/ygBhB0=";
-  };
+  src = ./.;
 
   # skip TRY_REQUIRE_PROG as it detects executable dependencies by running it
   postPatch = ''
@@ -40,6 +35,7 @@ stdenv.mkDerivation {
     install -Dt $out/bin try
     install -Dt $out/bin utils/try-commit
     install -Dt $out/bin utils/try-summary
+    install -Dt $out/bin utils/make-socket
     wrapProgram $out/bin/try --prefix PATH : ${
       lib.makeBinPath [
         coreutils

scripts/check_trycase.sh

@@ -38,7 +38,7 @@ check_file() {
         lf="${lf%)}"
 
         case "$cf" in
-            (file|dir|symlink|nonexist|opaque|whiteout);;
+            (file|dir|symlink|nonexist|opaque|whiteout|fifo|socket);;
             (*) printf "ERROR: $1: invalid changed file: %s\n" "$cf"
                 : $((ERRORS += 1))
                 ;;

test/all-commit-cases.sh

@@ -5,12 +5,11 @@ TRY_TOP="${TRY_TOP:-$(git rev-parse --show-toplevel --show-superproject-working-
 TRY="$TRY_TOP/try"
 
 cleanup() {
-    cd /
+  cd /
 
-    if [ -d "$try_workspace" ]
-    then
-        rm -rf "$try_workspace" >/dev/null 2>&1
-    fi
+  if [ -d "$try_workspace" ]; then
+    rm -rf "$try_workspace" >/dev/null 2>&1
+  fi
 }
 
 trap 'cleanup' EXIT
@@ -25,11 +24,10 @@ cd "$try_workspace" || exit 99
 
 COUNT=0
 fail() {
-    echo Case $COUNT: "$@"
-    exit ${COUNT}
+  echo Case $COUNT: "$@"
+  exit ${COUNT}
 }
 
-
 # // TRYCASE(opaque, dir)
 # // TRYCASE(dir, dir)
 
@@ -224,3 +222,82 @@ echo arrivederci >formerdir/it/file2 || fail
 [ -L formerdir ] || fail
 [ "$(readlink formerdir)" = "/this/is/a/broken/symlink" ] || fail
 rm formerdir || fail
+
+# // TRYCASE(fifo, file)
+
+: $((COUNT += 1))
+
+! [ -e newpipe ] || fail
+"$TRY" -y "mkfifo newpipe; rm newpipe; touch newpipe; echo new >newpipe"
+[ -f newpipe ] || fail
+[ "$(cat newpipe)" = "new" ] || fail
+rm newpipe || fail
+
+# // TRYCASE(fifo, dir)
+
+: $((COUNT += 1))
+
+! [ -e newpipe ] || fail
+"$TRY" -y "mkfifo newpipe; rm newpipe; mkdir newpipe"
+[ -d newpipe ] || fail
+rm -r newpipe
+
+# // TRYCASE(fifo, symlink)
+
+: $((COUNT += 1))
+
+! [ -e newpipe ] || fail
+ln -s "$TRY" newpipe
+[ -L newpipe ] || fail
+"$TRY" -y "rm newpipe; mkfifo newpipe"
+[ -p newpipe ] || fail
+rm newpipe
+
+# // TRYCASE(fifo, nonexist)
+
+: $((COUNT += 1))
+
+! [ -e newpipe ] || fail
+"$TRY" -y "mkfifo newpipe"
+[ -p newpipe ] || fail
+rm newpipe
+
+# // TRYCASE(socket, file)
+
+: $((COUNT += 1))
+
+! [ -e newsock ] || fail
+"$TRY" -y "make-socket newsock; rm newsock; touch newsock; echo hello> newsock"
+[ -f newsock ] || fail
+[ "$(cat newsock)" = "hello" ] || fail
+rm newsock
+
+# // TRYCASE(socket, dir)
+
+: $((COUNT += 1))
+
+! [ -e newsock ] || fail
+"$TRY" -y "make-socket newsock; rm newsock; mkdir newsock"
+[ -d newsock ] || fail
+rm -r newsock
+
+# // TRYCASE(socket, symlink)
+
+: $((COUNT += 1))
+
+! [ -e newsock ] || fail
+ln -s "$TRY" newsock
+[ -L newsock ] || fail
+"$TRY" -y "rm newsock; make-socket newsock"
+! [ -e newlink ] || fail
+[ -S newsock ] || fail
+rm newsock
+
+# // TRYCASE(socket, nonexist)
+
+: $((COUNT += 1))
+
+! [ -e newsock ]
+"$TRY" -y "make-socket newsock"
+[ -S newsock ] || fail
+rm newsock