On Sat, Nov 30, 2024 at 11:55:33AM +0100, Oswald Buddenhagen via isync-devel
wrote:
> On Fri, Nov 29, 2024 at 06:21:45PM -0500, Paymon MARANDI wrote:
> > any feedback?
> >
> yeah, sorry, i've been procrastinating and then it went out of view ...
>
> i don't like it, because it makes the version info useless, which would
> make worse log files. adding --always to the describe command seems less
> disruptive.
oh, didn't know about that one.
> the use case also feels a bit silly - given that the worktree is over
> half a meg and the build tree several times that much, shrinking the
> pack from just 2 megs to 200k seems mostly pointless.
i updated the log, hopefully will add proper justification.
--
Paymon
>From 4885752d0b8b56da90b609e10891aeaa363b51ad Mon Sep 17 00:00:00 2001
From: Paymon <[email protected]>
Date: Tue, 3 Dec 2024 15:25:09 -0500
Subject: [PATCH] build: also consider builds off of git with `git clone
--depth 1`
one case this could happen is a shallow clone, purely for build
purposes. in source based distros, like gentoo, setting depth of the
clone to 1, globally, is a common configuration. this will avoid those
builds to fail.
Signed-off-by: Paymon MARANDI <[email protected]>
---
version.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/version.sh b/version.sh
index fc1ed7d..5dfc61b 100755
--- a/version.sh
+++ b/version.sh
@@ -22,7 +22,7 @@ else
# find out whether we have local work, and if so, collapse it into
# a single suffix. otherwise, we'd cause pointless rebuilds during
# development.
- gver=$(git describe --tags $mb)
+ gver=$(git describe --always --tags $mb)
lcl=$(git rev-list -n 1 $mb..HEAD)
if test -n "$lcl"; then
gver="$gver-plus"
--
2.47.1
_______________________________________________
isync-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/isync-devel
