From 7ac36fd8db2a39a7e2528eec6debe29b675908a0 Mon Sep 17 00:00:00 2001
From: Florian Klink <flokli@flokli.de>
Date: Fri, 28 Oct 2022 16:26:24 +0200
Subject: [PATCH] chore(tvix/proto/castore.proto): rename link to symlinks

Make it clear these are symlinks, not hardlinks.

The term "link" is too heavily correlated to other meanings in IPFS/IPLD
world, and calling this symlink removes this confusion.

Change-Id: Id3f1eaa32098510b05f3e1a1348344503bcb4d5a
Reviewed-on: https://cl.tvl.fyi/c/depot/+/7129
Tested-by: BuildkiteCI
Reviewed-by: tazjin <tazjin@tvl.su>
Reviewed-by: Adam Joseph <adam@westernsemico.com>
---
 tvix/proto/castore.proto | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/tvix/proto/castore.proto b/tvix/proto/castore.proto
index b47959582..ec0266035 100644
--- a/tvix/proto/castore.proto
+++ b/tvix/proto/castore.proto
@@ -5,7 +5,7 @@ syntax = "proto3";
 
 package fossil.store;
 
-// A Directory can contain Directory, File or Link nodes.
+// A Directory can contain Directory, File or Symlink nodes.
 // Each of these nodes have a name attribute, which is the basename in that directory
 // and node type specific attributes.
 // The name attribute:
@@ -16,7 +16,7 @@ package fossil.store;
 message Directory {
     repeated DirectoryNode directories = 1;
     repeated FileNode files = 2;
-    repeated LinkNode links = 3;
+    repeated SymlinkNode symlinks = 3;
 }
 
 // A DirectoryNode represents a directory in a Directory.
@@ -26,7 +26,7 @@ message DirectoryNode {
     // The blake3 hash of a Directory message, serialized in protobuf canonical form.
     bytes ref = 2;
     // Number of child elements in the Directory referred to by ref.
-    // Calculated by summing up the numbers of `directories`, `files` and `links`,
+    // Calculated by summing up the numbers of `directories`, `files` and `symlinks`,
     // and for each directory, its size field. Used for inode number calculation
     // TODO(flokli): These numbers can't really be verified until you know the whole graph.
     //               Is this a problem? Should we instead only have this field
@@ -46,8 +46,8 @@ message FileNode {
     bool executable = 4;
 }
 
-// A LinkNode represents a symbolic link in a Directory.
-message LinkNode {
+// A SymlinkNode represents a symbolic link in a Directory.
+message SymlinkNode {
     // The (base)name of the symlink
     string name = 1;
     // The target of the symlink.