From owner-freebsd-arch@freebsd.org Sat Aug 18 04:43:49 2018 Return-Path: Delivered-To: freebsd-arch@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id CA94410857F3 for ; Sat, 18 Aug 2018 04:43:49 +0000 (UTC) (envelope-from lists@eitanadler.com) Received: from mail-yw1-xc34.google.com (mail-yw1-xc34.google.com [IPv6:2607:f8b0:4864:20::c34]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 524F581B7F for ; Sat, 18 Aug 2018 04:43:49 +0000 (UTC) (envelope-from lists@eitanadler.com) Received: by mail-yw1-xc34.google.com with SMTP id n207-v6so46832ywn.9 for ; Fri, 17 Aug 2018 21:43:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=eitanadler.com; s=0xdeadbeef; h=mime-version:references:in-reply-to:from:date:message-id:subject:to; bh=qA7e2ZeiSx0xC7aMMT0q1gl3uxleXSlPTyu7Sh74bXc=; b=YTlzJLiCkmzslXSMjcPJN/yoALKTSSWTeTRO2QIjuGxkLc1BvofnuKDT0RWLNwVN24 IneDTWGE4cwn/cq+4rz7rwFcBGHBDcVmF+XuF/5U7/+cz24JL2lEwurJ/zA2VHPLid1n 10jzPxSQazQBxjZ1wZAZ275e1Wz1gXD3Njqm8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to; bh=qA7e2ZeiSx0xC7aMMT0q1gl3uxleXSlPTyu7Sh74bXc=; b=iDeCKH7zf3ut0jWN6Y0VcXOUrMv164l5pph7smtrLktPqTqTQccf9tdY3sUs9mS8jM UcLU00WDlhrujf3wX2WvBv6MKjmiKLZnYrsPV9MwtMpNjirgcmvcxygswZnao+2JCnsV 36uNUidn7l/Xv9E+z76NYS4b6cjRhLk7UGwxNWE01zUQfAfbwripu1xyHw1iIdemGiAc eegL14DExTAiQsCIotaus3BoKhWos0oLL7kY+rspaYslOXjH0tRZjXIawMcPdiXPCdMp KYZLPKI85iWVCJ8QW/qHSkOFwvEM1Ruu/kftHWolDg8i5yb1ki7kx4bfP9PNYNexIe6X ThcA== X-Gm-Message-State: AOUpUlFQHE+8FgkUDkXEJMIUNvY41KhyNlc/sd/IUlNKsOrdnmARe9Ku qPcDQsIZbIckAbKp8wztpxCymAb/fTptg5AtjMIVCRETPO/LTQ== X-Google-Smtp-Source: AA+uWPzSh/lWt5ihotI9qHnqlEuJQm+MaUuTRVpAAloabArEgHvefW7wsQuCl2nXB3FNI+YfLWhJ7+u7kYGvyAJEWbA= X-Received: by 2002:a81:b248:: with SMTP id q69-v6mr3207534ywh.38.1534567427952; Fri, 17 Aug 2018 21:43:47 -0700 (PDT) MIME-Version: 1.0 References: <20a85a8f-29a7-0d8f-64d1-9ba005ffe79c@FreeBSD.org> <10e35d23-37d8-edf4-fa3b-9663bfdaa629@FreeBSD.org> <7C8DB02A-8C15-4917-A941-A10DD2F3AB50@trouble.is> <8B61C2A5-DCEC-403A-B8F3-0B5BEF958612@freebsd.org> <20180812233252.GD30769@rincewind.trouble.is> In-Reply-To: <20180812233252.GD30769@rincewind.trouble.is> From: Eitan Adler Date: Fri, 17 Aug 2018 21:43:21 -0700 Message-ID: Subject: Re: upstream for contrib/tzcode/stdtme? To: "freebsd-arch@freebsd.org" Content-Type: text/plain; charset="UTF-8" X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.27 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 18 Aug 2018 04:43:50 -0000 On Sun, 12 Aug 2018 at 16:32, Philip Paeps wrote: > > On 2018-08-12 15:09:00 (-0700), Eitan Adler wrote: > >On Sun, 12 Aug 2018 at 06:00, Philip Paeps wrote: > >> > >> On 2018-08-12 20:36:23 (+0800), Philip Paeps wrote: > >> > >> > On 2018-08-12 20:23:23 (+0800), Eitan Adler wrote: > >> > > >> >> On Mon, 18 Jun 2018 at 10:08, John Baldwin wrote: > >> >>> > >> >>> On 6/15/18 4:09 PM, Eitan Adler wrote: > >> >>>> On 15 June 2018 at 11:31, John Baldwin wrote: > >> >>>> > >> >>>>> I think this second approach is actually a better plan and is not > >> >>>>> quite what you > >> >>>>> were suggesting. > >> >>>>> > >> >>>>> That is, import the vendor bits corresponding to our existing code > >> >>>>> first into > >> >>>>> the vendor area, > >> >>>> > >> >>>>> then svn cp our existing files from libc, zic, etc. into the > >> >>>>> contrib/stdtime tree in the correct layout and do a single 'svn > >> >>>>> merge --record-only' > >> >>>>> merge to initialize the vendor mergeinfo. > >> >>>>> At that point you should be able to > >> >>>>> svn diff contrib/stdtime against the vendor/stdtime/dist to > >> >>>>> determine what local > >> >>>>> changes we have and start to think about them. > >> >> > >> >> Coming back to this. In r337683 and r337684 I imported a recent copy > >> >> of of TZDB. I was unable to find a copy of the 2010n distribution in > >> >> the original layout. > >> > > >> > That's because we split up the tzcode in our vendor area (for reasons > >> > I've never understood). As far as I know, there is no 1:1 mapping > >> > from any tzcode distribution to what we have in our repository. > >> > > >> >> The next step, if I understand, is to do the following: > >> >> cd /srv/srv/freebsd/svn/head/contrib > >> >> mkdir tzdb > >> >> cd tzdb > >> >> svn cp ../tzcode/stdtime/* . > >> >> svn cp ../tzcode/zic/* . > >> >> svn cp ../tzdata/* . > >> >> svn ci > >> >> (the above ignores duplicated files, but that's just expanding the > >> >> wildcards appropriately). > >> >> > >> >> After that: > >> >> svn merge --record-only '^/vendor/tzdb' . > >> >> > >> >> At this point we'll be able to diff contrib/tzdb and vendor/tzdb to > >> >> show the most current vendor code compared our, modified old code. > >> >> > >> >> Is this correct? Is this the optimal plan? > > > >... > >> please import tzcode separately and leave tzdata alone. > > > >Done in r337693 and r337694. > > Thank you! > > For the next tzdata import, I'll consider moving vendor/tzdata to > vendor/tzdb/tzdata before doing the copies to contrib/. > > >The next step, if I understand, is to do the following: > >cd /srv/srv/freebsd/svn/head/contrib > >mkdir tzdb > >cd tzdb > >svn cp ../tzcode/stdtime/* . > >svn cp ../tzcode/zic/* . > >svn ci > >(the above ignores duplicated files, but that's just expanding the > >wildcards appropriately). > > > >After that: > >svn merge --record-only '^/vendor/tzdb' . > > > >At this point we'll be able to diff contrib/tzdb and vendor/tzdb to > >show the most current vendor code compared our, modified old code. > > > >Is this correct? Is this the optimal plan? > > I would keep contrib/tzcode (and contrib/tzdata) rather than moving to > contrib/tzdb. Having them together under vendor/ makes sense because > it's the same vendor, but under contrib, they're definitely separate. I wanted to use a separate directory to make it clearler what's clean and what's not but lets go with the current directory. cd /srv/srv/freebsd/svn/head/ cd contrib/tzcode svn mv stdtime/* . svn cp zic/* . cd ../.. $EDITOR usr.sbin/zic/zic/Makefile $EDITOR usr.sbin/zic/zdump/Makefile $EDITOR lib/libc/stdtime/Makefile.inc svn ci svn merge --record-only '^/vendor/tzcode' . svn ci This will leave contrib with the old code and vendor with the new code and mergeinfo claiming we're fully merged. Is that the state of affairs that we want? I don't know subversion well enough to formulate a better plan. At the end of this, what I'd like to do is see what the diff is, merge the latest vendor code, and ensure that any local patches we still require remain applied. -- Eitan Adler