> Seems like a pretty niche use case, but if you can come up with a
> patch that doesn't break too much of current behaviour - why not.
Thank you for considering it. A patch is in the attachment.
> Worst case, it could be an option (that defaulted to false) ?
I would not consider this the worst case. For me, it would be still
more convenient than keeping my own modified version around.
Andreas
>From b7acbbf1c61b137d10904ed8c4e3e2cbf27ddb6f Mon Sep 17 00:00:00 2001
From: Andreas Wettstein <wetts...@gmail.com>
Date: Sat, 16 Sep 2017 20:24:27 +0200
Subject: [PATCH] When computing a relative path with respect to the current
working directory, use the original path, without making it canonical.
Signed-off-by: Andreas Wettstein <wetts...@gmail.com>
---
ccache.c | 22 +++++++---------------
1 file changed, 7 insertions(+), 15 deletions(-)
diff --git a/ccache.c b/ccache.c
index edbc5cc..2e605b3 100644
--- a/ccache.c
+++ b/ccache.c
@@ -709,23 +709,15 @@ make_relative_path(char *path)
free(p);
}
- char *canon_path = x_realpath(path);
- if (canon_path) {
- free(path);
- char *relpath = get_relative_path(get_current_working_dir(), canon_path);
- free(canon_path);
- if (path_suffix) {
- path = format("%s/%s", relpath, path_suffix);
- free(relpath);
- free(path_suffix);
- return path;
- } else {
- return relpath;
- }
- } else {
- // path doesn't exist, so leave it as it is.
+ char *relpath = get_relative_path(get_current_working_dir(), path);
+ free(path);
+ if (path_suffix) {
+ path = format("%s/%s", relpath, path_suffix);
+ free(relpath);
free(path_suffix);
return path;
+ } else {
+ return relpath;
}
}
--
2.14.1
_______________________________________________
ccache mailing list
ccache@lists.samba.org
https://lists.samba.org/mailman/listinfo/ccache
