From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Sassmannshausen Subject: Re: [PATCH 01/26] gnu: Add perl-file-pushd. Date: Sat, 25 Mar 2017 17:53:37 +0100 Message-ID: <87a889wb5a.fsf@gmail.com> References: <20170323140056.23173-1-alex@pompo.co> <87lgrv2wht.fsf@openmailbox.org> <87bmsrks7s.fsf@gmail.com> <87inmy4x6c.fsf@gmail.com> <87h92i3g4s.fsf@openmailbox.org> Reply-To: alex.sassmannshausen@gmail.com Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:55344) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1crowh-0006qc-UH for guix-devel@gnu.org; Sat, 25 Mar 2017 12:53:45 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1crowe-0003sw-R4 for guix-devel@gnu.org; Sat, 25 Mar 2017 12:53:43 -0400 Received: from mail-wm0-x241.google.com ([2a00:1450:400c:c09::241]:34041) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1crowe-0003sG-KX for guix-devel@gnu.org; Sat, 25 Mar 2017 12:53:40 -0400 Received: by mail-wm0-x241.google.com with SMTP id u132so3505316wmg.1 for ; Sat, 25 Mar 2017 09:53:40 -0700 (PDT) In-reply-to: <87h92i3g4s.fsf@openmailbox.org> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Kei Kebreau Cc: guix-devel@gnu.org Heya, Thank you all for your advice! I can confirm that the builds were successful locally. On the back of this I will push to master later this evening. Cheers, Alex Kei Kebreau writes: > Alex Sassmannshausen writes: > >> Hi Ben, >> >> Ben Woodcroft writes: >> >>> Hi, >>> >>> >>> On 24/03/17 18:13, Alex Sassmannshausen wrote: >>>> Hi Kei, >>>> >>>> Kei Kebreau writes: >>>> >>>>> This series of packages builds and lints fine for me. There are over 200 >>>>> dependent packages that would have to be rebuilt though, and >>>>> core-updates is frozen to my knowledge. We can save this for the next >>>>> core-updates cycle, though! >>>>> >>>>> Other Guix users, please correct me if I'm wrong. >>> According to the established guidelines, this should be OK for master >>> since it affects <300 packages, so if all dependent packages build then >>> I don't see why we shouldn't do that. >>> >>> https://lists.gnu.org/archive/html/guix-devel/2016-10/msg00933.html >> >> Cheers for your comments, very enlightening! I'm afraid I'm out of my >> depth making this decision here. I'm happy to let my computer rebuild >> those 200 packages to see whether the dependent packages build — but >> I'm not sure how I would go about testing that. See below for a theory…? >> >>>> Thanks for the review! >>>> >>>> What command do you use to check the number of packages that would have >>>> to be rebuilt? I'm asking, because if they are mainly perl libraries >>>> then that figure of 200 really isn't that big of a deal: most perl libs >>>> build super fast. >>> To check the number of packages use "guix refresh -l" >> >> Interesting… so to establish the packages to be rebuilt as a result of >> my patch series, would that be: >> >> guix refresh -l perl-scalar-list-utils perl-parse-cpan-meta \ >> perl-cpan-meta-requirements perl-yaml perl-variable-magic \ >> perl-time-duration-parse perl-test-warnings perl-test-simple \ >> perl-test-exception perl-test-cleannamespaces perl-sub-name \ >> perl-params-validate perl-package-deprecationmanager perl-moose \ >> perl-module-runtime-conflicts perl-devel-partialdump \ >> perl-devel-overloadinfo perl-cpan-meta-check perl-common-sense \ >> perl-clone perl-class-load perl-capture-tiny perl-b-hooks-endofscope >> >> Building the following 24 packages would ensure 248 dependent packages >> are rebuilt: edirect-4.10 perl-modern-perl-1.20150127 >> perl-text-neattemplate-0.1101 perl-encode-detect-1.01 >> perl-list-someutils-0.52 perl-db-file-1.838 pam-krb5-4.7 >> perl-test-trailingspace-0.0300 sslh-1.18 perl-mail-spf-v2.9.0 >> gerbv-2.6.1 netsurf-3.6 perl-config-ini-0.025 perl-anyevent-i3-0.16 >> tophat-2.1.0 perl-test-class-most-0.08 gnucash-2.6.15 surfraw-2.2.9 >> biber-2.5 biber-next-2.6 stow-2.2.2 perl-x11-xcb-0.16 roary-3.7.0 >> hydra-20151030.1ff48da >> >> Does this mean, that in order to test this I would now simply try to >> rebuild the 24 listed packages? > > Yes. Just make sure you use `./pre-inst-env guix build` so that you're > testing the updated perl packages. Thanks! > >> >> Cheers, >> >> Alex