Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 17 May 2018 08:46:14 +0200
From:      Mateusz Guzik <mjguzik@gmail.com>
To:        Eitan Adler <lists@eitanadler.com>
Cc:        FreeBSD Hackers <freebsd-hackers@freebsd.org>
Subject:   Re: including a checksum of the diff in uname
Message-ID:  <CAGudoHFth_%2BfySpC%2BPda3MDTFypn_GcsFE7tJEmY2Y3ZASTHsg@mail.gmail.com>
In-Reply-To: <CAF6rxg=SYbDAAcoc=XRswL%2BW-zr=3MLyA3-gYn9hee6rKMmXoA@mail.gmail.com>
References:  <CAF6rxgmL559nLMT8WeEQhscHjVgO8O0Nr-sA7Jf5pvtUKk7T_Q@mail.gmail.com> <CAGudoHEupAa0gk03KYwt=AHrWXAjdwS5tjqW1A5NV92sULO-qw@mail.gmail.com> <CAF6rxg=SYbDAAcoc=XRswL%2BW-zr=3MLyA3-gYn9hee6rKMmXoA@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, May 17, 2018 at 8:37 AM, Eitan Adler <lists@eitanadler.com> wrote:

> On 12 May 2018 at 19:50, Mateusz Guzik <mjguzik@gmail.com> wrote:
> > On Sun, May 13, 2018 at 4:41 AM, Eitan Adler <lists@eitanadler.com>
> wrote:
> >>
> >> Hi all,
> >>
> >> I often have several variants of the same revision that I'm testing or
> >> working on at any given time.
> >>
> >> What do you y'all think of something like this:
> >>
> >> Index: sys/conf/newvers.sh
> >> ===================================================================
> >> --- sys/conf/newvers.sh (revision 333575)
> >> +++ sys/conf/newvers.sh (working copy)
> >> @@ -203,6 +203,8 @@ if [ -n "$svnversion" ] ; then
> >>   [0-9]*[MSP]|*:*)
> >>   svn=" r${svn}"
> >>   modified=true
> >> + ck="$(svn diff | cksum | cut -w -f 1)"
> >> + svn="$svn-$ck"
> >>   ;;
> >>   [0-9]*)
> >>   svn=" r${svn}"
> >>
> >> to help differentiate.
> >>
> >> You end up with a version that looks like: r333575M-4294967295?
> >>
> >
> > Don't develop with svn. get yourself a git or hg setup and a tmp branch.
> > The branch name is always included along with the commit hash.
>
> That makes sense, and I might do that in the future. In the past
> adding files was more painful with git, though I'm told this is no
> longer the case.
>
> In any case, is there any reason not to add the above patch?
>
>
I don't see any added value of doing so. So far it just looks like
patching up a deficient development process.

-- 
Mateusz Guzik <mjguzik gmail.com>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGudoHFth_%2BfySpC%2BPda3MDTFypn_GcsFE7tJEmY2Y3ZASTHsg>